Skip to content

Add porter build cache#4827

Merged
tamirkamara merged 3 commits intomainfrom
tamirkamara/bundle-build-cache
Jan 28, 2026
Merged

Add porter build cache#4827
tamirkamara merged 3 commits intomainfrom
tamirkamara/bundle-build-cache

Conversation

@tamirkamara
Copy link
Collaborator

@tamirkamara tamirkamara commented Jan 28, 2026

What is being addressed

With a recent upgrade of Porter we can introduce build cache much like we do with the application images we build. This will help with the following:

  1. dockerhub pull throttling - since we use anonymous pulls from docker hub it's limited and we sometimes see errors in CI bundle builds in this repo. Caching will reduce this issue from appearing.
  2. The actual build time is reduced by 1-2 mins. On its own not that important but anything helps.

@github-actions
Copy link

github-actions bot commented Jan 28, 2026

Unit Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit c2ce4d4.

♻️ This comment has been updated with latest results.

@tamirkamara tamirkamara marked this pull request as ready for review January 28, 2026 07:38
@tamirkamara tamirkamara requested a review from a team as a code owner January 28, 2026 07:38
Copilot AI review requested due to automatic review settings January 28, 2026 07:38
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR introduces a Docker/ACR-backed build cache for Porter bundle builds to reduce Docker Hub pulls and improve CI build times.

Changes:

  • Extend bundle-build in the Makefile to log into an optional CI cache ACR and export its FQDN for downstream scripts.
  • Update devops/scripts/porter_build_bundle.sh to configure a Docker buildx builder and pass registry cache parameters to porter build when CI cache ACR is configured.
  • Document the enhancement in CHANGELOG.md under ENHANCEMENTS.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
devops/scripts/porter_build_bundle.sh Adds conditional Docker buildx builder setup and registry-based build cache configuration for Porter bundle builds.
Makefile Wires CI cache ACR into the bundle build flow by logging in and exporting CI_CACHE_ACR_FQDN before calling the Porter build script.
CHANGELOG.md Records the addition of Porter build cache in CI as an enhancement.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@tamirkamara tamirkamara merged commit a3f8ee0 into main Jan 28, 2026
74 checks passed
@tamirkamara tamirkamara deleted the tamirkamara/bundle-build-cache branch January 28, 2026 13:15
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.

3 participants