From 18c24f7c7c27ac732c4a760caa9524e7319cd47e Mon Sep 17 00:00:00 2001
From: Yaniv Bronhaim <ybronhei@redhat.com>
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 <ybronhei@redhat.com>
Reviewed-on: http://gerrit.ovirt.org/11491
Reviewed-by: Dan Kenigsberg <danken@redhat.com>
Reviewed-by: Federico Simoncelli <fsimonce@redhat.com>
Tested-by: Federico Simoncelli <fsimonce@redhat.com>
---
 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