Antoni Segura Puimedon has uploaded a new change for review.

Change subject: [WIP] Keep nic link up after delNetwork.
......................................................................

[WIP] Keep nic link up after delNetwork.

First version of the patch that will solve the linked
bug. Now it is far too general, as it leaves all the
interfaces up.

TODO: Only ifup if the interface was previously up.

Change-Id: I9bc0e6e5c7bc92c8ff3aa007572ee461ea69c366
Bug-Url: https://bugzilla.redhat.com/691484
Signed-off-by: Antoni S. Puimedon <[email protected]>
---
M vdsm/configNetwork.py
1 file changed, 3 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/43/8443/1

diff --git a/vdsm/configNetwork.py b/vdsm/configNetwork.py
index 9256609..d08a895 100755
--- a/vdsm/configNetwork.py
+++ b/vdsm/configNetwork.py
@@ -1111,6 +1111,7 @@
             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
@@ -1203,6 +1204,7 @@
             for nic in _netinfo.getNicsForBonding(bond):
                 ifdown(nic)
                 configWriter.removeNic(nic)
+                ifup(nic)
 
         # Note! In case we have bridge up and connected to the bond
         # we will get error in log:
@@ -1241,6 +1243,7 @@
             for nic in nics:
                 ifdown(nic)
                 configWriter.removeNic(nic)
+                ifup(nic)
 
             del bondings[bond]
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9bc0e6e5c7bc92c8ff3aa007572ee461ea69c366
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to