Skip to content

Publish: pull each platform by its own manifest digest before the checks - #17

Merged
qisthidev merged 1 commit into
mainfrom
fix/pull-platform-manifests
Nov 8, 2026
Merged

qisthidev merged 1 commit into
mainfrom
fix/pull-platform-manifests

Conversation

@qisthidev

Copy link
Copy Markdown
Member

The first publish after #16 (run 42440752714) stopped at "Pull both architectures of that digest" on all three PHP versions:

cannot overwrite digest sha256:e6dd45de…

The runner's image store does not accept pulling one index digest for a second platform. The runtime digests were pushed, but no tag moved, so the published images are unchanged.

.github/scripts/pull-platforms.sh reads the index with docker buildx imagetools inspect --raw, picks the single linux/amd64 and linux/arm64 manifests (attestation manifests are skipped), and pulls and tags each by its own digest. It fails when either platform is missing. Both the runtime and the dev image steps use it, so each local tag also names exactly one platform for the scans.

Checked with a stub docker against the real index of that run: it selects the amd64 and arm64 manifests and fails when arm64 is removed. The publish workflow does not run on pull requests, so the next publish after merge is the real test.

🤖 Generated with Claude Code

https://claude.ai/code/session_016ZbncuXneGAhsgTaizfjVi

The runner's image store refuses a second platform pull of the same index
digest ("cannot overwrite digest"), so the first publish after #16 stopped
before the smoke test and the scans. No tag moved.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016ZbncuXneGAhsgTaizfjVi
@qisthidev
qisthidev merged commit 3835f10 into main Nov 8, 2026
7 checks passed
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