Skip to content

build: fix digest-only oci-layout named contexts#3811

Draft
crazy-max wants to merge 2 commits intodocker:masterfrom
crazy-max:fix-oci-layout
Draft

build: fix digest-only oci-layout named contexts#3811
crazy-max wants to merge 2 commits intodocker:masterfrom
crazy-max:fix-oci-layout

Conversation

@crazy-max
Copy link
Copy Markdown
Member

@crazy-max crazy-max commented Apr 22, 2026

fixes #3793
needs #3812

Buildx now preserves the valid oci-layout://<store>@<digest> form instead of rebuilding the reference with an empty tag segment. The build input path now rebuilds OCI layout references piece by piece so it only appends a tag when one exists and only appends a digest when one exists.

Comment thread build/opt_test.go Outdated
Comment on lines +166 to +169
// FIXME: Use a relative temp dir here instead of t.TempDir(). On Windows,
// t.TempDir() returns an absolute path with a drive-letter colon,
// and this test feeds the path through an oci-layout:// ref where
// that colon can be misread as the tag separator.
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one looks like a bug in util/ocilayout on Windows. I will open a follow-up PR to fix this.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Opened #3812

Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Regression on digest-only oci-layout:// build contexts

1 participant