Skip to content

release: wait for the simple index before exercising the published wheel - #95

Merged
stefan-jansen merged 1 commit into
mainfrom
infra/release-verify-waits-for-the-simple-index
Sep 12, 2026
Merged

release: wait for the simple index before exercising the published wheel#95
stefan-jansen merged 1 commit into
mainfrom
infra/release-verify-waits-for-the-simple-index

Conversation

@stefan-jansen

Copy link
Copy Markdown
Contributor

The post-release job checks the publication against two different PyPI endpoints and treats them as one. Verify PyPI identity and digests reads https://pypi.org/pypi/<name>/<version>/json; Install and exercise the published wheel reads https://pypi.org/simple/ through uv. The two propagate independently, so the first can pass on a version the second has not listed yet.

Measured on the 0.1.7 release run: publish completed at 11:59:22Z, the install failed 24 seconds later with no version of ml4t-backtest==0.1.7, and a re-run of the same job at 12:03:22Z passed with no change to anything. The package was correctly published throughout; the check was faster than the index.

This retries the install for up to 12 attempts over 120 seconds, the same budget check_documentation_identity.py already uses two steps below it, and fails with a message naming the simple index if the version never appears. A genuinely missing artifact still fails the job, two minutes later than before.

🤖 Generated with Claude Code

https://claude.ai/code/session_01UPDj1nioTuD4xATMVyhZxB

The post-release job verified the publication against two different PyPI
endpoints and treated them as one. 'Verify PyPI identity and digests' reads
https://pypi.org/pypi/<name>/<version>/json; 'Install and exercise the
published wheel' reads https://pypi.org/simple/ through uv. They propagate
independently, so the first can pass on a version the second has not listed.

Measured on 0.1.7: publish completed 11:59:22Z, the install failed 24s later
with 'no version of ml4t-backtest==0.1.7', and a re-run of the same job at
12:03:22Z passed unchanged. Nothing was wrong with the release; the check was
faster than the index.

Retry the install for up to 12 attempts over 120s, the same budget the
deployed-documentation recheck already uses below it, and fail with an
explicit message naming the simple index if it never appears. A genuinely
absent artifact still fails, two minutes later.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UPDj1nioTuD4xATMVyhZxB
Copilot AI lite review requested due to automatic review settings September 12, 2026 12:15

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@stefan-jansen
stefan-jansen merged commit 2f717e3 into main Sep 12, 2026
43 checks passed
@stefan-jansen
stefan-jansen deleted the infra/release-verify-waits-for-the-simple-index branch September 12, 2026 12:54
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