Change in vdsm[master]: ipwrapper: fix version to family parameter name

asegurap at redhat.com asegurap at redhat.com
Mon May 26 08:44:01 UTC 2014


Antoni Segura Puimedon has uploaded a new change for review.

Change subject: ipwrapper: fix version to family parameter name
......................................................................

ipwrapper: fix version to family parameter name

In commit e435e680fdae8f1baaf8dac549c72db32c07549f there is a change
to add ipv6 support to various ipwrapper functions. During the
review process it was suggested to change the parameter from
'version' to 'family' but I missed one of the parameters.

Change-Id: I2ee57be88cb0037a337bfa28aa296cd01c8799a7
Signed-off-by: Antoni S. Puimedon <asegurap at redhat.com>
---
M lib/vdsm/ipwrapper.py
1 file changed, 2 insertions(+), 2 deletions(-)


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

diff --git a/lib/vdsm/ipwrapper.py b/lib/vdsm/ipwrapper.py
index c8d6b36..632c6eb 100644
--- a/lib/vdsm/ipwrapper.py
+++ b/lib/vdsm/ipwrapper.py
@@ -509,8 +509,8 @@
     return _execCmd(command)
 
 
-def routeAdd(route, version=4, dev=None):
-    command = [_IP_BINARY.cmd, '-%s' % version, 'route', 'add']
+def routeAdd(route, family=4, dev=None):
+    command = [_IP_BINARY.cmd, '-%s' % family, 'route', 'add']
     command += route
     if dev is not None:
         command += ['dev', dev]


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2ee57be88cb0037a337bfa28aa296cd01c8799a7
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <asegurap at redhat.com>


More information about the vdsm-patches mailing list