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

[release/1.7] Update unpacker to fetch all provided content #10233

Merged
merged 2 commits into from
May 15, 2024

Conversation

dmcgowan
Copy link
Member

When a set of layers are provided to the unpacker, then the unpacker should still fetch them regardless of whether they will be used for unpack. The image handler filters are responsible for removing content which is not intended to be fetched. Currently there is no way to use an unpacker and also fetch all platforms.

This fixes an issues where the transfer service pull could error out if specifying all platforms but only providing an unpack configuration for a subset.

Backports #10202

When a set of layers are provided to the unpacker, then the unpacker
should still fetch them regardless of whether they will be used for
unpack. The image handler filters are responsible for removing content
which is not intended to be fetched. Currently there is no way to use an
unpacker and also fetch all platforms.

Signed-off-by: Derek McGowan <derek@mcg.dev>
(cherry picked from commit 681a083)
Signed-off-by: Derek McGowan <derek@mcg.dev>
Allows supporting fetching of all platforms while unpacking for a subset
of platforms.

Signed-off-by: Derek McGowan <derek@mcg.dev>
(cherry picked from commit 2788604)
Signed-off-by: Derek McGowan <derek@mcg.dev>
@kzys kzys merged commit 62af107 into containerd:release/1.7 May 15, 2024
56 checks passed
@dmcgowan dmcgowan changed the title [release/1.7] Unpack fetch all [release/1.7] Update unpacker to fetch all provided content May 16, 2024
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.

None yet

4 participants