Skip to content

ci: fail php-qa jobs after 20 minutes instead of riding out a bad runner - #422

Merged
erikfrerejean merged 1 commit into
6from
ci-php-qa-timeout
Aug 6, 2026
Merged

ci: fail php-qa jobs after 20 minutes instead of riding out a bad runner#422
erikfrerejean merged 1 commit into
6from
ci-php-qa-timeout

Conversation

@erikfrerejean

Copy link
Copy Markdown
Member

Why

In run 31094035518 (PR #421), the PHP QA (fluent / PHP 8.3) job took 32 minutes while its 8.4/8.5 siblings finished in ~11.5 min and the base-matrix jobs in ~9 min. Investigation showed nothing wrong with the code or workflow:

  • All three fluent jobs run the identical PHPUnit command and all used PCOV 1.0.12 as coverage driver.
  • The overrun sat entirely inside the test step, smeared evenly across the DB-heavy suites (~3–5s/test vs the usual ~1–2s/test) — the signature of a degraded/contended hosted runner, not a hung test.
  • The two earlier runs of the same PR ran fluent/8.3 in 11–12 minutes.

The slow job also held its runner slot, queueing E2E (chromium / tailwind) until it finished and stretching the run to ~39 minutes wall clock.

What

Add timeout-minutes: 20 to the php-qa job — roughly double the healthy fluent duration. A degenerate runner now fails fast instead of burning 3× the minutes; a re-run lands on a fresh runner.

@erikfrerejean
erikfrerejean enabled auto-merge (rebase) August 6, 2026 11:25
@erikfrerejean
erikfrerejean merged commit 12f58e8 into 6 Aug 6, 2026
17 checks passed
@erikfrerejean
erikfrerejean deleted the ci-php-qa-timeout branch August 6, 2026 12:05
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