Skip to content

ci: install zstd in the build image - #16336

Merged
pitasi merged 2 commits into
mainfrom
anto/ci-build-image-zstd
Aug 10, 2026
Merged

ci: install zstd in the build image#16336
pitasi merged 2 commits into
mainfrom
anto/ci-build-image-zstd

Conversation

@pitasi

@pitasi pitasi commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

what

installs zstd in the build image.

why

actions/cache only uses zstd if a zstd binary is on PATH, and it runs inside the job container. the build image never had one, so every job running in it fell back to gzip which is slower and leads to bigger artifacts:

gzip zstd
restore step, median 35s 20s
restore step, min/max 31s / 46s 15s / 32s
archive size 1062 MB 1014 MB

that 30s of gunzip is on the critical path for many jobs, and the warmup jobs on main pay it again on the compress side.

notes

  • integration jobs are already using zstd so this applies to unit tests
  • merging this will invalidate the cache for unit and image-and-lint, but after a first cold start caching will resume as always

@github-actions

Copy link
Copy Markdown
Contributor

Building new version of mimir-build-image. After image is built and pushed to the registry, a new commit will automatically be added to this PR with new image version us-docker.pkg.dev/grafanalabs-dev/docker-mimir-build-image/mimir-build-image:pr16336-25522b6629. This can take a few minutes.

@pitasi pitasi closed this Aug 10, 2026
@pitasi pitasi reopened this Aug 10, 2026
@pitasi
pitasi force-pushed the anto/ci-build-image-zstd branch from 64782de to c1418bb Compare August 10, 2026 11:29
@github-actions

Copy link
Copy Markdown
Contributor

Building new version of mimir-build-image. After image is built and pushed to the registry, a new commit will automatically be added to this PR with new image version us-docker.pkg.dev/grafanalabs-dev/docker-mimir-build-image/mimir-build-image:pr16336-400d7ac8a1. This can take a few minutes.

@github-actions

Copy link
Copy Markdown
Contributor

Not building new version of mimir-build-image. This PR modifies the build image or the build image build workflow, but the image us-docker.pkg.dev/grafanalabs-dev/docker-mimir-build-image/mimir-build-image:pr16336-400d7ac8a1 already exists.

actions/cache only uses zstd if a zstd binary is on PATH, and it runs inside the
job container, so every job that runs in this image was falling back to gzip while
jobs on the bare runner used zstd.
@pitasi
pitasi force-pushed the anto/ci-build-image-zstd branch from ee18077 to b7180cb Compare August 10, 2026 12:41
@github-actions

Copy link
Copy Markdown
Contributor

Building new version of mimir-build-image. After image is built and pushed to the registry, a new commit will automatically be added to this PR with new image version us-docker.pkg.dev/grafanalabs-dev/docker-mimir-build-image/mimir-build-image:pr16336-e30cf26c29. This can take a few minutes.

@pitasi pitasi added the changelog-not-needed PRs that don't need a CHANGELOG.md entry label Aug 10, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Not building new version of mimir-build-image. This PR modifies the build image or the build image build workflow, but the image us-docker.pkg.dev/grafanalabs-dev/docker-mimir-build-image/mimir-build-image:pr16336-e30cf26c29 already exists.

@pitasi
pitasi marked this pull request as ready for review August 10, 2026 12:53
@pitasi
pitasi requested review from a team, bubu11e, vaxvms and wilfriedroset as code owners August 10, 2026 12:53
@pitasi
pitasi merged commit ce1d162 into main Aug 10, 2026
101 checks passed
@pitasi
pitasi deleted the anto/ci-build-image-zstd branch August 10, 2026 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-not-needed PRs that don't need a CHANGELOG.md entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants