Change in vdsm[master]: move _netinfo and libvirt_nets decleration right before it's...

ibarkan at redhat.com ibarkan at redhat.com
Thu Nov 27 12:11:05 UTC 2014


Ido Barkan has uploaded a new change for review.

Change subject: move _netinfo and libvirt_nets decleration right before it's used
......................................................................

move _netinfo and libvirt_nets decleration right before it's used

Change-Id: I8b754fb5a3927fbfccd5aa17dd7bde2eeb7b256e
Signed-off-by: ibarkan <ibarkan at redhat.com>
---
M vdsm/network/api.py
1 file changed, 4 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/55/35655/1

diff --git a/vdsm/network/api.py b/vdsm/network/api.py
index 470cc97..bae03e7 100755
--- a/vdsm/network/api.py
+++ b/vdsm/network/api.py
@@ -643,11 +643,6 @@
     """
     logger = logging.getLogger("setupNetworks")
 
-    libvirt_nets = netinfo.networks()
-    _netinfo = netinfo.NetInfo(_netinfo=netinfo.get(
-        netinfo._libvirtNets2vdsm(libvirt_nets)))
-    networksAdded = set()
-
     logger.debug("Setting up network according to configuration: "
                  "networks:%r, bondings:%r, options:%r" % (networks,
                                                            bondings, options))
@@ -666,6 +661,10 @@
     bondings = results['request']['bondings']
     options = results['request']['options']
 
+    libvirt_nets = netinfo.networks()
+    _netinfo = netinfo.NetInfo(_netinfo=netinfo.get(
+        netinfo._libvirtNets2vdsm(libvirt_nets)))
+    networksAdded = set()
     logger.debug("Applying...")
     with ConfiguratorClass(options.get('_inRollback', False)) as configurator:
         # Remove edited networks and networks with 'remove' attribute


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8b754fb5a3927fbfccd5aa17dd7bde2eeb7b256e
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