mediatypes: support zstd compression#9859
Conversation
|
Hi @JulienBalestra. Thanks for your PR. I'm waiting for a containerd member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Signed-off-by: Julien Balestra <julien.balestra@gmail.com>
207d649 to
d651cb7
Compare
|
So, what tools are creating images with the docker zstd mediatype? I thought the goal would be to work towards an OCI media type for zstd although I know that discussion has somewhat stalled. It would be good to understand the intention going forward since we already support zstd compression, just not using a unique media type at this point. |
|
According to moby/buildkit#3968 and comments in moby/buildkit#2344, buildkit is capable of producing |
|
How impactful is this really? It seems years ago BuildKit had an experimental support for zstd with this non-standard media type and then it full added support with the standard OCI media type. Our recommendation here would normally be to re-build anything that was built with an experimental version. Its a small change but the impact still needs to be evaluated. We usually only accept these types of changes with a comment explaining why it is there and the range of impact (which version of the builder and when it was deprecated/removed). |
You're reading moby/buildkit#3968 the wrong way. It added an ability to unpack |
|
Thank you all for your feedback !
As @slonopotamus mentioned, it's possible to use zstd compression with buildx like: docker buildx build ... \
--output compression=zstd,force-compression=true,push=true,type=image,compression-level=22Officially documented here. @estesp & @dmcgowan you would be more inclined to migrate to OCI media for use cases that require a different compression than gzip ? |
|
Latest versions of buildkit create OCI mediatypes by default but users can set |
|
@JulienBalestra I would say it is our expectation that newer functionality is using the OCI media types, but ultimately the |
|
Shouldn't this be |
|
@jonjohnsonjr For the OCI mediatypes, I would expect |
|
That is really unfortunate. |
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.
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.
|
This fixed #9263 |
This PR enables to support zstd compression over docker media types.
Before this change, containerd is unable to pull zstd compressed images and is returning the following error log: