Skip to content

ci: warm BuildKit cache on trusted pull requests - #44

Draft
jelias2 wants to merge 1 commit into
ethereum-optimism:mainfrom
jelias2:jelias2/warm-pr-build-cache
Draft

ci: warm BuildKit cache on trusted pull requests#44
jelias2 wants to merge 1 commit into
ethereum-optimism:mainfrom
jelias2:jelias2/warm-pr-build-cache

Conversation

@jelias2

@jelias2 jelias2 commented Jul 16, 2026

Copy link
Copy Markdown

Summary

Enable GitHub Actions BuildKit cache exports for same-repository pull requests. Fork PRs remain excluded.

Why

The reusable workflow already restores type=gha cache for every build, but only protected branches exported new cache entries. Dependency-changing PRs could not warm their own cache, so later pushes repeated expensive container compilation.

Change

  • Replace the protected-branch condition with is_fork_pr == false in build and Bake modes.
  • Preserve cache isolation by image name and platform.
  • Keep cache export disabled for forks.

Validation

  • Ruby YAML parse: passed
  • Diff whitespace check: passed

Rollout

Consumers pinned to this reusable workflow need a follow-up SHA bump after this merges. The paired op-rbuilder PR documents that dependency.

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