Skip to content

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

Merged
vstinner merged 1 commit into
python:mainfrom
vstinner:regrtest_refactor7
Sep 9, 2023
Merged

gh-109162: libregrtest: fix _decode_worker_job()#109202
vstinner merged 1 commit into
python:mainfrom
vstinner:regrtest_refactor7

Conversation

@vstinner

@vstinnervstinner commented Sep 9, 2023

Copy link
Copy Markdown
Member

Decode also HuntRefleak() object inside the RunTests object.

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

Decode also HuntRefleak() object inside the RunTests object.
Add an unit test on huntrleaks with multiprocessing (-R -jN).
@AlexWaygoodAlexWaygood added the 🔨 test-with-refleak-buildbots Test PR w/ refleak buildbots; report in status section label Sep 9, 2023
@bedevere-bot

Copy link
Copy Markdown

🤖 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-botbedevere-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:mainSep 9, 2023
@vstinner
vstinner deleted the regrtest_refactor7 branch September 9, 2023 22:51
@vstinner

Copy link
Copy Markdown
MemberAuthor

🤖 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
Copy Markdown
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
Copy Markdown
MemberAuthor

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 freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@vstinner@bedevere-bot@AlexWaygood