Change in ovirt-engine[master]: backend: replace network name with vdsm name when needed

Code Review gerrit at ovirt.org
Fri Aug 18 05:32:17 UTC 2017


>From Alona Kaplan <alkaplan at redhat.com>:

Alona Kaplan has submitted this change and it was merged. ( https://gerrit.ovirt.org/80469 )

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, 36 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/80469
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I93fca4940deae3bff497c85fcaf46e1a87f07c02
Gerrit-PatchSet: 12
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Leon Goldberg <lgoldber at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Dominik Holler <dholler 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: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list