Skip to content

gh-118433: Temporarily skip test_interrupt_main_subthread in free-threaded builds #118485

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.

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

Merged
merged 1 commit into from
May 1, 2024

Conversation

mpage
Copy link
Contributor

@mpage mpage commented May 1, 2024

Free-threaded builds can intermittently tickle a longstanding bug (24 years!) in the implementation of threading.Condition, leading to flakiness in the test suite. Fixing the underlying issue will require more discussion, and will likely apply to most of the concurrency primitives in the threading module that are written in Python. See gh-118433 for more details.

Disable the test in free-threaded builds as a temporary measure to eliminate flakiness.

Free-threaded builds can intermittently tickle a longstanding bug (24 years!)
in the implementation of `threading.Condition`, leading to flakiness in the
test suite. Fixing the underlying issue will require more discussion, and will
likely apply to most of the concurrency primitives in the `threading` module
that are written in Python. See pythongh-118433 for more details.

Disable the test in free-threaded builds as a temporary measure to eliminate
flakiness.
@bedevere-app bedevere-app bot added the tests Tests in the Lib/test dir label May 1, 2024
@mpage mpage added the skip news label May 1, 2024
@mpage mpage requested a review from colesbury May 1, 2024 20:02
@mpage mpage marked this pull request as ready for review May 1, 2024 20:02
@colesbury colesbury requested a review from encukou May 1, 2024 20:13
@colesbury
Copy link
Contributor

!buildbot M1 Refleaks NoGIL

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @colesbury for commit de8ab42 🤖

The command will test the builders whose names match following regular expression: M1 Refleaks NoGIL

The builders matched are:

  • ARM64 MacOS M1 Refleaks NoGIL PR

Copy link
Contributor

@colesbury colesbury left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I think this is worth doing while we figure out a solution to #118433 so that the free-threaded CI is more reliable.

I'd like to run it by @encukou too.

@colesbury
Copy link
Contributor

@brandtbucher is seeing repeated failures of this test on the Windows GitHub actions CI (https://github.com/python/cpython/actions/runs/8912122661/job/24481577797?pr=118481) so I'm going to enable auto-merge now.

@colesbury colesbury enabled auto-merge (squash) May 1, 2024 20:57
@colesbury colesbury disabled auto-merge May 1, 2024 20:57
@colesbury colesbury enabled auto-merge (squash) May 1, 2024 20:57
@colesbury colesbury merged commit 8a50544 into python:main May 1, 2024
36 of 37 checks passed
@mpage
Copy link
Contributor Author

mpage commented May 1, 2024

@colesbury - Isn't that a different test (test_multiprocessing_spawn.test_threads.WithThreadsTestPool.test_release_task_refs)?

@colesbury
Copy link
Contributor

Well, that's doubly embarrassing...

SonicField pushed a commit to SonicField/cpython that referenced this pull request May 8, 2024
…free-threaded builds (python#118485)

Free-threaded builds can intermittently tickle a longstanding bug (24 years!)
in the implementation of `threading.Condition`, leading to flakiness in the
test suite. Fixing the underlying issue will require more discussion, and will
likely apply to most of the concurrency primitives in the `threading` module
that are written in Python. See pythongh-118433 for more details.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants