Skip to content

un bitrot flaky tests#23166

Merged
cburroughs merged 1 commit intopantsbuild:mainfrom
cburroughs:csb/flaky-bitrot
Mar 17, 2026
Merged

un bitrot flaky tests#23166
cburroughs merged 1 commit intopantsbuild:mainfrom
cburroughs:csb/flaky-bitrot

Conversation

@cburroughs
Copy link
Copy Markdown
Contributor

The goal isn't to fix the flakyness, just to make them runnable again as a prerequisite to eventual fixing.

For pantsd_integration_test.py the files were moved around in 2d1794f.

For jdk_rules_test.py the prefix switch pattern was already intoduced in 45c51ae.

This was accomplished by having an LLM go through the list of all flaky tests and uncomment the 'skip' one by one and then try to fix any bitrot.

The goal isn't to fix the flakyness, just to make them runnable again
as a prerequisite to eventual fixing.

For `pantsd_integration_test.py` the files were moved around in
2d1794f.

For `jdk_rules_test.py` the prefix switch pattern was already
intoduced in 45c51ae.

This was accomplished by having an LLM go through the list of all
flaky tests and uncomment the 'skip' one by one and then try to fix
any bitrot.
@cburroughs cburroughs self-assigned this Mar 11, 2026
@cburroughs cburroughs added the release-notes:not-required [CI] PR doesn't require mention in release notes label Mar 11, 2026
@cburroughs
Copy link
Copy Markdown
Contributor Author

The test failure is -- I think -- https://discourse.osgeo.org/t/service-disruption-tomorrow/152805

@cburroughs cburroughs marked this pull request as ready for review March 16, 2026 17:37
@cburroughs cburroughs merged commit 3df6308 into pantsbuild:main Mar 17, 2026
48 of 50 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-notes:not-required [CI] PR doesn't require mention in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants