Change in ovirt-engine[master]: webadmin: Wrong validation on update SAN SD

amureini at redhat.com amureini at redhat.com
Sun Jun 19 14:55:21 UTC 2016


Allon Mureinik has submitted this change and it was merged.

Change subject: webadmin: Wrong validation on update SAN SD
......................................................................


webadmin: Wrong validation on update SAN SD

If the user edits an existing SAN storage domain, and changes only the
name, a wrong and uneeded check is done on all LUNs of the SD.

In this patch, the validation is done only in case the user had extended
the SD by adding additional LUNs.

Change-Id: Id2085e95c577ffab016f0695706d4f970172e038
Bug-Url: https://bugzilla.redhat.com/1347974
Signed-off-by: Fred Rolland <frolland at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
1 file changed, 4 insertions(+), 0 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Allon Mureinik: Looks good to me, approved
  Freddy Rolland: Verified



-- 
To view, visit https://gerrit.ovirt.org/59442
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id2085e95c577ffab016f0695706d4f970172e038
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list