fix(ci): smoke test accepts release-assets.githubusercontent.com CDN#509
Merged
fix(ci): smoke test accepts release-assets.githubusercontent.com CDN#509
Conversation
v2.0.4 rerun of update-apt-repo made it past hops 0 and 1 (the smoke test scheme fix in #506 worked — Pages' http:// redirect no longer trips the chain walker), but failed on hop 2: Hop 2: 302 .../releases/download/v2.0.4+claude1.3883.0/...deb -> https://release-assets.githubusercontent.com/... ::error::Hop 2 mismatch: expected https://objects\.githubusercontent\.com/, got https://release-assets.githubusercontent.com/... GitHub migrated the Release asset CDN from objects.githubusercontent.com to release-assets.githubusercontent.com (both have been serving in the past; release-assets is the current canonical hostname). Accept either hostname via alternation. Verified against the actual v2.0.4 Release: $ curl -Is https://github.com/aaddrick/claude-desktop-debian/releases/download/v2.0.4+claude1.3883.0/claude-desktop_1.3883.0-2.0.4_amd64.deb \ | grep -i location location: https://release-assets.githubusercontent.com/github-production-release-asset/... Same fix in three sites: - .github/workflows/ci.yml (update-apt-repo smoke test) - .github/workflows/ci.yml (update-dnf-repo smoke test) - .github/workflows/apt-repo-heartbeat.yml (daily heartbeat) docs/worker-apt-plan.md has historical references to objects.githubusercontent.com too; those can be updated in a follow-up docs sweep — the architectural claim (binary bytes flow direct from GitHub CDN, never through Cloudflare) is unchanged. Refs #493, #503
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
v2.0.4 CI surfaced the next smoke-test mismatch — at hop 2:
GitHub migrated Release assets from
objects.githubusercontent.com→release-assets.githubusercontent.com. Both have served in the past;release-assetsis current. Accept either via alternation:Same fix applied to all three sites:
.github/workflows/ci.yml:540(APT smoke test).github/workflows/ci.yml:754(DNF smoke test).github/workflows/apt-repo-heartbeat.yml:94(daily heartbeat)Validation
Hops 0 and 1 already pass with #506's scheme fix — the v2.0.4 run log confirms. This PR completes the chain walker so v2.0.4's update-apt-repo smoke test will pass and
update-dnf-repowill finally run (was skipped both on v2.0.3 and v2.0.4 due toneeds: update-apt-repo).Post-merge
Cut v2.0.5 to re-exercise the full pipeline with the fix — same as v2.0.4 flow but this time both jobs should complete cleanly and test the #500
gh release upload --clobberpath for the first time.docs/worker-apt-plan.mdhas historical references toobjects.githubusercontent.com— a docs-only cleanup in a follow-up.Refs #493, #503
Generated with Claude Code
Co-Authored-By: Claude Opus 4.7 [email protected]
Claude: walked through v2.0.3 → #506 → v2.0.4 → this; each iteration surfaced the next assumption in the plan that no longer matched GitHub's current behavior
Human: delegated Phase 4a-APT cutover autonomously