Change in vdsm[master]: netinfo: improvements from using Link devices
asegurap at redhat.com
asegurap at redhat.com
Fri Nov 22 00:19:21 UTC 2013
Antoni Segura Puimedon has posted comments on this change.
Change subject: netinfo: improvements from using Link devices
......................................................................
Patch Set 2:
(3 comments)
....................................................
File lib/vdsm/netinfo.py
Line 93: return res
Line 94:
Line 95:
Line 96: def vlans():
Line 97: return [link.name for link in getLinks() if link.isVLAN() and not
Consistency with the upper one. I agree with not near the operand. Will change it on line 73 instead.
Line 98: link.isHidden()]
Line 99:
Line 100:
Line 101: def bridges():
Line 533:
Line 534: for net, netAttr in networks().iteritems():
Line 535: try:
Line 536: d['networks'][net] = _getNetInfo(netAttr.get('iface', net),
Line 537: netAttr['bridged'],
Will put that back (artifact of taking pieces from a bigger patch :P)
Line 538: gateways, ipv6routes,
Line 539: netAttr.get('qosInbound'),
Line 540: netAttr.get('qosOutbound'))
Line 541: except KeyError:
....................................................
File tests/netinfoTests.py
Line 149: '35: fake: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc '
Line 150: 'pfifo_fast state UP mode DEFAULT group default qlen 1000\\ '
Line 151: ' link/ether ff:aa:f1:da:bb:e7 brd ff:ff:ff:ff:ff:ff '
Line 152: 'promiscuity 0 \\ dummy ',
Line 153: '419: jbond: <BROADCAST,MULTICAST,MASTER,UP,LOWER_UP> mtu '
Well... It's so that the lines match what the system would really be in the tests below. It kinda bothered me when I added bond getting from the links and I realized that in fact they where not on the "system" ;-)
Line 154: '1500 qdisc noqueue state UP mode DEFAULT group default \ '
Line 155: 'link/ether 66:de:f1:da:aa:e7 brd ff:ff:ff:ff:ff:ff '
Line 156: 'promiscuity 1 \ bond')
Line 157: return [ipwrapper.Link.fromText(line) for line in lines]
--
To view, visit http://gerrit.ovirt.org/21408
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I7a51c0d3bcec1358521846238e0cbd4c13ae1d17
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <asegurap 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: Nir Soffer <nsoffer at redhat.com>
Gerrit-HasComments: Yes
More information about the vdsm-patches
mailing list