Change in vdsm[master]: supervdsm: Switch multiprocessing to CPopen

dkuznets at redhat.com dkuznets at redhat.com
Wed Jun 18 16:50:40 UTC 2014


Dima Kuznetsov has uploaded a new change for review.

Change subject: supervdsm: Switch multiprocessing to CPopen
......................................................................

supervdsm: Switch multiprocessing to CPopen

Default implementation of multiprocessing.Process used Popen to create
processes, this resulted in zombies being present after the process was
done. This change monkeypatches Process to use cpopen.

Change-Id: I5333caf7a1162c5d783f556bb9204fb74d1ab469
Bug-url: https://bugzilla.redhat.com/show_bug.cgi?id=841486
Signed-off-by: Dima Kuznetsov <dkuznets at redhat.com>
---
M vdsm/supervdsmServer
1 file changed, 2 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/15/28915/1

diff --git a/vdsm/supervdsmServer b/vdsm/supervdsmServer
index f9395eb..0a88a21 100755
--- a/vdsm/supervdsmServer
+++ b/vdsm/supervdsmServer
@@ -44,7 +44,9 @@
     log.warn("Could not init proper logging", exc_info=True)
 
 from storage import fuser
+import cpopen
 from multiprocessing import Pipe, Process
+Process._Popen = cpopen.CPopen
 try:
     from gluster import listPublicFunctions
     _glusterEnabled = True


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5333caf7a1162c5d783f556bb9204fb74d1ab469
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov <dkuznets at redhat.com>


More information about the vdsm-patches mailing list