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(a)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(a)redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>