Change in ovirt-engine[master]: webadmin: Unneeded Vdsm call when editing iSCSI SD

[derez at redhat.com]

Daniel Erez has submitted this change and it was merged.

Change subject: webadmin: Unneeded Vdsm call when editing iSCSI SD
......................................................................


webadmin: Unneeded Vdsm call when editing iSCSI SD

When editing an existing iSCSI, we need to get from Vdsm the list of
available LUNs.

However, due to multiple events triggered, we are sending the VDSM query
more than needed.

In this patch, we are minimizing the handling of the events triggered and
therefore the queries from 6 to 2 in the edit SD flow.

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

Approvals:
  Jenkins CI: Passed CI tests
  Daniel Erez: Looks good to me, approved
  Freddy Rolland: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic6f5db4f74efaad05a6b84e14d12a700efc27b50
Gerrit-PatchSet: 4
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: Fred Rolland <frolland at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>