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

piotr.kliczewski at gmail.com piotr.kliczewski at gmail.com
Mon Jun 20 10:50:33 UTC 2016


Piotr Kliczewski 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/59446
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id2085e95c577ffab016f0695706d4f970172e038
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list