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

iheim at redhat.com iheim at redhat.com
Wed Dec 4 14:52:47 UTC 2013


Itamar Heim 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:
  Itamar Heim: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3d94298b0bd29be423d0b1b90c2174630de96092
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.3
Gerrit-Owner: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim at redhat.com>
Gerrit-Reviewer: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list