Skip to content

ci: keep docker bake cache and build targets in one solve#83

Merged
jleni merged 1 commit intomainfrom
codex/docker-publish-contract-docs
Apr 16, 2026
Merged

ci: keep docker bake cache and build targets in one solve#83
jleni merged 1 commit intomainfrom
codex/docker-publish-contract-docs

Conversation

@jleni
Copy link
Copy Markdown
Member

@jleni jleni commented Apr 16, 2026

Summary:

  • keep consumer-defined Docker Bake cache configuration intact during publish jobs
  • replace per-target bake execution with a single bake solve for all resolved targets
  • keep the earlier docs clarifications about the shared workflow contract

Why:

  • the current reusable workflow disables cache with CLI overrides (--set "*.cache-from=" and --set "*.cache-to=")
  • it also loops over resolved bake targets one-by-one, which repeats shared builder work across targets
  • in Omega this caused repeated COPY . ., bun run build, and bundle steps inside one release job

Behavior change:

  • release builds now run one docker buildx bake invocation for the resolved target set
  • registry-backed cache from consumer docker-bake.hcl files is preserved instead of being blanked by the workflow
  • metadata/digest collection still happens per resolved target after the build completes

Testing:

  • inspected the completed Omega release log for Zondax/omega run 24508764392
  • verified the reusable workflow was disabling cache and rebuilding targets one-by-one
  • patched _publish-docker-bake.yaml locally on this PR branch and confirmed the diff is clean

@jleni jleni changed the title docs: clarify docker bake publish contract ci: keep docker bake cache and build targets in one solve Apr 16, 2026
@jleni jleni marked this pull request as ready for review April 16, 2026 14:16
@jleni jleni merged commit 504ab9b into main Apr 16, 2026
1 of 2 checks passed
@jleni jleni deleted the codex/docker-publish-contract-docs branch April 16, 2026 14:16
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