Change in ovirt-engine[master]: webadmin: Created interdependence between profile actions

lvernia at redhat.com lvernia at redhat.com
Wed Dec 4 14:07:06 UTC 2013


Lior Vernia has submitted this change and it was merged.

Change subject: webadmin: Created interdependence between profile actions
......................................................................


webadmin: Created interdependence between profile actions

When creating multiple profiles from the Add/Edit Network dialog, the
backend actions were run using Frontend.RunMultipleAction() which
caused the validity of the actions to be checked
independently. However, the validity should be checked in an
interdependent manner (i.e. serially), because the completion of one
command can create conditions where another command is invalid
(e.g. if two commands try to create two vNIC profiles with the same
name). Solved by running Frontend.RunMultipleActions() instead.

Change-Id: I3d94298b0bd29be423d0b1b90c2174630de96092
Bug-Url: https://bugzilla.redhat.com/1031577
Signed-off-by: Lior Vernia <lvernia at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NetworkModel.java
1 file changed, 5 insertions(+), 5 deletions(-)

Approvals:
  Lior Vernia: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3d94298b0bd29be423d0b1b90c2174630de96092
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list