Skip to content

Commit e1ea006

Browse files
committed
ci: include the architecture in the docker cache key
We're starting to include native aarch64 machines in our CI, but before this commit the architecture wasn't included in the cache key for our Docker images. This means there could be conflicts between images produced on different architectures, hurting our CI times. This commit fixes the problem by including the output of `uname -m` in the cache key.
1 parent 1149cf0 commit e1ea006

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

src/ci/docker/run.sh

+4
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,10 @@ if [ -f "$docker_dir/$image/Dockerfile" ]; then
5757
# Sort the file names and cat the content into the hash key
5858
sort $copied_files | xargs cat >> $hash_key
5959

60+
# Include the architecture in the hash key, since our Linux CI does not
61+
# only run in x86_64 machines.
62+
uname -m >> $hash_key
63+
6064
docker --version >> $hash_key
6165
cksum=$(sha512sum $hash_key | \
6266
awk '{print $1}')

0 commit comments

Comments
 (0)