Skip to content

Commit 28eb857

Browse files
committed
Auto merge of #114763 - Kobzol:fix-ci-docker-caching, r=Mark-Simulacrum
CI: fix Docker layer caching As reported by `@klensy` on [Zulip](https://rust-lang.zulipchat.com/#narrow/stream/242791-t-infra/topic/docker.20images.20always.20rebuilded), Github Actions have recently updated their Docker version from 20.x to 23.x, which enabled the BuildKit build backend by default. This broke our way of performing Docker layer caching on CI, which immediately made all non-PR CI builds (including try builds) ~1 hour longer (Docker caching didn't work on PR builds before, so it wasn't affected). The moment this started happening can be seen [here](https://github.com/rust-lang-ci/rust/actions?page=2&query=branch%3Aauto+is%3Asuccess). The problem is with the following command: ``` docker history -q rust-ci | \ grep -v missing | \ xargs docker save | \ gzip | \ $upload ``` which returns the intermediate layers as `<missing>`, if BuildKit is enabled. This was investigated by `@klensy` in #114621. Thanks for that! I will continue experimenting with how we can enable the cache with BuildKit in #114762, but for the time being, I think that we should just hotfix this. This PR reverts the build backend back to the old one, which fixes the caching. However, we also have to bust the cache of all Dockerfiles, otherwise caching would only start kicking in for them the next time they are updated (or the next time GH updates their docker version). Because when the Docker version was updated the last time, the Dockerfiles were cached on S3 with basically an empty cache, and unless we bust it, even after reverting to the old build engine, the CI script would just download the empty cache and rebuild the Dockerfile from scratch, thus nullifying our fix. r? `@Mark-Simulacrum`
2 parents cbb48a5 + 6ca13d0 commit 28eb857

File tree

1 file changed

+11
-0
lines changed

1 file changed

+11
-0
lines changed

src/ci/docker/run.sh

+11
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,11 @@ if [ -f "$docker_dir/$image/Dockerfile" ]; then
6363
uname -m >> $hash_key
6464

6565
docker --version >> $hash_key
66+
67+
# Include cache version. Currently it is needed to bust Docker
68+
# cache key after opting in into the old Docker build backend.
69+
echo "1" >> $hash_key
70+
6671
cksum=$(sha512sum $hash_key | \
6772
awk '{print $1}')
6873

@@ -90,6 +95,12 @@ if [ -f "$docker_dir/$image/Dockerfile" ]; then
9095
context="$script_dir"
9196
fi
9297
echo "::group::Building docker image for $image"
98+
99+
# As of August 2023, Github Actions have updated Docker to 23.X,
100+
# which uses the BuildKit by default. It currently throws aways all
101+
# intermediate layers, which breaks our usage of S3 layer caching.
102+
# Therefore we opt-in to the old build backend for now.
103+
export DOCKER_BUILDKIT=0
93104
retry docker \
94105
build \
95106
--rm \

0 commit comments

Comments
 (0)