Skip to content

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

Merged
kzys merged 2 commits intocontainerd:release/1.7from
dmcgowan:1.7-unpack-fetch-all
May 15, 2024
Merged

[release/1.7] Update unpacker to fetch all provided content#10233
kzys merged 2 commits intocontainerd:release/1.7from
dmcgowan:1.7-unpack-fetch-all

Conversation

@dmcgowan
Copy link
Copy Markdown
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

dmcgowan added 2 commits May 15, 2024 11:54
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 <[email protected]>
(cherry picked from commit 681a083)
Signed-off-by: Derek McGowan <[email protected]>
Allows supporting fetching of all platforms while unpacking for a subset
of platforms.

Signed-off-by: Derek McGowan <[email protected]>
(cherry picked from commit 2788604)
Signed-off-by: Derek McGowan <[email protected]>
@kzys kzys merged commit 62af107 into containerd:release/1.7 May 15, 2024
@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.

4 participants