Title: [2.7] Enhance support reap_children() and threading_cleanup()
Type: Stage: resolved
Components: Tests Versions: Python 2.7
Status: closed Resolution: rejected
Dependencies: Superseder:
Assigned To: Nosy List: vstinner
Priority: normal Keywords:

Created on 2017-08-22 14:36 by vstinner, last changed 2017-09-13 00:52 by vstinner. This issue is now closed.

Pull Requests
URL Status Linked Edit
PR 3182 closed vstinner, 2017-08-22 14:38
PR 3183 merged vstinner, 2017-08-22 14:39
PR 3184 merged vstinner, 2017-08-22 14:40
Messages (5)
msg300695 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2017-08-22 14:36
Backport reap_children() and threading_cleanup() enhancements from master to 2.7 to make tests more reliables, reduce random failures on Python CIs.
msg300696 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2017-08-22 14:48
Using PR 3182, test_socketserver fails with --fail-env-changed because of bpo-31151 "socketserver.ForkingMixIn.server_close() leaks zombie processes".
msg300698 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2017-08-22 14:51
New changeset 1460ce5a9325690d8a1913c296a9bba437ff0f39 by Victor Stinner in branch '2.7':
bpo-31258: test_signal: call waitpid() to prevent zombie process (#3183)
msg300699 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2017-08-22 14:51
New changeset 02e03672e6766b1da847b1635982a70346780744 by Victor Stinner in branch '2.7':
bpo-31258: test_urllib2_localnet uses addCleanup(server.stop) (#3184)
msg302018 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2017-09-13 00:52
While backporting fixes on tests make sense, I'm not sure that it's doable to make regrtest more strict. It took me a while to fix all "dangling threads" and "dangling processes" on the master branch, and 6 months later, I'm not done yet. There are tricky issues the socketserver module (bpo-31233, bpo-31151). So I prefer to abandon this issue, and only backport test fixes.
Date User Action Args
2017-09-13 00:52:24vstinnersetstatus: open -> closed
resolution: rejected
messages: + msg302018

stage: resolved
2017-08-22 15:07:03vstinnersetpull_requests: - pull_request3224
2017-08-22 15:05:24vstinnersetpull_requests: + pull_request3224
2017-08-22 14:51:22vstinnersetmessages: + msg300699
2017-08-22 14:51:11vstinnersetmessages: + msg300698
2017-08-22 14:48:34vstinnersetmessages: + msg300696
2017-08-22 14:40:57vstinnersetpull_requests: + pull_request3222
2017-08-22 14:39:07vstinnersetpull_requests: + pull_request3221
2017-08-22 14:38:05vstinnersetpull_requests: + pull_request3220
2017-08-22 14:36:08vstinnercreate