Skip to content

Revert zstd build due to incompatibility with containerd 1.7#609

Merged
EricMountain merged 1 commit intov1.17-ddfrom
eric.mountain/cilium-zstd-revert
May 27, 2025
Merged

Revert zstd build due to incompatibility with containerd 1.7#609
EricMountain merged 1 commit intov1.17-ddfrom
eric.mountain/cilium-zstd-revert

Conversation

@EricMountain
Copy link
Copy Markdown
Member

buildx is producing application/vnd.docker.image.rootfs.diff.tar.zstd layers which are incompatible with containerd 1.7 as it is lacking containerd/containerd#9859.

Revert while we look for a solution.

CMPT-3383

buildx is producing `application/vnd.docker.image.rootfs.diff.tar.zstd`
layers which are incompatible with containerd 1.7 as it is lacking
containerd/containerd#9859.

Revert while we look for a solution.
--target debug \
--push \
--metadata-file "$METADATA_FILE_DEBUG" \
--output type=image,push=true,compression=zstd,force-compression=true \
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I know there is an output flag in buildx to output images with oci media types. Maybe this is just what's missing for this to work in containers 1.7?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

Yes, --oci-mediatypes this is what I’m going to look into next.

@EricMountain EricMountain merged commit edc6128 into v1.17-dd May 27, 2025
5 of 7 checks passed
@EricMountain EricMountain deleted the eric.mountain/cilium-zstd-revert branch May 27, 2025 07:39
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.

2 participants