Change in vdsm[master]: BZ#848299 - Fix delete VLANed network with MTU=9000

ilvovsky at redhat.com ilvovsky at redhat.com
Wed Aug 15 13:49:31 UTC 2012


Igor Lvovsky has uploaded a new change for review.

Change subject: BZ#848299 - Fix delete VLANed network with MTU=9000
......................................................................

BZ#848299 - Fix delete VLANed network with MTU=9000

Change-Id: I485c8bd3dfa84c421dab2c5d121ad7a5efc348a3
Signed-off-by: Igor Lvovsky <ilvovsky at redhat.com>
---
M vdsm/configNetwork.py
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/14/7214/1

diff --git a/vdsm/configNetwork.py b/vdsm/configNetwork.py
index 9dffcdd..1f494c5 100755
--- a/vdsm/configNetwork.py
+++ b/vdsm/configNetwork.py
@@ -551,8 +551,8 @@
             return
 
         if bonding:
-            # FIXME! Check whether we need to care of bridgeless too
-            _, vlans = _netinfo.getBridgedNetworksAndVlansForIface(bonding)
+            iface_bridged = _netinfo.getBridgedNetworksAndVlansForIface(bonding)
+            vlans = [v for (_, v) in iface_bridged]
             delvlan = bonding + '.' + delvlan
         else:
             vlans = _netinfo.getVlansForNic(nics[0])
@@ -562,7 +562,7 @@
         for vlan in vlans:
             if vlan == delvlan:
                 continue
-            cf = self.NET_CONF_PREF + vlan
+            cf = self.NET_CONF_PREF + delvlan
             mtu = self._getConfigValue(cf, 'MTU')
             newmtu = max(newmtu, mtu)
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I485c8bd3dfa84c421dab2c5d121ad7a5efc348a3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky <ilvovsky at redhat.com>


More information about the vdsm-patches mailing list