TwistedLayer/SIGCHLD handler confusion
Bug #399118 reported by
Michael Hudson-Doyle
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Triaged
|
High
|
Unassigned | ||
Bug Description
We have test cases in the TwistedLayer that launch subprocesses with reactor.
We also have test cases in the TwistedLayer that use tachandler to launch subprocesses. These should not run with a sigchld handler installed, because subprocess.py is lame.
I'm not sure what we should do to fix this. Perhaps robustifying tachandler against EINTR really is the right thing (even though I've sort of argued against that in the past).
Revision history for this message
| Jonathan Lange (jml) wrote | #1 |
| tags: | added: tech-debt |
| Changed in launchpad: | |
| importance: | Medium → High |
| tags: | added: build-infrastructure |
To post a comment you must log in.
Robustifying tachandler against EINTR is the right thing to do, imho.