Skip to content

versions: an image's pin is asked of its registry, not of docker - #44

Merged
aledbf merged 1 commit into
mainfrom
versions/digest-from-registry
Oct 17, 2026
Merged

aledbf merged 1 commit into
mainfrom
versions/digest-from-registry

Conversation

@aledbf

@aledbf aledbf commented Oct 17, 2026 •

Copy link
Copy Markdown
Contributor

ami's first weekly refresh (refresh.yml) failed on versions bump go: on the Blacksmith runner docker buildx imagetools inspect --format '{{.Manifest.Digest}}' printed its whole listing, and that became the pin (the pin "Name: mirror.gcr.io/library/golang:..." is not a sha256: digest).

resolve now asks the registry itself with go-containerregistry's crane.Digest (already in the module graph, now direct), using the same keychain docker reads (~/.docker/config.json, credential helpers). It returns the digest of what the tag names: for a multi-platform image the index, which is what was pinned before.

Checked:

  • task test and task lint. The test runs a registry in-process (pkg/registry, plain HTTP for a localhost reference): a missing tag is an error, and a tag's pin is its index's digest, not another tag's.
  • task mutate BASE=origin/main: 0 survived.
  • Against the real registries, with ami's versions.yaml: bump go and bump wolfi-base give the same digest as before, and bump python gives the one versions check reported.

Followup: ami bumps go-tools to this and dispatches Refresh again.

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

The buildx Blacksmith's builder installs printed its whole listing for
imagetools inspect --format, and ami's weekly refresh wrote that as go's pin.
crane.Digest asks the registry for the tag's manifest digest with the same
keychain docker reads; the test runs a registry in-process.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@aledbf
aledbf merged commit da91217 into main Oct 17, 2026
1 check passed
@aledbf
aledbf deleted the versions/digest-from-registry branch October 17, 2026 21:06
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