Skip to content

Conversation

@vstinner
Copy link
Member

@vstinnervstinner commented Sep 13, 2023

The test now calls free_tool_id() so it can be run multiple times in the same process. For example, the following command no longer fails:

python -m test test_monitoring -R 3:3 

The test now calls free_tool_id() so it can be run multiple times in the same process. For example, the following command no longer fails: python -m test test_monitoring -R 3:3
@vstinner
Copy link
MemberAuthor

cc @markshannon

@vstinnervstinner enabled auto-merge (squash) September 13, 2023 01:44
@vstinnervstinner merged commit 388d91c into python:mainSep 13, 2023
@vstinnervstinner deleted the fix_test_monitoring branch September 13, 2023 01:54
@miss-islington
Copy link
Contributor

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 13, 2023
The test now calls free_tool_id() so it can be run multiple times in the same process. For example, the following command no longer fails: python -m test test_monitoring -R 3:3 (cherry picked from commit 388d91c) Co-authored-by: Victor Stinner <[email protected]>
@bedevere-app
Copy link

GH-109359 is a backport of this pull request to the 3.12 branch.

@bedevere-appbedevere-appbot removed the needs backport to 3.12 only security fixes label Sep 13, 2023
Yhg1s pushed a commit that referenced this pull request Sep 13, 2023
…09359) gh-109357: Fix test_monitoring.test_gh108976() (GH-109358) The test now calls free_tool_id() so it can be run multiple times in the same process. For example, the following command no longer fails: python -m test test_monitoring -R 3:3 (cherry picked from commit 388d91c) Co-authored-by: Victor Stinner <[email protected]>
vstinner added a commit to vstinner/cpython that referenced this pull request Sep 13, 2023
The test now calls free_tool_id() so it can be run multiple times in the same process. For example, the following command no longer fails: python -m test test_monitoring -R 3:3
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip newstestsTests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@vstinner@miss-islington