Skip to content

Commit 424c8cb

Browse files
Drop the gha cache from PR CI
A mode=max export took the job from ~45s to ~2m on an image this small, so the cache cost more than it saved. Back to a plain buildx run step, which is also how security-rebuild.yml builds. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
1 parent 14e0bb0 commit 424c8cb

1 file changed

Lines changed: 9 additions & 14 deletions

File tree

.github/workflows/pr.yml

Lines changed: 9 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -28,22 +28,17 @@ jobs:
2828
- name: Download CLI binaries
2929
run: bash downloadBinaries.sh
3030

31-
# amd64 only, and cached: this gates merges, so it has to be quick. The weekly
32-
# rebuild is the uncached multi-arch build. Equivalent locally: make smoke-test.
31+
# amd64 only: this gates merges, so it has to be quick. The weekly rebuild is
32+
# the multi-arch one. Equivalent locally: make smoke-test.
3333
#
34-
# Built through build-push-action rather than a plain `docker buildx build`
35-
# run step on purpose: type=gha needs ACTIONS_RUNTIME_TOKEN, which GitHub
36-
# exposes only to JS actions, so the cache flags silently no-op in a shell step.
34+
# Deliberately uncached. A gha cache export costs more than it saves on a
35+
# build this small, and type=gha only works from a JS action anyway, since
36+
# ACTIONS_RUNTIME_TOKEN is not exposed to run: steps.
3737
- name: Build candidate
38-
uses: docker/build-push-action@53b7df96c91f9c12dcc8a07bcb9ccacbed38856a # v7.3.0
39-
with:
40-
context: .
41-
file: ./Dockerfile
42-
platforms: linux/amd64
43-
load: true
44-
tags: dde-ci:pr
45-
cache-from: type=gha
46-
cache-to: type=gha,mode=max
38+
run: |
39+
docker buildx build --load \
40+
--platform linux/amd64 \
41+
--tag dde-ci:pr .
4742
4843
- name: Smoke-test candidate
4944
run: ./scripts/smoke-test.sh dde-ci:pr

0 commit comments

Comments
 (0)