Change in ovirt-engine[ovirt-engine-4.1]: backend: replace network name with vdsm name when needed
Code Review
gerrit at ovirt.org
Mon Aug 21 07:59:53 UTC 2017
>From Tal Nisan <tnisan at redhat.com>:
Tal Nisan has submitted this change and it was merged. ( https://gerrit.ovirt.org/80813 )
Change subject: backend: replace network name with vdsm name when needed
......................................................................
backend: replace network name with vdsm name when needed
In a scenario where a network with a long name is removed or detached
from a cluster, detaching the network (which is now unmanaged) from
hosts will be impossible as mapping network name to vdsm name is no
longer possible.
To remedy this, on network removal, replace network_name in
vds_interface from network name to vdsm name in each interface
where the removed network is attached to in the datacenter or cluster.
Change-Id: I93fca4940deae3bff497c85fcaf46e1a87f07c02
Bug-Id: https://bugzilla.redhat.com/show_bug.cgi?id=1478790
Signed-off-by: Leon Goldberg <lgoldber at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/cluster/DetachNetworkFromClusterInternalCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/cluster/NetworkHelper.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/dc/RemoveNetworkCommand.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/network/InterfaceDao.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/network/InterfaceDaoImpl.java
5 files changed, 35 insertions(+), 1 deletion(-)
Approvals:
Alona Kaplan: Looks good to me, approved
Jenkins CI: Passed CI tests
Leon Goldberg: Verified
--
To view, visit https://gerrit.ovirt.org/80813
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I93fca4940deae3bff497c85fcaf46e1a87f07c02
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Leon Goldberg <lgoldber at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Leon Goldberg <lgoldber at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
More information about the Engine-commits
mailing list