Skip to content

bpo-37224: To fix the random failed test cases of test___xxsubinterpreters in multiprocess. #27240

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
Merged
merged 1 commit into from
Jul 29, 2021
Merged

bpo-37224: To fix the random failed test cases of test___xxsubinterpreters in multiprocess. #27240

merged 1 commit into from
Jul 29, 2021

Conversation

shihai1991
Copy link
Member

@shihai1991 shihai1991 commented Jul 19, 2021 •
edited by bedevere-bot
Loading

@bedevere-bot bedevere-bot added the tests Tests in the Lib/test dir label Jul 19, 2021
@shihai1991 shihai1991 changed the title bpo_37224: To fix the random failed test cases of test___xxsubinterpreters in multiprocess. bpo-37224: To fix the random failed test cases of test___xxsubinterpreters in multiprocess. Jul 19, 2021
@shihai1991
Copy link
Member Author

I haven't catch the error again in my vm over 29 hours by ./python -m test test__xxsubinterpreters -v -F -j300.

@shihai1991 shihai1991 mentioned this pull request Jul 19, 2021
Closed
@shihai1991
Copy link
Member Author

shihai1991 commented Jul 20, 2021 •
edited
Loading

Hi, Eric, Pablo. Would you mind to take a look this RP again? Thanks. @ericsnowcurrently @pablogsal
cc @vstinner
And my analysis process in: https://bugs.python.org/issue37224#msg397809

@ambv ambv merged commit 9101b39 into python:main Jul 29, 2021
@ambv ambv added the needs backport to 3.10 only security fixes label Jul 29, 2021
@miss-islington
Copy link
Contributor

Thanks @shihai1991 for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 29, 2021
@shihai1991 @miss-islington
To fix the random failed test cases of test___xxsubinterpreters in mu…
701e096 
…ltiprocess. (pythonGH-27240)

(cherry picked from commit 9101b39)

Co-authored-by: Hai Shi <shihai1992@gmail.com>
@bedevere-bot
Copy link

GH-27452 is a backport of this pull request to the 3.10 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Jul 29, 2021
@ambv ambv added the needs backport to 3.9 only security fixes label Jul 29, 2021
@miss-islington
Copy link
Contributor

Thanks @shihai1991 for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

@bedevere-bot
Copy link

GH-27453 is a backport of this pull request to the 3.9 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.9 only security fixes label Jul 29, 2021
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 29, 2021
@shihai1991 @miss-islington
To fix the random failed test cases of test___xxsubinterpreters in mu…
1b2201e 
…ltiprocess. (pythonGH-27240)

(cherry picked from commit 9101b39)

Co-authored-by: Hai Shi <shihai1992@gmail.com>
ambv pushed a commit that referenced this pull request Jul 29, 2021
@miss-islington @shihai1991
To fix the random failed test cases of test___xxsubinterpreters in mu…
a438b37 
…ltiprocess. (GH-27240) (GH-27452)

(cherry picked from commit 9101b39)

Co-authored-by: Hai Shi <shihai1992@gmail.com>
ambv pushed a commit that referenced this pull request Jul 29, 2021
@miss-islington @shihai1991
To fix the random failed test cases of test___xxsubinterpreters in mu…
e627c79 
…ltiprocess. (GH-27240) (GH-27453)

(cherry picked from commit 9101b39)

Co-authored-by: Hai Shi <shihai1992@gmail.com>
@shihai1991
Copy link
Member Author

Thanks Łukasz for your merge :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vstinner vstinner Awaiting requested review from vstinner

@ericsnowcurrently ericsnowcurrently Awaiting requested review from ericsnowcurrently

@pablogsal pablogsal Awaiting requested review from pablogsal

Assignees
No one assigned
Labels
skip news tests Tests in the Lib/test dir
Projects
None yet
Milestone
No milestone
Development

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