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

iheim at redhat.com iheim at redhat.com
Wed Mar 19 15:04:17 UTC 2014


Itamar Heim 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_0690_change_storage_server_connections_password_length.sql
2 files changed, 7 insertions(+), 1 deletion(-)

Approvals:
  Allon Mureinik: Looks good to me, approved
  Francesco Romani: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibe5dcb58dd445216b43bbce63b342b96d5e225e6
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.4
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.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: Itamar Heim <iheim at redhat.com>
Gerrit-Reviewer: Leonardo Bianconi <leonardo.bianconi at eldorado.org.br>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Vitor de Lima <vitor.lima at eldorado.org.br>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list