From 18c24f7c7c27ac732c4a760caa9524e7319cd47e Mon Sep 17 00:00:00 2001 From: Yaniv Bronhaim Date: Tue, 29 Jan 2013 13:49:46 +0200 Subject: [PATCH 27/27] removing the use of zombie reaper from supervdsm This may solve validateAccess errors, but can cause defuct subprocesses. This patch is signed as WIP until we'll find better solution, until then this patch helps to verify if the previous errors that was caused thanks to zombie reaper handling don't occur. Change-Id: If3f9bae47f2894cc95785de8f19f6ec388ea58da Signed-off-by: Yaniv Bronhaim Reviewed-on: http://gerrit.ovirt.org/11491 Reviewed-by: Dan Kenigsberg Reviewed-by: Federico Simoncelli Tested-by: Federico Simoncelli --- vdsm/supervdsmServer.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/vdsm/supervdsmServer.py b/vdsm/supervdsmServer.py index 833e91f..21e7c94 100755 --- a/vdsm/supervdsmServer.py +++ b/vdsm/supervdsmServer.py @@ -56,7 +56,6 @@ import tc import ksm import mkimage from storage.multipath import MPATH_CONF -import zombieReaper _UDEV_RULE_FILE_DIR = "/etc/udev/rules.d/" _UDEV_RULE_FILE_PREFIX = "99-vdsm-" @@ -199,7 +198,6 @@ class _SuperVdsm(object): pipe, hisPipe = Pipe() proc = Process(target=child, args=(hisPipe,)) proc.start() - zombieReaper.autoReapPID(proc.pid) if not pipe.poll(RUN_AS_TIMEOUT): try: @@ -391,8 +389,6 @@ def main(): if os.path.exists(address): os.unlink(address) - zombieReaper.registerSignalHandler() - log.debug("Setting up keep alive thread") monThread = threading.Thread(target=__pokeParent, -- 1.8.1