Yaniv Bronhaim has uploaded a new change for review. Change subject: Verify cleaning of supervdsm subprocesses after terminated to avoid defunct ......................................................................
Verify cleaning of supervdsm subprocesses after terminated to avoid defunct After killing subprocess register its pid to zombieRepear to verify that svdsm process will wait and clear the pid when the process is terminated. Change-Id: I21837f6f707e5e6f64fbb6968ac25294622bd22b Signed-off-by: Yaniv Bronhaim <[email protected]> --- M vdsm/supervdsmServer.py 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/36/9236/1 diff --git a/vdsm/supervdsmServer.py b/vdsm/supervdsmServer.py index 34fefdb..0979ba0 100755 --- a/vdsm/supervdsmServer.py +++ b/vdsm/supervdsmServer.py @@ -30,6 +30,7 @@ from multiprocessing import Pipe, Process import storage.misc as misc from vdsm import utils +import zombieReaper try: from gluster import cli as gcli @@ -199,6 +200,7 @@ res, err = pipe.recv() pipe.send("Bye") proc.terminate() + zombieReaper.autoReapPID(proc.pid) if err is not None: raise err -- To view, visit http://gerrit.ovirt.org/9236 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I21837f6f707e5e6f64fbb6968ac25294622bd22b Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
