Skip to content

python3Packages.twisted: skip flaky test on macOS#500926

Merged
SuperSandro2000 merged 1 commit intoNixOS:stagingfrom
JamieMagee:twisted-skip-flaky-macos-test
Apr 11, 2026
Merged

python3Packages.twisted: skip flaky test on macOS#500926
SuperSandro2000 merged 1 commit intoNixOS:stagingfrom
JamieMagee:twisted-skip-flaky-macos-test

Conversation

@JamieMagee
Copy link
Copy Markdown
Member

Skip test_fullWriteBufferAfterByteExchange on macOS. The test hits a reactor timeout after 120 seconds due to what upstream believes is a macOS kernel bug: select/kqueue sometimes fail to wake up when a socket's peer closes. Upstream has been seeing this flake since May 2024 and it's gotten worse on newer Python/macOS combos.

twisted/twisted#12151

Also updated the comment on the existing test_resumeProducingAbort/test_resumeProducingAbortLater skips to reference the upstream issue instead of pointing at the source file.

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

github-actions[bot]

This comment was marked as outdated.

…ange on macOS

Reactor timeout caused by a suspected macOS kernel bug where
select/kqueue fail to notify that a socket's peer has closed.

twisted/twisted#12151
@JamieMagee JamieMagee force-pushed the twisted-skip-flaky-macos-test branch from 2a6f6e5 to 63c072d Compare March 18, 2026 04:56
@JamieMagee JamieMagee changed the base branch from master to staging March 18, 2026 04:56
@nixpkgs-ci nixpkgs-ci Bot closed this Mar 18, 2026
@nixpkgs-ci nixpkgs-ci Bot reopened this Mar 18, 2026
@ofborg ofborg Bot added the 6.topic: darwin Running or building packages on Darwin label Mar 18, 2026
@nixpkgs-ci nixpkgs-ci Bot added 10.rebuild-darwin: 501+ This PR causes many rebuilds on Darwin and should normally target the staging branches. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. 10.rebuild-darwin: 1001-2500 This PR causes many rebuilds on Darwin and should most likely target the staging branches. 9.needs: reviewer This PR currently has no reviewers requested and needs attention. 6.topic: python Python is a high-level, general-purpose programming language. labels Mar 18, 2026
@nixpkgs-ci nixpkgs-ci Bot added 12.approvals: 1 This PR was reviewed and approved by one person. and removed 9.needs: reviewer This PR currently has no reviewers requested and needs attention. labels Mar 26, 2026
@nixos-discourse
Copy link
Copy Markdown

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-already-reviewed/2617/2935

@SuperSandro2000 SuperSandro2000 added this pull request to the merge queue Apr 11, 2026
Merged via the queue into NixOS:staging with commit c97ed6f Apr 11, 2026
47 of 50 checks passed
@JamieMagee JamieMagee deleted the twisted-skip-flaky-macos-test branch April 11, 2026 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: darwin Running or building packages on Darwin 6.topic: python Python is a high-level, general-purpose programming language. 10.rebuild-darwin: 501+ This PR causes many rebuilds on Darwin and should normally target the staging branches. 10.rebuild-darwin: 1001-2500 This PR causes many rebuilds on Darwin and should most likely target the staging branches. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants