Skip to content

ci: revert upload-pages-artifact to v3.0.1 (Pages deploy hang A/B test) - #279

Merged
RGood merged 1 commit into
mainfrom
ci/pages-artifact-v3-ab-test
Nov 10, 2026
Merged

RGood merged 1 commit into
mainfrom
ci/pages-artifact-v3-ab-test

Conversation

@RGood

@RGood RGood commented Nov 10, 2026

Copy link
Copy Markdown
Collaborator

Why

After #278 (bump upload-pages-artifact to v4.0.0), the Pages deploy hangs: deploy-pages polls "Getting Pages deployment status" with an empty status for 10 minutes, then times out and cancels. This happened twice on de8c0dc:

The build and artifact upload succeed (157 MB, same size as before), and githubstatus shows Pages/Actions operational. Every successful deploy used the v3 artifact; both hung deploys used the v4.0.0 artifact. This PR is an A/B test to see whether the artifact version is the cause.

Change

Revert only actions/upload-pages-artifact to the previously used v3.0.1 (56afc60…), keeping the SHA pin and version comment. devvit-docs is now exempt from the "require full SHA" setting, and v3 only calls GitHub-owned actions/upload-artifact@v4, which is allowed.

What to check after merge

The "Deploy to GitHub Pages" run on main should finish in roughly 3-4 minutes (previous successes: 184-480s).

  • Succeeds: v4.0.0 (or the nested change) is the cause. Next try v5.0.0 (fc324d3…) or investigate the artifact.
  • Still hangs: the artifact version isn't the cause; look at Pages service/site size (157 MB) and try a longer timeout on deploy-pages.

Notes

  • This workflow only runs on push to main, so it cannot be verified on the PR itself.
  • Temporary. If v3 works, decide whether to keep it, move to v5, or re-enable SHA pinning with a different fix.

@RGood
RGood requested review from a team and ben-harris-5 November 10, 2026 20:15
@RGood
RGood merged commit 2e4e7f7 into main Nov 10, 2026
5 checks passed
@RGood
RGood deleted the ci/pages-artifact-v3-ab-test branch November 10, 2026 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants