Skip to content

Ignore failing buildbots that last ran over 14 days ago - #415

Merged
hugovk merged 4 commits into
python:mainfrom
hugovk:ignore-stale-buildbots
Sep 1, 2026
Merged

Ignore failing buildbots that last ran over 14 days ago#415
hugovk merged 4 commits into
python:mainfrom
hugovk:ignore-stale-buildbots

Conversation

@hugovk

@hugovk hugovk commented Jun 23, 2026

Copy link
Copy Markdown
Member

Fixes #410.

@hugovk hugovk mentioned this pull request Jul 8, 2026

@savannahostrowski savannahostrowski left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall, LGTM.

Maybe we could also print out "ignoring N stale builders" or something in check_buildbots as well?

Comment thread buildbotapi.py Outdated
Comment thread tests/test_buildbotapi.py
("tests/buildbotapi/success.json", False),
("tests/buildbotapi/failure.json", True),
("tests/buildbotapi/no-builds.json", False),
# Recent builds: judged on their result

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can add a case for the cutoff (i.e. FAILURE_COMPLETE_AT + 14 * DAY)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added in 48dcbc2.

@hugovk hugovk left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we could also print out "ignoring N stale builders" or something in check_buildbots as well?

Added in e63c48d.

Could perhaps use an enum instead of bool | None.

Comment thread tests/test_buildbotapi.py
("tests/buildbotapi/success.json", False),
("tests/buildbotapi/failure.json", True),
("tests/buildbotapi/no-builds.json", False),
# Recent builds: judged on their result

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added in 48dcbc2.

@hugovk
hugovk merged commit 375621b into python:main Sep 1, 2026
25 of 26 checks passed
@hugovk
hugovk deleted the ignore-stale-buildbots branch September 1, 2026 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants