Change in vdsm[master]: Added required ipwrapper functions for iproute2 configurator

danken at redhat.com danken at redhat.com
Fri Aug 30 11:10:31 UTC 2013


Dan Kenigsberg has posted comments on this change.

Change subject: Added required ipwrapper functions for iproute2 configurator
......................................................................


Patch Set 3: Code-Review-1

(1 comment)

....................................................
File lib/vdsm/ipwrapper.py
Line 265:     command = [_IP_BINARY.cmd, 'addr', 'flush', 'dev', dev]
Line 266:     _execCmd(command)
Line 267: 
Line 268: 
Line 269: def linkAdd(name, link=None, linkType=None, args=()):
Darn, only a partial fix of Mark's comment. should make linkType compulsory, too.
Line 270:     command = [_IP_BINARY.cmd, 'link', 'add']
Line 271:     if link:
Line 272:         command += ['link', link]
Line 273: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9283ae90f84fa930e401b3ecd2e18935ba9ca2f5
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Assaf Muller <amuller at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Assaf Muller <amuller at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw at linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list