Change in vdsm[master]: Allow to remove NIC if it not in use

ilvovsky at redhat.com ilvovsky at redhat.com
Tue Nov 27 12:31:32 UTC 2012


Igor Lvovsky has uploaded a new change for review.

Change subject: Allow to remove NIC if it not in use
......................................................................

Allow to remove NIC if it not in use

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


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/08/9508/1

diff --git a/vdsm/configNetwork.py b/vdsm/configNetwork.py
index 09d70ac..b188860 100755
--- a/vdsm/configNetwork.py
+++ b/vdsm/configNetwork.py
@@ -1133,12 +1133,12 @@
             configWriter.removeBonding(bonding)
             iface = None if bonding == iface else iface
 
-        for nic in nics:
-            if not nicOtherUsers(network, vlan, bonding, nic):
-                ifdown(nic)
-                configWriter.removeNic(nic)
-                ifup(nic)
-                iface = None if nic == iface else iface
+    for nic in nics:
+        if not nicOtherUsers(network, vlan, bonding, nic):
+            ifdown(nic)
+            configWriter.removeNic(nic)
+            ifup(nic)
+            iface = None if nic == iface else iface
 
     # Now we can restart changed interface
     if iface:


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

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