Skip to content

600000 ms is deploy-pages' maximum, not its default - #42

Merged
mbakalarski merged 1 commit into
mainfrom
fix-deploy-timeout-comment
Aug 6, 2026
Merged

600000 ms is deploy-pages' maximum, not its default#42
mbakalarski merged 1 commit into
mainfrom
fix-deploy-timeout-comment

Conversation

@mbakalarski

Copy link
Copy Markdown
Member

The timeout raised in #41 never applied. actions/deploy-pages accepts a
larger value, warns, and clamps it straight back:

##[warning]Warning: timeout value is greater than the allowed maximum
           - timeout set to the maximum of 600000 milliseconds.

Confirmed against the action's own action.yml: timeout is documented as
"default: 10 minutes", and 600000 turns out to be a ceiling as well as a default.
Both repositories then failed again at 10m10s and 10m07s with the larger value
set — which is the measurement that proves the input did nothing.

So a Pages deploy here can never wait longer than ten minutes, and nothing in
this repository widens that. The input is dropped; the comment stays, because
the obvious fix looks available and is not, and that is the part worth
someone's time.

What is actually wrong

Nothing in this repository. Every poll returns deployment_queued, GitHub
reported Pages fully operational throughout, and the same workflow shape
succeeded at 8m04s and 9m12s earlier the same day. The queue is simply slower
than the ceiling this afternoon.

install.md from #39 is still unpublished for that reason. Merging this
provides the new commit a fresh deployment needs — a re-run cannot, since the
cancelled deployment is keyed by SHA.

🤖 Generated with Claude Code

The timeout raised in #41 never applied. The action accepts a larger value,
warns, and clamps it back:

  Warning: timeout value is greater than the allowed maximum
           - timeout set to the maximum of 600000 milliseconds.

So a Pages deploy can never wait longer than ten minutes, and no setting in
this repository changes that. The input is dropped again; what is worth
keeping is the comment saying why raising it is not an option, since the
obvious fix looks available and is not.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@mbakalarski
mbakalarski merged commit 752bff1 into main Aug 6, 2026
3 checks passed
@mbakalarski
mbakalarski deleted the fix-deploy-timeout-comment branch August 6, 2026 13:58
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.

1 participant