Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

[tests] test_subprocess maybe avoid a timeout race condition?#133420

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

gpshead
Copy link
Member

The few buildbot failures on#133103 are possibly just due to racing a child process launch and exit?

The few buildbot failures onpython#133103are possibly just due to racing a child process launch and exit?
@gpsheadgpshead added testsTests in the Lib/test dir skip issue skip news needs backport to 3.13bugs and security fixes labelsMay 5, 2025
@gpshead
Copy link
MemberAuthor

!buildbot .s390.

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by@gpshead for commit9063e79 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F133420%2Fmerge

The command will test the builders whose names match following regular expression:.*s390.*

The builders matched are:

  • s390x Fedora Rawhide PR
  • s390x Fedora Stable LTO + PGO PR
  • s390x Fedora Stable PR
  • s390x Fedora Rawhide NoGIL PR
  • s390x Fedora Rawhide Refleaks PR
  • s390x RHEL9 LTO + PGO PR
  • s390x Fedora Rawhide NoGIL refleaks PR
  • s390x RHEL9 PR
  • s390x Fedora Stable LTO PR
  • s390x RHEL8 PR
  • s390x RHEL9 LTO PR
  • s390x Fedora Stable Clang Installed PR
  • s390x Fedora Stable Refleaks PR
  • s390x Fedora Rawhide Clang PR
  • s390x RHEL9 Refleaks PR
  • s390x Fedora Rawhide LTO + PGO PR
  • s390x Fedora Rawhide Clang Installed PR
  • s390x RHEL8 Refleaks PR
  • s390x RHEL8 LTO + PGO PR
  • s390x Fedora Stable Clang PR
  • s390x Fedora Rawhide LTO PR
  • s390x RHEL8 LTO PR

@gpshead
Copy link
MemberAuthor

!buildbot .Server.2022.

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by@gpshead for commit9063e79 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F133420%2Fmerge

The command will test the builders whose names match following regular expression:.*Server.2022.*

The builders matched are:

  • AMD64 Windows Server 2022 NoGIL PR

@emmatyping
Copy link
Member

It looks like the race is surfacing on Ubuntu freethreaded CI (e.g.https://github.com/python/cpython/actions/runs/14827808058/job/41623568354?pr=133365) so if it is fixed there that should be a good signal.

@emmatyping
Copy link
Member

The hypothesis failure looks similar to a failure I saw. I filed#133419 to track thetest_external_inspection failures.

@gpsheadgpshead merged commitb64aa30 intopython:mainMay 5, 2025
58 of 66 checks passed
@miss-islington-app
Copy link

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull requestMay 5, 2025
…GH-133420)The few buildbot failures onpython#133103are possibly just due to racing a child process launch and exit?(cherry picked from commitb64aa30)Co-authored-by: Gregory P. Smith <greg@krypto.org>
@bedevere-app
Copy link

GH-133421 is a backport of this pull request to the3.13 branch.

@bedevere-appbedevere-appbot removed the needs backport to 3.13bugs and security fixes labelMay 5, 2025
@Zheaoli
Copy link
Contributor

ops, sorry about this. Thanks for the patch!

gpshead added a commit that referenced this pull requestMay 5, 2025
…H-133420) (#133421)[tests] test_subprocess maybe avoid a timeout race condition? (GH-133420)The few buildbot failures on#133103are possibly just due to racing a child process launch and exit?(cherry picked from commitb64aa30)Co-authored-by: Gregory P. Smith <greg@krypto.org>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@emmatypingemmatypingemmatyping approved these changes

Assignees
No one assigned
Labels
skip issueskip newstestsTests in the Lib/test dir
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@gpshead@bedevere-bot@emmatyping@Zheaoli
[8]ページ先頭
©2009-2025 Movatter.jp