Change in ovirt-engine[master]: core: Fix validation of Updating attached network

masayag at redhat.com masayag at redhat.com
Sun Jul 8 12:45:28 UTC 2012


Moti Asayag has submitted this change and it was merged.

Change subject: core: Fix validation of Updating attached network
......................................................................


core: Fix validation of Updating attached network

The patch modifies the existing validation of network which about to be
modified:

1. The old network name is obtained from the engine database, rather getting
it from the clients. This prevents the possibility to perform 2-in-1
action (removing a network and updating and existing one).

2. The validation of existence of the old network name is removed since
the network is obtained from an interface reported by the host.

Change-Id: I25ab34d64cc4891dba176e1cd11d073f731d2d8a
Signed-off-by: Moti Asayag <masayag at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateNetworkToVdsInterfaceCommand.java
1 file changed, 24 insertions(+), 29 deletions(-)

Approvals:
  Moti Asayag: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I25ab34d64cc4891dba176e1cd11d073f731d2d8a
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>



More information about the Engine-commits mailing list