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 <osvob...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Ondřej Svoboda <osvob...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches