Skip to content

CI: raise Linux build job timeout to 180 minutes - #1974

Merged
bkaradzic-microsoft merged 2 commits into
BabylonJS:masterfrom
bkaradzic-microsoft:ci/linux-timeout-60
Oct 26, 2026
Merged

bkaradzic-microsoft merged 2 commits into
BabylonJS:masterfrom
bkaradzic-microsoft:ci/linux-timeout-60

Conversation

@bkaradzic-microsoft

@bkaradzic-microsoft bkaradzic-microsoft commented Oct 26, 2026 •

Copy link
Copy Markdown
Member

Ubuntu_Clang_JSC routinely takes 28–29 minutes (about 10 minutes of build plus about 19 minutes of validation) against the 30-minute job timeout in build-linux.yml:

PR run Ubuntu_Clang_JSC
#1971 28m ✅
#1964 29m ✅
#1965 28m ✅
#1967 cancelled at 30m26s, after Run complete. ran=577 passed=577 failed=0

Any slightly slower runner cancels the job, and more re-enabled validation tests will push it past the limit every time. This PR sets a 180-minute limit for every Linux engine, matching the Nightly job timeout (#1968).

Ubuntu_Clang_JSC routinely takes 28-29 minutes (about 10 minutes of build plus about 19 minutes of validation tests)
against a 30-minute job timeout. On BabylonJS#1967 it ran all 577 tests successfully and was then cancelled at 30m26s. Use
the Hermes job's 60-minute limit for every engine.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 60c2ec68-6de1-445d-9fc9-b699db737eae
Copilot AI balanced review requested due to automatic review settings October 26, 2026 21:49
Matches the Nightly job timeout (BabylonJS#1968).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 60c2ec68-6de1-445d-9fc9-b699db737eae
@bkaradzic-microsoft bkaradzic-microsoft changed the title CI: raise Linux build job timeout to 60 minutes CI: raise Linux build job timeout to 180 minutes Oct 26, 2026

Copilot AI 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.

🟢 Approval recommended

The focused job-level timeout change matches the stated intent and has no identified blocking issues.

No open issues 🎉

What changed in this PR

Raises the reusable Linux build job timeout to accommodate full validation runs that approach the existing limit, matching the 180-minute Nightly timeout.

Changes:

  • Replaces the engine-dependent 30/60-minute timeout with 180 minutes for all Linux engines.
  • Adds a comment explaining the observed Clang/JSC runtimes and timeout cancellations.
File Description
.github/​workflows/​build-linux.yml Increases the job timeout and documents the reason.

Review effort: Balanced

@bghgary bghgary 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.

[Reviewed by Copilot on behalf of @bghgary]

LGTM, but we will need to reel this back in later as 180 minutes is too long for CI. We will be targeting 30 minutes or less.

@bkaradzic-microsoft
bkaradzic-microsoft enabled auto-merge (squash) October 26, 2026 22:15
@bkaradzic-microsoft
bkaradzic-microsoft merged commit 0587635 into BabylonJS:master Oct 26, 2026
27 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.

3 participants