Skip to content

Raise publish build timeout to 10 minutes#340

Merged
corporate-gadfly merged 1 commit into
OpenVoxProject:masterfrom
miharp:fix/publish-build-timeout
Jun 17, 2026
Merged

Raise publish build timeout to 10 minutes#340
corporate-gadfly merged 1 commit into
OpenVoxProject:masterfrom
miharp:fix/publish-build-timeout

Conversation

@miharp

@miharp miharp commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

What

Raise the publish build job's timeout-minutes from 5 to 10.

Why

The merge of #339 left master unpublished: its publish run was cancelled at the Archive website step and deploy was skipped. Root cause is the 5-minute build timeout, not a build error:

Run build job duration Result
#335 (last green) 4m41s ✅ (only ~19s under the 5m cap)
#339 (cancelled) 5m06s ⏱️ timed out mid-Archive

references:all clones and generates docs for three products (network-variable), and the component-version refresh tasks added in #339 make additional live GitHub API calls. The build was already at the edge of the 5-minute cap; the extra calls tipped it over. "Build website" itself succeeds — it's purely cumulative time, so more headroom is the fix.

The publish `build` job is capped at `timeout-minutes: 5`, but the build was
already running ~4m41s (references:all clones and generates docs for three
products, which is network-variable). The component-version refresh tasks added
in OpenVoxProject#339 make additional live GitHub API calls, pushing a representative build to
~5m06s — over the cap — so the job was cancelled at the Archive step and the
deploy was skipped, leaving master unpublished.

Raise the cap to 10 minutes to restore publishing and give headroom for the
network-variable build.

Signed-off-by: Michael Harp <mike@mikeharp.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@miharp miharp requested a review from a team as a code owner June 17, 2026 13:17

@corporate-gadfly corporate-gadfly left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@corporate-gadfly corporate-gadfly merged commit 9d67f4c into OpenVoxProject:master Jun 17, 2026
5 checks passed
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