Skip to content

gh-110184: Fix subprocess test_pipesize_default() #110465

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 6, 2023

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Oct 6, 2023

For proc.stdin, get the size of the read end of the pipe.

Use subprocess context manager ("with proc:").

For proc.stdin, get the size of the read end of the pipe.

Use subprocess context manager ("with proc:").
@vstinner
Copy link
Member Author

vstinner commented Oct 6, 2023

!buildbot s390x

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @vstinner for commit f08aa06 🤖

The command will test the builders whose names match following regular expression: s390x

The builders matched are:

  • s390x SLES PR
  • s390x RHEL8 Refleaks PR
  • s390x Fedora Clang Installed PR
  • s390x RHEL7 LTO PR
  • s390x RHEL7 LTO + PGO PR
  • s390x RHEL8 LTO PR
  • s390x RHEL8 LTO + PGO PR
  • s390x Debian PR
  • s390x RHEL7 PR
  • s390x Fedora LTO PR
  • s390x Fedora Refleaks PR
  • s390x RHEL7 Refleaks PR
  • s390x RHEL8 PR
  • s390x Fedora PR
  • s390x Fedora LTO + PGO PR
  • s390x Fedora Clang PR

@vstinner
Copy link
Member Author

vstinner commented Oct 6, 2023

buildbot/s390x RHEL8 LTO PR

Failed with: Re-running test.test_multiprocessing_fork.test_processes in verbose mode.

@vstinner
Copy link
Member Author

vstinner commented Oct 6, 2023

buildbot/s390x RHEL8 Refleaks PR

I stopped the build, Refleaks test is useless here.

@miss-islington
Copy link
Contributor

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 6, 2023
)

For proc.stdin, get the size of the read end of the test pipe.

Use subprocess context manager ("with proc:").
(cherry picked from commit d023d41)

Co-authored-by: Victor Stinner <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Oct 6, 2023

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

@bedevere-app bedevere-app bot removed the needs backport to 3.12 only security fixes label Oct 6, 2023
@bedevere-app
Copy link

bedevere-app bot commented Oct 6, 2023

GH-110472 is a backport of this pull request to the 3.11 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.11 only security fixes label Oct 6, 2023
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 6, 2023
)

For proc.stdin, get the size of the read end of the test pipe.

Use subprocess context manager ("with proc:").
(cherry picked from commit d023d41)

Co-authored-by: Victor Stinner <[email protected]>
vstinner added a commit that referenced this pull request Oct 6, 2023
…110471)

gh-110184: Fix subprocess test_pipesize_default() (GH-110465)

For proc.stdin, get the size of the read end of the test pipe.

Use subprocess context manager ("with proc:").
(cherry picked from commit d023d41)

Co-authored-by: Victor Stinner <[email protected]>
vstinner added a commit that referenced this pull request Oct 6, 2023
…110472)

gh-110184: Fix subprocess test_pipesize_default() (GH-110465)

For proc.stdin, get the size of the read end of the test pipe.

Use subprocess context manager ("with proc:").
(cherry picked from commit d023d41)

Co-authored-by: Victor Stinner <[email protected]>
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
For proc.stdin, get the size of the read end of the test pipe.

Use subprocess context manager ("with proc:").
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants