Igor Lvovsky has uploaded a new change for review.

Change subject: Allow break bond with attached network while VM is running.
......................................................................

Allow break bond with attached network while VM is running.

The problem was that we counted tap device on the bridge as part of bond
because of wrong behaviour of getNicsVlanAndBondingForNetwork.

Change-Id: I183e2f4cf780a0eb79a416cdc4afc387893ee106
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=875487
Signed-off-by: Igor Lvovsky <[email protected]>
---
M vdsm/netinfo.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/84/9384/1

diff --git a/vdsm/netinfo.py b/vdsm/netinfo.py
index b6d1a1f..80b0925 100644
--- a/vdsm/netinfo.py
+++ b/vdsm/netinfo.py
@@ -488,7 +488,7 @@
                 assert bonding is None
                 bonding = port
                 lnics += self.bondings[bonding]['slaves']
-            else:
+            elif port in self.nics:
                 lnics.append(port)
 
         return lnics, vlan, bonding


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I183e2f4cf780a0eb79a416cdc4afc387893ee106
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to