Skip to content

ci_scripts/check_versions.py: gracefully handle existence of upgrade PRs - #273

Open
threexc wants to merge 2 commits into
mainfrom
tgamblin/fix_check_versions
Open

ci_scripts/check_versions.py: gracefully handle existence of upgrade PRs#273
threexc wants to merge 2 commits into
mainfrom
tgamblin/fix_check_versions

Conversation

@threexc

@threexc threexc commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

The workflow currently fails every night because of a backlog of version upgrade PRs already being open from previous runs. Improve check_versions.py script so that if it can't open these PRs for versions RISE doesn't yet build, then it finishes gracefully. Do this by trying to confirm the existence of a PR for the given version(s), then warning the maintainer that they should check for them manually if they can't be detected automatically.

AI-Generated: Uses Claude Sonnet 5

@threexc
threexc force-pushed the tgamblin/fix_check_versions branch from 9754d56 to ecd04ab Compare August 19, 2026 15:30
Comment thread .github/workflows/nightly.yml
Comment thread ci_scripts/check_versions.py Outdated
Comment thread ci_scripts/check_versions.py Outdated
Comment thread ci_scripts/check_versions.py
Comment thread ci_scripts/check_versions.py Outdated
@threexc
threexc force-pushed the tgamblin/fix_check_versions branch from ecd04ab to 7bd2493 Compare August 19, 2026 15:55
The workflow currently fails every night because of a backlog of version
upgrade PRs already being open from previous runs. Improve
check_versions.py script so that if it can't open these PRs for versions
RISE doesn't yet build, then it finishes gracefully. Do this by trying
to confirm the existence of a PR for the given version(s), then warning
the maintainer that they should check for them manually if they can't be
detected automatically.

Additionally, add three retries when attempting to retrieve package
information from PyPI, with a backoff time, so that the check doesn't
fail if information isn't successfully retrieved for one or two
packages during the run.

AI-Generated: Uses Claude Sonnet 5

Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
….py is modified

Run the nightly workflow on PRs when appropriate, but do a dry-run
(i.e. don't actually try creating PRs) in this case so we don't open
things during testing.

AI-Generated: Uses Claude Sonnet 5

Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
@threexc
threexc force-pushed the tgamblin/fix_check_versions branch from 7bd2493 to 79252ec Compare August 19, 2026 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants