Skip to content

Commit 7614164

Browse files
committed
Revert "feat(justfile, Dockerfile): make apt cache sharing configurable"
This reverts commit 27ab1a5333cc5c5af743c6878f42da23aa5050bf.
1 parent 4ddb3a5 commit 7614164

File tree

2 files changed

+18
-28
lines changed

2 files changed

+18
-28
lines changed

Dockerfile

Lines changed: 18 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -277,13 +277,11 @@ COPY --link --from=tools-script /bin/* /bin/
277277
## Base images
278278
##
279279

280-
ARG APT_CACHE_SHARING=",sharing=locked"
281-
282280
# A Go build environment.
283281
FROM docker.io/library/golang:${GO_TAG} as go
284282
RUN --mount=type=cache,from=apt-base,source=/etc/apt,target=/etc/apt,ro \
285-
--mount=type=cache,from=apt-base,source=/var/cache/apt,target=/var/cache/apt${APT_CACHE_SHARING} \
286-
--mount=type=cache,from=apt-base,source=/var/lib/apt/lists,target=/var/lib/apt/lists${APT_CACHE_SHARING} \
283+
--mount=type=cache,from=apt-base,source=/var/cache/apt,target=/var/cache/apt,sharing=locked \
284+
--mount=type=cache,from=apt-base,source=/var/lib/apt/lists,target=/var/lib/apt/lists,sharing=locked \
287285
DEBIAN_FRONTEND=noninteractive apt-get install -y file jo jq
288286
COPY --link --from=tools-script /bin/* /usr/local/bin/
289287
COPY --link --from=tools-go /bin/* /usr/local/bin/
@@ -296,8 +294,8 @@ ENV PROTOC_NO_VENDOR=1 \
296294
# A Rust build environment.
297295
FROM docker.io/library/rust:${RUST_TAG}-slim-bookworm as rust
298296
RUN --mount=type=cache,from=apt-base,source=/etc/apt,target=/etc/apt,ro \
299-
--mount=type=cache,from=apt-base,source=/var/cache/apt,target=/var/cache/apt${APT_CACHE_SHARING} \
300-
--mount=type=cache,from=apt-base,source=/var/lib/apt/lists,target=/var/lib/apt/lists${APT_CACHE_SHARING} \
297+
--mount=type=cache,from=apt-base,source=/var/cache/apt,target=/var/cache/apt,sharing=locked \
298+
--mount=type=cache,from=apt-base,source=/var/lib/apt/lists,target=/var/lib/apt/lists,sharing=locked \
301299
DEBIAN_FRONTEND=noninteractive apt-get install -y \
302300
cmake \
303301
curl \
@@ -309,8 +307,8 @@ RUN --mount=type=cache,from=apt-base,source=/etc/apt,target=/etc/apt,ro \
309307
libssl-dev \
310308
pkg-config
311309
RUN --mount=type=cache,from=apt-llvm,source=/etc/apt,target=/etc/apt,ro \
312-
--mount=type=cache,from=apt-llvm,source=/var/cache/apt,target=/var/cache/apt${APT_CACHE_SHARING} \
313-
--mount=type=cache,from=apt-llvm,source=/var/lib/apt/lists,target=/var/lib/apt/lists${APT_CACHE_SHARING} \
310+
--mount=type=cache,from=apt-llvm,source=/var/cache/apt,target=/var/cache/apt,sharing=locked \
311+
--mount=type=cache,from=apt-llvm,source=/var/lib/apt/lists,target=/var/lib/apt/lists,sharing=locked \
314312
DEBIAN_FRONTEND=noninteractive apt-get install -y clang-19 llvm-19
315313
RUN rustup component add clippy rustfmt
316314
COPY --link --from=tools-lint /bin/checksec /usr/local/bin/
@@ -334,8 +332,8 @@ RUN rustup target add \
334332
aarch64-unknown-linux-musl \
335333
x86_64-unknown-linux-musl
336334
RUN --mount=type=cache,from=apt-base,source=/etc/apt,target=/etc/apt,ro \
337-
--mount=type=cache,from=apt-base,source=/var/cache/apt,target=/var/cache/apt${APT_CACHE_SHARING} \
338-
--mount=type=cache,from=apt-base,source=/var/lib/apt/lists,target=/var/lib/apt/lists${APT_CACHE_SHARING} \
335+
--mount=type=cache,from=apt-base,source=/var/cache/apt,target=/var/cache/apt,sharing=locked \
336+
--mount=type=cache,from=apt-base,source=/var/lib/apt/lists,target=/var/lib/apt/lists,sharing=locked \
339337
DEBIAN_FRONTEND=noninteractive apt-get install -y \
340338
binutils-aarch64-linux-gnu \
341339
g++-aarch64-linux-gnu \
@@ -348,8 +346,8 @@ RUN --mount=type=cache,from=apt-base,source=/etc/apt,target=/etc/apt,ro \
348346

349347
FROM docker.io/library/debian:bookworm as devcontainer
350348
RUN --mount=type=cache,from=apt-base,source=/etc/apt,target=/etc/apt,ro \
351-
--mount=type=cache,from=apt-base,source=/var/cache/apt,target=/var/cache/apt${APT_CACHE_SHARING} \
352-
--mount=type=cache,from=apt-base,source=/var/lib/apt/lists,target=/var/lib/apt/lists${APT_CACHE_SHARING} \
349+
--mount=type=cache,from=apt-base,source=/var/cache/apt,target=/var/cache/apt,sharing=locked \
350+
--mount=type=cache,from=apt-base,source=/var/lib/apt/lists,target=/var/lib/apt/lists,sharing=locked \
353351
DEBIAN_FRONTEND=noninteractive apt-get install -y \
354352
cmake \
355353
curl \
@@ -386,13 +384,13 @@ RUN groupadd --gid=1000 code \
386384

387385
# git v2.34+ has new subcommands and supports code signing via SSH.
388386
RUN --mount=type=cache,from=apt-base,source=/etc/apt,target=/etc/apt,ro \
389-
--mount=type=cache,from=apt-base,source=/var/cache/apt,target=/var/cache/apt${APT_CACHE_SHARING} \
390-
--mount=type=cache,from=apt-base,source=/var/lib/apt/lists,target=/var/lib/apt/lists${APT_CACHE_SHARING} \
387+
--mount=type=cache,from=apt-base,source=/var/cache/apt,target=/var/cache/apt,sharing=locked \
388+
--mount=type=cache,from=apt-base,source=/var/lib/apt/lists,target=/var/lib/apt/lists,sharing=locked \
391389
DEBIAN_FRONTEND=noninteractive apt-get install -y -t bookworm-backports git
392390

393391
RUN --mount=type=cache,from=apt-llvm,source=/etc/apt,target=/etc/apt,ro \
394-
--mount=type=cache,from=apt-llvm,source=/var/cache/apt,target=/var/cache/apt${APT_CACHE_SHARING} \
395-
--mount=type=cache,from=apt-llvm,source=/var/lib/apt/lists,target=/var/lib/apt/lists${APT_CACHE_SHARING} \
392+
--mount=type=cache,from=apt-llvm,source=/var/cache/apt,target=/var/cache/apt,sharing=locked \
393+
--mount=type=cache,from=apt-llvm,source=/var/lib/apt/lists,target=/var/lib/apt/lists,sharing=locked \
396394
DEBIAN_FRONTEND=noninteractive apt-get install -y clang-19 llvm-19
397395
ENV CC=clang-19 \
398396
CXX=clang++-19
@@ -410,16 +408,16 @@ RUN --mount=type=cache,id=apt-docker,from=apt-base,source=/etc/apt,target=/etc/a
410408
#
411409
# TODO(ver): replace this with a devcontainer feature?
412410
RUN --mount=type=cache,id=apt-docker,from=apt-base,source=/etc/apt,target=/etc/apt \
413-
--mount=type=cache,id=apt-docker,from=apt-base,source=/var/cache/apt,target=/var/cache/apt${APT_CACHE_SHARING} \
414-
--mount=type=cache,id=apt-docker,from=apt-base,source=/var/lib/apt/lists,target=/var/lib/apt/lists${APT_CACHE_SHARING} \
411+
--mount=type=cache,id=apt-docker,from=apt-base,source=/var/cache/apt,target=/var/cache/apt,sharing=locked \
412+
--mount=type=cache,id=apt-docker,from=apt-base,source=/var/lib/apt/lists,target=/var/lib/apt/lists,sharing=locked \
415413
--mount=type=bind,from=tools,source=/bin/scurl,target=/usr/local/bin/scurl \
416414
scurl https://raw.githubusercontent.com/microsoft/vscode-dev-containers/main/script-library/docker-debian.sh | bash -s
417415
ENV DOCKER_BUILDKIT=1
418416

419417
ARG MARKDOWNLINT_VERSION=0.15.0
420418
RUN --mount=type=cache,from=apt-node,source=/etc/apt,target=/etc/apt,ro \
421-
--mount=type=cache,from=apt-node,source=/var/cache/apt,target=/var/cache/apt${APT_CACHE_SHARING} \
422-
--mount=type=cache,from=apt-node,source=/var/lib/apt/lists,target=/var/lib/apt/lists${APT_CACHE_SHARING} \
419+
--mount=type=cache,from=apt-node,source=/var/cache/apt,target=/var/cache/apt,sharing=locked \
420+
--mount=type=cache,from=apt-node,source=/var/lib/apt/lists,target=/var/lib/apt/lists,sharing=locked \
423421
DEBIAN_FRONTEND=noninteractive apt-get install -y nodejs
424422
RUN npm install "markdownlint-cli2@${MARKDOWNLINT_VERSION}" --global
425423

justfile

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -33,13 +33,6 @@ _pull_policy := if _docker_bin_name == 'podman' {
3333
''
3434
}
3535

36-
# apt cache sharing mode hits a bug in buildah
37-
_apt_cache_sharing := if _docker_bin_name == 'podman' {
38-
''
39-
} else {
40-
',sharing=locked'
41-
}
42-
4336
targets := 'go rust rust-musl tools devcontainer'
4437

4538
load := 'false'
@@ -170,7 +163,6 @@ _build *args='':
170163
--progress='{{ DOCKER_PROGRESS }}' \
171164
{{ output }} \
172165
{{ if docker_arch != '' { '--platform=' + docker_arch } else { '' } }} \
173-
--build-arg APT_CACHE_SHARING={{ _apt_cache_sharing }} \
174166
{{ args }}"
175167

176168
echo "{{ style('error') }}$cmd{{ NORMAL }}"

0 commit comments

Comments
 (0)