Francesco Romani has posted comments on this change.
Change subject: network.api: EL7's dhclient doesn't support -df so stop using it
temporarily
......................................................................
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/45713/2/vdsm/network/configurators/dhclient.py
File vdsm/network/configurators/dhclient.py:
Line 137:
Line 138:
Line 139: @memoized
Line 140: def supports_duid_file():
Line 141: probe = DhcpClient('-invalid-option') # dhclient doesn't have
-h/--help
I wouldn't be able to branch on the return code (it would be 0
once --help
fair enough!
Line 142: rc, out, err = probe.start(blocking=True)
Line 143: if rc:
Line 144: for line in err:
Line 145: if '-df' in line:
--
To view, visit
https://gerrit.ovirt.org/45713
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia096d42d24b00e7ef075f1a2dde7e3a951c0b81c
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ondřej Svoboda <osvoboda(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Ido Barkan <ibarkan(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com>
Gerrit-Reviewer: Ondřej Svoboda <osvoboda(a)redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: Yes