Edward Haas has posted comments on this change. Change subject: net: Introduce dhclient state check. ......................................................................
Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/61298/1/lib/vdsm/network/ip/dhclient.py File lib/vdsm/network/ip/dhclient.py: Line 101: return True Line 102: return False Line 103: Line 104: Line 105: def pid_lookup(device_name, family): > no real need to expose this, right? Done Line 106: for pid in pgrep('dhclient'): Line 107: try: Line 108: with open('/proc/%s/cmdline' % pid) as cmdline: Line 109: args = cmdline.read().strip('\0').split('\0') -- To view, visit https://gerrit.ovirt.org/61298 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia9941556e40dbeb1da206907f23b5c77e5358d7e Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Edward Haas <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Edward Haas <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Petr Horáček <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/admin/lists/[email protected]
