Skip to content

gh-109162: libregrtest: fix _decode_worker_job() #109202

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
Sep 9, 2023
Merged

gh-109162: libregrtest: fix _decode_worker_job() #109202

merged 1 commit into from
Sep 9, 2023

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Sep 9, 2023 •
edited by bedevere-bot
Loading

Decode also HuntRefleak() object inside the RunTests object.

Add an unit test on huntrleaks with multiprocessing (-R -jN).

@vstinner
pythongh-109162: libregrtest: fix _decode_worker_job()
833c6dc 
Decode also HuntRefleak() object inside the RunTests object.

Add an unit test on huntrleaks with multiprocessing (-R -jN).
@bedevere-bot bedevere-bot mentioned this pull request Sep 9, 2023
Closed
@vstinner vstinner mentioned this pull request Sep 9, 2023
Merged
@AlexWaygood AlexWaygood added the 🔨 test-with-refleak-buildbots Test PR w/ refleak buildbots; report in status section label Sep 9, 2023
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @AlexWaygood for commit 833c6dc 🤖

If you want to schedule another build, you need to add the 🔨 test-with-refleak-buildbots label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-refleak-buildbots Test PR w/ refleak buildbots; report in status section label Sep 9, 2023
@vstinner vstinner merged commit 24fa8f2 into python:main Sep 9, 2023
@vstinner vstinner deleted the regrtest_refactor7 branch September 9, 2023 22:51
@vstinner
Copy link
Member Author

vstinner commented Sep 9, 2023

🤖 New build scheduled with the buildbot fleet by @AlexWaygood for commit 833c6dc 🤖

Oh, I tested the exact command used by buildbots locally and I validated that my change fix this command.

I also added an unit test to check for non-regression.

I merged my PR since I would like to create following pull requests which rely on this fix.

@AlexWaygood
Copy link
Member

Oh, I tested the exact command used by buildbots locally and I validated that my change fix this command.

No worries! Some of the buildbots were already coming back green anyway, so I'm pretty confident we're okay here :-)

Thanks for the quick fix!

@vstinner
Copy link
Member Author

vstinner commented Sep 9, 2023

I merged my PR since I would like to create following pull requests which rely on this fix.

Following PR: PR #109204.

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
skip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@vstinner @bedevere-bot @AlexWaygood