Change in vdsm[master]: net: fix _emergencyNetworkCleanup

ibarkan at redhat.com ibarkan at redhat.com
Thu Sep 3 07:55:28 UTC 2015


Ido Barkan has uploaded a new change for review.

Change subject: net: fix _emergencyNetworkCleanup
......................................................................

net: fix _emergencyNetworkCleanup

The code assumed that 'bridged' was always a part of the network
attributes. It should default to this parameter default if it is not
there.

Change-Id: Ib6fb1217e7ec106211900c8c7b49e76a333cb469
Signed-off-by: Ido Barkan <ibarkan at redhat.com>
---
M vdsm/network/api.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/03/45703/1

diff --git a/vdsm/network/api.py b/vdsm/network/api.py
index 5b3a905..12cac64 100755
--- a/vdsm/network/api.py
+++ b/vdsm/network/api.py
@@ -713,7 +713,7 @@
         vlan_name = '%s.%s' % (topNetDev.name, networkAttrs['vlan'])
         if vlan_name in _netinfo.vlans:
             topNetDev = Vlan(topNetDev, networkAttrs['vlan'], configurator)
-    if networkAttrs['bridged']:
+    if networkAttrs.get('bridged', True):
         if network in _netinfo.bridges:
             topNetDev = Bridge(network, configurator, port=topNetDev)
 


-- 
To view, visit https://gerrit.ovirt.org/45703
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib6fb1217e7ec106211900c8c7b49e76a333cb469
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan <ibarkan at redhat.com>


More information about the vdsm-patches mailing list