Skip to content

[3.12] gh-107851: Fix spurious failures in fcntl eintr tests (GH-121556)#121586

Merged
colesbury merged 1 commit into
python:3.12from
miss-islington:backport-af9f6de-3.12
Jul 10, 2024
Merged

[3.12] gh-107851: Fix spurious failures in fcntl eintr tests (GH-121556)#121586
colesbury merged 1 commit into
python:3.12from
miss-islington:backport-af9f6de-3.12

Conversation

@miss-islington

@miss-islington miss-islington commented Jul 10, 2024

Copy link
Copy Markdown
Contributor

On heavily loaded machines, the subprocess may finish its sleep before
the parent process manages to synchronize with it via a failed lock.

This leads to errors like:

Exception: failed to sync child in 300.3 sec

Use pipes instead to mutually synchronize between parent and child.
(cherry picked from commit af9f6de)

Co-authored-by: Sam Gross colesbury@gmail.com

…-121556)

On heavily loaded machines, the subprocess may finish its sleep before
the parent process manages to synchronize with it via a failed lock.

This leads to errors like:

  Exception: failed to sync child in 300.3 sec

Use pipes instead to mutually synchronize between parent and child.
(cherry picked from commit af9f6de)

Co-authored-by: Sam Gross <colesbury@gmail.com>
@bedevere-app bedevere-app Bot added the tests Tests in the Lib/test dir label Jul 10, 2024
@colesbury colesbury enabled auto-merge (squash) July 10, 2024 14:37
@colesbury colesbury merged commit 847c803 into python:3.12 Jul 10, 2024
@miss-islington miss-islington deleted the backport-af9f6de-3.12 branch January 2, 2026 18:07
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.

2 participants