Bug 841486
| Summary: | [vdsm] super vdsm server leave child process defunct after create vm | ||
|---|---|---|---|
| Product: | [oVirt] vdsm | Reporter: | Royce Lv <lvroyce> |
| Component: | General | Assignee: | Oved Ourfali <oourfali> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Petr Kubica <pkubica> |
| Severity: | medium | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | 4.14.0 | CC: | abaron, adevolder, agkesos, bazulay, bugs, danken, dyasny, gklein, mgoldboi, oourfali, rbalakri, shyu, s.kieske, ybronhei, ykaul, ylavi |
| Target Milestone: | ovirt-3.6.0-rc | Keywords: | Reopened |
| Target Release: | 4.17.0 | Flags: | rule-engine:
ovirt-3.6.0+
ylavi: planning_ack+ rule-engine: devel_ack+ rule-engine: testing_ack+ |
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | infra | ||
| Fixed In Version: | 3.6.0-4 | Doc Type: | Bug Fix |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2016-01-13 14:40:24 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | Infra | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
| Bug Depends On: | 1180864, 1181624, 1210347 | ||
| Bug Blocks: | |||
|
Description
Royce Lv
2012-07-19 06:59:56 UTC
Closing old bugs. If this issue is still relevant/important in current version, please re-open the bug. please reopen, as I have defunct supervdsm process in ovirt 3.3.3 again. See discussion on devel: http://lists.ovirt.org/pipermail/devel/2014-May/007289.html Starting to use zombiereaper may solve this issue, but there's another one in supervdsm: it uses multiprocessing, which uses subprocess.Popen, which is known to be buggy on python2. Please consider monkey-patching multiprocessing.process.Process._Popen = CPopen before use. any progress regarding this problem? I still see supervdsmServer defunct processes popping up. installed version is atm: rpm -q vdsm vdsm-4.13.3-3.el6.x86_64 if you need any additional logs, please tell me. is this just about replacing subprocess.Popen with CPopen? It would be nice if someone could explain where the exact problem is, which prevents this from getting fixed, if it's just time to crawl through vdsm code, I'm happy to assist. (In reply to Sven Kieske from comment #4) > is this just about replacing subprocess.Popen with CPopen? No. That's an unrelated issue that I've noticed while reading the code. Note that my suggestion for consideration is wrong, as multiprocessing.forking.Popen does not have the same API as CPopen and subprocess.Popen. I suppose that a properly-placed zombiereaper.autoReapPID(proc.pid) would take care of your zombies. We have to revert the patch from the 3.5 branch, as it makes the much more annoying Bug 1168217 more evident. Adding dependency on Bug 1180864 which its solution allows to use zombiereaper without worries in supervdsmServer. After backport multiprocessing fix to python 2.6 which allows to handle SIGCHILD interuppts, we'll be able to merge http://gerrit.ovirt.org/#/c/28915/ back will merge when rhel 6.7 be out (See Bug 1180864). moving to 3.6 The former patches have been reverted when we realized that Python still had the EINTR bug. They must be re-posted. Bug tickets that are moved to testing must have target release set to make sure tester knows what to test. Please set the correct target release before moving to ON_QA. Verified in vdsm-4.17.15-0.el7ev.noarch oVirt 3.6.0 has been released, closing current release |