Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[3.7] bpo-38546: Backport multiprocessing tests fixes from master #19689

Merged
merged 4 commits into from
Apr 23, 2020
Merged

[3.7] bpo-38546: Backport multiprocessing tests fixes from master #19689

merged 4 commits into from
Apr 23, 2020

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Apr 23, 2020 •
edited by bedevere-bot
Loading

miss-islington and others added 3 commits April 23, 2020 23:23
@miss-islington @vstinner
bpo-37421: multiprocessing tests call _run_finalizers() (GH-14527)
90cc64e 
multiprocessing tests now call explicitly _run_finalizers() to remove
immediately temporary directories created by
multiprocessing.util.get_temp_dir().
(cherry picked from commit 039fb49)

Co-authored-by: Victor Stinner <[email protected]>
(cherry picked from commit 632cb36)
@miss-islington @vstinner
bpo-37421: multiprocessing tests now stop ForkServer (GH-14601)
87804b5 
multiprocessing tests now stop the ForkServer instance if it's
running: close the "alive" file descriptor to ask the server to stop
and then remove its UNIX address.
(cherry picked from commit 8fbeb14)

Co-authored-by: Victor Stinner <[email protected]>
(cherry picked from commit 229f6e8)
@vstinner
bpo-38546: multiprocessing tests stop the resource tracker (GH-17641) (…
53adc98 
…GH-17647)

Multiprocessing and concurrent.futures tests now stop the resource
tracker process when tests complete.

Add ResourceTracker._stop() method to
multiprocessing.resource_tracker.

Add _cleanup_tests() helper function to multiprocessing.util: share
code between multiprocessing and concurrent.futures tests.

(cherry picked from commit 9707e8e)
(cherry picked from commit 35acb35)
@vstinner vstinner changed the title [3.7] Backport multiprocessing tests fixes from master [3.7] bpo-38546: Backport multiprocessing tests fixes from master Apr 23, 2020
@vstinner vstinner closed this Apr 23, 2020
@vstinner vstinner reopened this Apr 23, 2020
@vstinner
Copy link
Member Author

I closed/reopened the PR to re-run the Azure Pipelines CI: its Ubuntu job failed at downloading https://www.openssl.org/source/openssl-1.1.1f.tar.gz :-(

@vstinner
Remove NEWS about resource tracker
43be998 
Python 3.7 multiprocessing does not have resource tracker.
@vstinner vstinner merged commit fd32a0e into python:3.7 Apr 23, 2020
@vstinner vstinner deleted the mp_cleanup_tests37 branch April 23, 2020 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@vstinner @the-knights-who-say-ni @bedevere-bot @miss-islington