Skip to content

Conversation

@mtrmac
Copy link
Contributor

@mtrmac mtrmac commented Nov 25, 2025

Fixes containers/skopeo#2627 .

See individual commit messages for details; note the last comment adding FIXMEs for unfixed bugs.

@podmanbot
Copy link

✅ A new PR has been created in buildah to vendor these changes: containers/buildah#6536

@mtrmac mtrmac changed the title Fix copying of non-Podman Zstd images with --preserve-digests Fix copying of non-Buildah Zstd images with --preserve-digests Nov 25, 2025
Format conversions should have been rejected earlier; this
conceptually means something else has probably changed.

That's not _actually_ reliably true at this point because
determineListConversion does not take cannotModifyManifestListReason
into account, but that's a different bug, to be fixed some other time.

Signed-off-by: Miloslav Trmač <[email protected]>
In that case, just copy the image as is, giving up on enabling the zstd
prioritization.

Signed-off-by: Miloslav Trmač <[email protected]>
Signed-off-by: Miloslav Trmač <[email protected]>
podmanbot pushed a commit to podmanbot/buildah that referenced this pull request Nov 25, 2025
Copy link
Member

@Luap99 Luap99 left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

image Related to "image" package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Skopeo copy fails with preserve-digests and OCI image with zstd compression

3 participants