Repository navigation
ci: ask Ubuntu's mirror for lists only when a library is missing - #728
Merged
Merged
Conversation
The site job failed on 2026-11-11 after twelve minutes because azure.archive.ubuntu.com did not answer any of the four apt-get update tries. That update was the job's only use of the mirror. Playwright's dry run reads the package lists that come with the runner image, and without fonts the image is missing nothing, so the job had nothing to download. playwright-deps now asks Playwright first and runs apt-get update only when a package has to be downloaded, or when the image's lists can't answer the dry run. A job missing nothing never reaches the mirror. Closes: #727
ixuz
enabled auto-merge
November 11, 2026 15:38
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What and why
The
sitejob failed on 2026-11-11 after 12 minutes becauseazure.archive.ubuntu.comdidn't answer any of the fourapt-get updatetries. That update was the job's only use of the mirror. Playwright's check of what is missing (install-deps --dry-run, which runsapt-get install -s) reads the package lists that come with the runner image. Without the fonts (#607), the image is missing nothing, so the job had nothing to download..github/actions/playwright-depsnow asks Playwright first and runsapt-get updateonly:The
sitejob, the release job and the deploy's build are missing nothing on today's image, so they no longer reach the mirror. A job that downloads packages still updates once first: the journeys on a cache miss,warm-browsers, or any job on a new image that lacks a library. The four tries of three minutes are unchanged. The comment on thesitejob's timeout inci.ymlnow says the libraries come from the mirror only when the image lacks one.The step's paths were tested with
npxandapt-getstubbed: nothing missing, only fonts missing (with and without the fonts), other libraries missing, and no usable lists. Each made the expected apt calls. This pull request's ownsitejob is the first run on a real runner image.npm run check,actionlint,zizmorandshellcheckpass.Closes: #727