Skip to content

fix(ci): resolve connector digest via buildx imagetools in manifest step (Story 45.1)#41

Merged
ALLiDoizCode merged 1 commit intomainfrom
fix/45-1-connector-digest-resolution
May 9, 2026
Merged

fix(ci): resolve connector digest via buildx imagetools in manifest step (Story 45.1)#41
ALLiDoizCode merged 1 commit intomainfrom
fix/45-1-connector-digest-resolution

Conversation

@ALLiDoizCode
Copy link
Copy Markdown
Collaborator

Fix: build-image-manifest job failure — connector digest resolution

Run 25601541595 failed at Build image-manifest.json:

[build-image-manifest] failed to resolve connector digest: could not extract digest from manifest inspect output

Root cause: docker manifest inspect for a multi-arch OCI image index (the connector image) returns a manifest list with per-arch entries, but does NOT include the index's own digest in the JSON body. The script's parsed.config?.digest lookup returns null, triggering the error.

Fix: Pre-resolve the connector digest in the workflow step using docker buildx imagetools inspect --format '{{.Manifest.Digest}}' which correctly returns the OCI image index digest, then pass it to the script via --connector-digest (the explicit path that skips the internal inspect).

Note: All 4 image builds succeeded in this run (warm cache made them 10-16× faster — town in 16m vs 6h before). Only the manifest step failed.

🤖 Generated with Claude Code

docker manifest inspect for a multi-arch OCI image index does not expose
the index digest in its JSON output — only per-arch manifest digests are
present. The build-image-manifest.mjs script then fails with "could not
extract digest from manifest inspect output".

Fix: pre-resolve the connector digest in the workflow step using
`docker buildx imagetools inspect --format '{{.Manifest.Digest}}'`
which properly handles OCI image indexes and returns the index digest.
Then pass it explicitly via --connector-digest to bypass the script's
internal inspect path.

Observed: run 25601541595 build-image-manifest job failed on this step.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ALLiDoizCode ALLiDoizCode merged commit 05be490 into main May 9, 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