Skip to content

ci: add Docker build cache - #1647

Open
aishwaryabandapelly-ai wants to merge 1 commit into
open-telemetry:mainfrom
aishwaryabandapelly-ai:cycle2-build-performance
Open

ci: add Docker build cache#1647
aishwaryabandapelly-ai wants to merge 1 commit into
open-telemetry:mainfrom
aishwaryabandapelly-ai:cycle2-build-performance

Conversation

@aishwaryabandapelly-ai

Copy link
Copy Markdown

Summary

Adds GitHub Actions cache support to the Docker build steps in publish-docker.yml.

This updates the existing docker/build-push-action steps to use BuildKit's GitHub Actions cache backend with platform-specific cache scopes.

Changes

  • Adds a cache-scope value for each Docker build matrix platform
  • Adds cache-from: type=gha to Docker build steps
  • Adds cache-to: type=gha,mode=max to Docker build steps
  • Keeps cache scopes separate for linux/amd64 and linux/arm64

Why

This addresses #791 by improving Docker build performance through reusable Docker layer caching.

Testing

This is a GitHub Actions workflow-only change. I validated the diff locally with:

git diff --check

@aishwaryabandapelly-ai
aishwaryabandapelly-ai requested a review from a team as a code owner July 27, 2026 15:58
@linux-foundation-easycla

linux-foundation-easycla Bot commented Jul 27, 2026

Copy link
Copy Markdown

CLA Signed
The committers listed above are authorized under a signed CLA.

  • ✅ login: aishwaryabandapelly-ai / name: aishwaryabandapelly-ai (e3fb8dd)

@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Jul 27, 2026

Copy link
Copy Markdown

Pull request dashboard status

Waiting on reviewers · refreshed 2026-08-04 14:26 UTC

Review the latest changes.

Status above doesn't look right?
  • Just replied or pushed? Anything around or after the refresh time above may not be picked up yet — give it a few minutes.
  • Anything look wrong? Report it with what you expected; it helps us improve the dashboard.

@codecov

codecov Bot commented Aug 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.5%. Comparing base (b5978df) to head (e3fb8dd).
⚠️ Report is 14 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##            main   #1647     +/-   ##
=======================================
- Coverage   81.5%   81.5%   -0.1%     
=======================================
  Files        130     130             
  Lines      11616   11616             
=======================================
- Hits        9475    9473      -2     
- Misses      2141    2143      +2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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