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 <[email protected]>
---
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 <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Gustavo Frederico Temple Pedrosa 
<[email protected]>
Gerrit-Reviewer: Leonardo Bianconi <[email protected]>
Gerrit-Reviewer: Liran Zelkha <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Vitor de Lima <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to