Skip to content

refactor(docker): consolidate dual Dockerfile into single BuildKit-cached build#8216

Merged
h0lybyte merged 1 commit intodevfrom
trunk/axum-dockerfile-1773794170
Mar 18, 2026
Merged

refactor(docker): consolidate dual Dockerfile into single BuildKit-cached build#8216
h0lybyte merged 1 commit intodevfrom
trunk/axum-dockerfile-1773794170

Conversation

@h0lybyte
Copy link
Member

Summary

  • Remove Dockerfile.base and BUILD_BASE arg pattern across all apps (axum-kbve, discordsh, cryptothrone, mc)
  • Single Dockerfile with cargo-chef + BuildKit --mount=type=cache handles dependency caching inline
  • On arc-runner, switch from GHCR registry cache to local PVC-backed cache (/home/runner/_cache/docker/) for persistent layer caching across pod restarts
  • Remove build_base CI job, has_base/base_image config outputs, basex/base Nx targets, and _base file alteration filters

Changes

  • 4 Dockerfile.base files deleted (-388 lines)
  • 4 Dockerfiles simplifiedARG BUILD_BASE removed, FROM ${BUILD_BASE}FROM foundation
  • 4 project.json filesbasex/base targets removed, BUILD_BASE build-args stripped, axum-kbve CI uses type=local cache
  • ci-docker.ymlbuild_base job removed, has_base/base_image config dropped
  • utils-file-alterations.yml — dead _base filter outputs removed

Test plan

  • Verify nx container axum-kbve builds locally (inline foundation stages)
  • Verify CI docker pipeline runs without build_base job
  • Confirm arc-runner PVC cache populates on first build and accelerates subsequent builds

@github-actions
Copy link
Contributor

github-actions bot commented Mar 18, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

…ched build

Remove Dockerfile.base and BUILD_BASE arg pattern across all apps
(axum-kbve, discordsh, cryptothrone, mc). Single Dockerfile with
cargo-chef + BuildKit cache mounts handles everything. On arc-runner,
local cache backed by persistent PVC replaces the GHCR base image
warm-up step, eliminating the build_base CI job entirely.
@h0lybyte h0lybyte force-pushed the trunk/axum-dockerfile-1773794170 branch from cc17439 to 862bde2 Compare March 18, 2026 01:02
@h0lybyte h0lybyte merged commit 7286bed into dev Mar 18, 2026
@h0lybyte h0lybyte deleted the trunk/axum-dockerfile-1773794170 branch March 18, 2026 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant