Change in vdsm[master]: clean dhclient processes that are left after functional tests

ibarkan at redhat.com ibarkan at redhat.com
Thu Oct 23 11:13:37 UTC 2014


Ido Barkan has posted comments on this change.

Change subject: clean dhclient processes that are left after functional tests
......................................................................


Patch Set 3:

It's not clear on the documentation which pid file is referenced when issuing the dhclient -x command and you have multiple dhclient instances on your system. 

The lease, pid and conf files should be cleared by the caller after he is done with the temporary dir given to dhclient runner.

-- 
To view, visit http://gerrit.ovirt.org/34366
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I458aa38415c697d3863e173444ff921d759166a2
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No


More information about the vdsm-patches mailing list