Skip to content

Conversation

@colesbury
Copy link
Contributor

@colesbury colesbury commented Dec 23, 2025

TSan reports that these tests leak threads. We currently suppress the warning in a suppressions file. Skipping these tests instead of relying on the suppression file will allow us to catch regressions in other test cases.

@bedevere-app bedevere-app bot added the tests Tests in the Lib/test dir label Dec 23, 2025
@colesbury colesbury added skip news and removed tests Tests in the Lib/test dir labels Dec 23, 2025
@colesbury colesbury added the tests Tests in the Lib/test dir label Dec 23, 2025
@colesbury colesbury marked this pull request as ready for review December 23, 2025 23:48
@colesbury colesbury requested a review from gpshead as a code owner December 23, 2025 23:48
@colesbury colesbury requested a review from mpage December 23, 2025 23:48
@gpshead gpshead added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes labels Dec 24, 2025
@colesbury colesbury merged commit 4ee6929 into python:main Dec 24, 2025
62 checks passed
@colesbury colesbury deleted the gh-143121-multiprocessing-thread-leaks branch December 24, 2025 13:02
@miss-islington-app
Copy link

Thanks @colesbury for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14.
🐍🍒⛏🤖

@miss-islington-app
Copy link

Sorry, @colesbury, I could not cleanly backport this to 3.14 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 4ee6929d606fa7b976eba229de24219f0edac3d7 3.14

@miss-islington-app
Copy link

Sorry, @colesbury, I could not cleanly backport this to 3.13 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 4ee6929d606fa7b976eba229de24219f0edac3d7 3.13

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

Labels

needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants