Ondřej Svoboda has uploaded a new change for review.

Change subject: ifcfg: stop using IpConfig's namedtuple altogether
......................................................................

ifcfg: stop using IpConfig's namedtuple altogether

Change-Id: If84d90b6c69c4f87a17464c395f0c61c3976f49b
Signed-off-by: Ondřej Svoboda <osvob...@redhat.com>
---
M vdsm/network/configurators/ifcfg.py
1 file changed, 5 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/95/37795/1

diff --git a/vdsm/network/configurators/ifcfg.py 
b/vdsm/network/configurators/ifcfg.py
index c273a82..da45d5a 100644
--- a/vdsm/network/configurators/ifcfg.py
+++ b/vdsm/network/configurators/ifcfg.py
@@ -89,13 +89,13 @@
         if bridge.port:
             bridge.port.configure(**opts)
         self._addSourceRoute(bridge)
-        ifup(bridge.name, bridge.ipConfig.async)
+        ifup(bridge.name, bridge.ip.async)
 
     def configureVlan(self, vlan, **opts):
         self.configApplier.addVlan(vlan, **opts)
         vlan.device.configure(**opts)
         self._addSourceRoute(vlan)
-        ifup(vlan.name, vlan.ipConfig.async)
+        ifup(vlan.name, vlan.ip.async)
 
     def configureBond(self, bond, **opts):
         self.configApplier.addBonding(bond, **opts)
@@ -105,7 +105,7 @@
         for slave in bond.slaves:
             slave.configure(**opts)
         self._addSourceRoute(bond)
-        ifup(bond.name, bond.ipConfig.async)
+        ifup(bond.name, bond.ip.async)
         if self.unifiedPersistence:
             self.runningConfig.setBonding(
                 bond.name, {'options': bond.options,
@@ -158,7 +158,7 @@
         if nic.bond is None:
             if not netinfo.isVlanned(nic.name):
                 ifdown(nic.name)
-            ifup(nic.name, nic.ipConfig.async)
+            ifup(nic.name, nic.ip.async)
 
     def removeBridge(self, bridge):
         DynamicSourceRoute.addInterfaceTracking(bridge)
@@ -616,7 +616,7 @@
         opts['hotplug'] = 'no'  # So that udev doesn't trigger an ifup
         if bridge.stp is not None:
             conf += 'STP=%s\n' % ('on' if bridge.stp else 'off')
-        if not self.unifiedPersistence or bridge.ipConfig.defaultRoute:
+        if not self.unifiedPersistence or bridge.ip.ipv4.defaultRoute:
             conf += 'ONBOOT=%s\n' % 'yes'
         else:
             conf += 'ONBOOT=%s\n' % 'no'


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If84d90b6c69c4f87a17464c395f0c61c3976f49b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ondřej Svoboda <osvob...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to