Skip to content

ci(journeys): start the journeys jobs after the site job again - #564

Merged
ixuz merged 1 commit into
mainfrom
journeys-after-site
Nov 4, 2026
Merged

ixuz merged 1 commit into
mainfrom
journeys-after-site

Conversation

@ixuz

@ixuz ixuz commented Nov 4, 2026

Copy link
Copy Markdown
Member

What and why

Since #530, the journeys jobs have started with the rest of the run and polled the jobs API until the site job was done. When several runs share the 20 runners, the site job waits in the queue too, often longer than its journeys jobs. Those jobs then hold five runners doing nothing, while other pull requests' jobs wait for one. In run 41785995461 they spent 2042 s waiting (34 runner-minutes), and in other busy runs 482–868 s. The overlap saves about a minute, but only when the run has runners to spare.

The cost: when the run has runners to spare and check hits Turbo's cache, the journeys are the run's last jobs, and that run ends about a minute later. When check runs its tasks, it ends after the journeys anyway.

Closes: #561

The journeys jobs have started with the rest of the run since #530,
and polled the jobs API until the site job was done. When several runs
share the 20 runners, the site job waits in the queue too, often longer
than its journeys jobs, which then hold five runners doing nothing: up
to 34 runner-minutes in one run, when other pull requests' jobs are
waiting for a runner. The overlap saves about a minute only when the
run has runners to spare.

The jobs need the site job again, so a failed site job skips them
through needs, which ci-ok already fails on. The wait step goes, and
with it the actions: read permission only it used; the timeout goes
back to the journeys' own 25 minutes. The server still starts while
the browsers are set up, and #530's other changes stay.

Closes: #561
@ixuz ixuz added enhancement New feature or request ci/cd Countinuous integration and deployment labels Nov 4, 2026
@ixuz
ixuz added this pull request to the merge queue Nov 4, 2026
Merged via the queue into main with commit f808304 Nov 4, 2026
51 checks passed
@ixuz
ixuz deleted the journeys-after-site branch November 4, 2026 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd Countinuous integration and deployment enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The journeys jobs hold runners while they wait for the site's build, which slows busy runs

1 participant