Change in vdsm[master]: Fix defaultRoute handling

amuller at redhat.com amuller at redhat.com
Tue Feb 18 10:41:43 UTC 2014


Assaf Muller has uploaded a new change for review.

Change subject: Fix defaultRoute handling
......................................................................

Fix defaultRoute handling

defaultRoute wasn't being past correctly to objectivizeNetwork,
so that defaultRoute was always None inside the function,
no matter what was past to it.

This meant that DEFROUTE=yes/no was never written to ifcfg files,
and 'ip route add default...' never applied by the iproute2
configurator.

Change-Id: I643c5fe9dab39833d73541b7a1bfaae9ae3e749d
Signed-off-by: Assaf Muller <amuller at redhat.com>
---
M vdsm/configNetwork.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/31/24631/1

diff --git a/vdsm/configNetwork.py b/vdsm/configNetwork.py
index 7a2206a..5bf9838 100755
--- a/vdsm/configNetwork.py
+++ b/vdsm/configNetwork.py
@@ -279,7 +279,7 @@
     netEnt = objectivizeNetwork(network if bridged else None, vlan, bonding,
                                 bondingOptions, nics, mtu, ipaddr, netmask,
                                 gateway, bootproto, ipv6addr, ipv6gateway,
-                                defaultRoute, _netinfo=_netinfo,
+                                defaultRoute=defaultRoute, _netinfo=_netinfo,
                                 configurator=configurator, **options)
 
     netEnt.configure(**options)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I643c5fe9dab39833d73541b7a1bfaae9ae3e749d
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Assaf Muller <amuller at redhat.com>


More information about the vdsm-patches mailing list