What the developer wrote
Authored by will
ci: use warp docker buildkit cache
This was inadvertently broken in #35441
What changed, and why it matters
This commit fixes a CI (Continuous Integration) configuration mistake. It restores the intended Docker cache provider for Bitcoin Core's own GitHub workflows, using 'warp' for the official bitcoin/bitcoin repository and 'gha' for forks. There is no security issue in the code change itself.
No security action needed. This is a normal CI maintenance fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The change updates three places in .github/workflows/ci.yml where the Docker provider was being read from a previous workflow job output (needs.runners.outputs.provider). That output was no longer reliably providing the right provider after PR #35441, so the workflow now explicitly selects ‘warp’ when running in the bitcoin/bitcoin repository and ‘gha’ otherwise. This is purely a CI infrastructure/cache fix.
Changed components
.github/workflows/ci.ymlInspect captured patch +3 / −3
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 4f54497a..d28e4167 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -345,7 +345,7 @@ jobs:
- name: Configure Docker
uses: ./.github/actions/configure-docker
with:
- provider: ${{ needs.runners.outputs.provider }}
+ provider: ${{ github.repository == 'bitcoin/bitcoin' && 'warp' || 'gha' }}
- name: CI script
run: ./ci/test_run_all.sh
@@ -540,7 +540,7 @@ jobs:
- name: Configure Docker
uses: ./.github/actions/configure-docker
with:
- provider: ${{ matrix.provider || needs.runners.outputs.provider }}
+ provider: ${{ matrix.provider || (github.repository == 'bitcoin/bitcoin' && 'warp' || 'gha') }}
- name: Clear unnecessary files
if: ${{ github.repository != 'bitcoin/bitcoin' || matrix.provider == 'gha' }} # Only needed on GHA runners
@@ -584,7 +584,7 @@ jobs:
- name: Configure Docker
uses: ./.github/actions/configure-docker
with:
- provider: ${{ needs.runners.outputs.provider }}
+ provider: ${{ github.repository == 'bitcoin/bitcoin' && 'warp' || 'gha' }}
- name: CI script
run: |
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.