Change in vdsm[master]: netinfo: Modernize functions recognizing the usage of DHCP f...
osvoboda at redhat.com
osvoboda at redhat.com
Sat Jan 10 13:42:45 UTC 2015
Ondřej Svoboda has posted comments on this change.
Change subject: netinfo: Modernize functions recognizing the usage of DHCP from lease files
......................................................................
Patch Set 9: Verified+1
(1 comment)
Tests except testSetupNetworksAddDelDhcp passed.
The failing one is struck by:
AssertionError: routing rule from 240.0.0.0/24 table 4026531913 found
(in assertRuleDoesNotExist)
http://gerrit.ovirt.org/#/c/36038/9/lib/vdsm/netinfo.py
File lib/vdsm/netinfo.py:
Line 543:
Line 544: return dhcpv4_ifaces
Line 545:
Line 546:
Line 547: def _get_dhclient_ifaces(lease_files_globs):
> The ability was never there. I don't need the 'ipv6' parameter _parseLeaseF
The implementation is here, starting at line 513.
http://gerrit.ovirt.org/#/c/35985/12/lib/vdsm/netinfo.py
netinfo: Assess and report 'dhcpv6' from dhclient -6 leases, with tests
Line 548: """Return a set of interfaces configured using dhclient.
Line 549:
Line 550: dhclient stores DHCP leases to file(s) whose names can be specified
Line 551: by the lease_files_globs parameter (an iterable of glob strings).
--
To view, visit http://gerrit.ovirt.org/36038
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I5b95d0223b35c3bbbcd736c28ed1dd7c2f9bd2bb
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
More information about the vdsm-patches
mailing list