Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OCI: Add support for zstd:chunked compression format #678

Open
idleroamer opened this issue Mar 16, 2023 · 5 comments
Open

OCI: Add support for zstd:chunked compression format #678

idleroamer opened this issue Mar 16, 2023 · 5 comments

Comments

@idleroamer
Copy link

idleroamer commented Mar 16, 2023

What is the problem you're trying to solve
Using containers on embedded devices, we strive to optimize the network bandwidth as well as on-board resource utilization like disk and memory. There is a new compression format (under discussion to be added to OCI spec opencontainers/image-spec#803) namely zstd application/vnd.oci.image.layer.v1.tar+zstd which beside providing better compression enables file level de-duplication by partial tarball pulling which would be a huge resource optimization.

A good intro to partial image pulling https://www.redhat.com/sysadmin/faster-container-image-pulls

Describe the solution you'd like
Both docker.io and quay.io registries support the new format already, see also followings for relevant efforts
https://issues.redhat.com/browse/PROJQUAY-1417
quay/quay#801

We'd like to see ACR supporting this compression format as well. Please bear in mind according to podman developers:
For such feature to work, it is important that there is no recompression happening on the registry (i.e. the original compressed blob must be retrievable unchanged) since some metadata is embedded as part of the compression format itself.

@northtyphoon northtyphoon self-assigned this Mar 21, 2023
@idleroamer idleroamer changed the title OCI: Add support for zstd compression format OCI: Add support for zstd:chunked compression format May 9, 2023
@idleroamer
Copy link
Author

There are two issues with support of zstd:chuncked specifically with podman client.

  • podman creates a multi-range http request which depending on different factors could be a big range resulting in a header request > 8k which is not allowed by ACR.
  • the next issue is that ACR does not support multi-range http request at all and returns the whole blob by such requests.

@github-actions
Copy link

github-actions bot commented Oct 7, 2023

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 30 days.

@github-actions github-actions bot added the Stale label Oct 7, 2023
@dhirschfeld
Copy link

This is a feature request, wanted by the community.

Just because no maintainer has the time or inclination to work on it right now doesn't mean it's not a desired/beneficial feature to have, so this issue should remain open unless the resolution is that zstd:chunked support won't ever be added to the ACR.

@github-actions github-actions bot removed the Stale label Oct 8, 2023
Copy link

github-actions bot commented Dec 7, 2023

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 30 days.

@github-actions github-actions bot added the Stale label Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants