Antoni Segura Puimedon has uploaded a new change for review.

Change subject: Simplify removeVlan by removing duplicated code.
......................................................................

Simplify removeVlan by removing duplicated code.

Change-Id: If5f7f7f60796a4d22f3c945253e6235d35af5cc1
Signed-off-by: Antoni S. Puimedon <[email protected]>
---
M vdsm/configNetwork.py
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/17/12017/1

diff --git a/vdsm/configNetwork.py b/vdsm/configNetwork.py
index 963528c..2470419 100755
--- a/vdsm/configNetwork.py
+++ b/vdsm/configNetwork.py
@@ -540,8 +540,8 @@
     def removeVlan(self, vlan, iface):
         vlandev = iface + '.' + vlan
         ifdown(vlandev)
-        self._backup(netinfo.NET_CONF_PREF + iface + '.' + vlan)
-        self._removeFile(netinfo.NET_CONF_PREF + iface + '.' + vlan)
+        self._backup(netinfo.NET_CONF_PREF + vlandev)
+        self._removeFile(netinfo.NET_CONF_PREF + vlandev)
 
     def removeBonding(self, bonding):
         self._backup(netinfo.NET_CONF_PREF + bonding)


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

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