Alona Kaplan has uploaded a new change for review.

Change subject: engine: Remove migration role from old migration network
......................................................................

engine: Remove migration role from old migration network

Cluster can contain only one migration network.
On attach of a new migration network or update old network to become the
migration. The migration role should be deleted from the old migration
network in the cluster (if exists).

Change-Id: I3384311e374234eb7b9fd16dbd4d65d1d801f1e8
Signed-off-by: Alona Kaplan <alkap...@redhat.com>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/cluster/AttachNetworkToVdsGroupCommand.java
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/cluster/UpdateNetworkOnClusterCommand.java
2 files changed, 6 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/77/13177/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/cluster/AttachNetworkToVdsGroupCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/cluster/AttachNetworkToVdsGroupCommand.java
index c2793a2..6795b46 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/cluster/AttachNetworkToVdsGroupCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/cluster/AttachNetworkToVdsGroupCommand.java
@@ -57,6 +57,9 @@
         if (getNetwork().getCluster().isDisplay()) {
             
getNetworkClusterDAO().setNetworkExclusivelyAsDisplay(getVdsGroupId(), 
getNetwork().getId());
         }
+        if (getNetwork().getCluster().isMigration()) {
+            
getNetworkClusterDAO().setNetworkExclusivelyAsMigration(getVdsGroupId(), 
getNetwork().getId());
+        }
         NetworkClusterHelper.setStatus(getVdsGroupId(), getNetwork());
         setSucceeded(true);
     }
diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/cluster/UpdateNetworkOnClusterCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/cluster/UpdateNetworkOnClusterCommand.java
index 2fd5d16..0b2cf88 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/cluster/UpdateNetworkOnClusterCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/cluster/UpdateNetworkOnClusterCommand.java
@@ -59,6 +59,9 @@
         if (getNetworkCluster().isDisplay()) {
             
getNetworkClusterDAO().setNetworkExclusivelyAsDisplay(getVdsGroupId(), 
getNetwork().getId());
         }
+        if (getNetworkCluster().isMigration()) {
+            
getNetworkClusterDAO().setNetworkExclusivelyAsMigration(getVdsGroupId(), 
getNetwork().getId());
+        }
 
         NetworkClusterHelper.setStatus(getVdsGroupId(), getNetwork());
         setSucceeded(true);


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3384311e374234eb7b9fd16dbd4d65d1d801f1e8
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alona Kaplan <alkap...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to