Change in ovirt-engine[master]: dbscripts: fix password field length

mlipchuk at redhat.com mlipchuk at redhat.com
Mon Jan 27 11:49:44 UTC 2014


Maor Lipchuk has submitted this change and it was merged.

Change subject: dbscripts: fix password field length
......................................................................


dbscripts: fix password field length

the password column in the 'storage_server_connections' table has
a different type with respect to the corresponding parameter of
the Insertstorage_server_connection function.

This trigger SQL exception doing the following:

- configure an ISCSI data domain (bug triggered on the first one,
  but it should behave the same with the others too).
- in the 'New domain' window, issue fill all the fields,
  most notably username and password, and do a manual discovery.
- try to add the target which shows up as result of the discovery.

This patch fixes this problem by enlarging the password field on
the table, in order to let it hold the encrypted password.

Bug-Url: https://bugzilla.redhat.com/1052839
Change-Id: Ibe5dcb58dd445216b43bbce63b342b96d5e225e6
Signed-off-by: Francesco Romani <fromani at redhat.com>
---
M packaging/dbscripts/storages_san_sp.sql
A packaging/dbscripts/upgrade/03_04_0590_change_storage_server_connections_password_length.sql
2 files changed, 7 insertions(+), 1 deletion(-)

Approvals:
  Francesco Romani: Verified
  Maor Lipchuk: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/23133
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibe5dcb58dd445216b43bbce63b342b96d5e225e6
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Eli Mesika <elimesika at gmail.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Gustavo Frederico Temple Pedrosa <gustavo.pedrosa at eldorado.org.br>
Gerrit-Reviewer: Leonardo Bianconi <leonardo.bianconi at eldorado.org.br>
Gerrit-Reviewer: Liran Zelkha <liran.zelkha at gmail.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Vitor de Lima <vitor.lima at eldorado.org.br>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list