Skip to content

Conversation

@mtrmac
Copy link
Collaborator

@mtrmac mtrmac commented Oct 17, 2024

... and, also, indirectly, confirm the hypothesis in containers/podman#24308 .

@mtrmac mtrmac added the kind/feature A request for, or a PR adding, new functionality label Oct 17, 2024
@mtrmac
Copy link
Collaborator Author

mtrmac commented Oct 17, 2024

Confirmed in manual testing:

# bin/podman --storage-driver vfs pull quay.io/libpod/testimage:20241010
Trying to pull quay.io/libpod/testimage:20241010...
Getting image source signatures
Copying blob a3ed95caeb02 done   | 
Error: copying system image from manifest list: reading blob sha256:…: partial layer pull is not supported by the graph driver, and fallback is not allowed: not supported

@mtrmac mtrmac marked this pull request as draft October 18, 2024 19:05
@mtrmac mtrmac force-pushed the partial-not-supported branch from 5fd9d1b to e1d3ad1 Compare October 18, 2024 20:06
@mtrmac
Copy link
Collaborator Author

mtrmac commented Oct 18, 2024

Rewritten to just add / improve context. Please review again.

Right now, we should see

copying system image from manifest list: partial pull of blob sha256:…: staging a partially-pulled layer: not supported

and with containers/storage#2140:

copying system image from manifest list: partial pull of blob sha256:…: graph driver vfs does not support partial pull but convert_images requires that

@mtrmac mtrmac marked this pull request as ready for review October 18, 2024 20:09
Copy link
Member

@vrothberg vrothberg left a comment

Choose a reason for hiding this comment

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

Code LGTM

@rhatdan
Copy link
Member

rhatdan commented Oct 22, 2024

LGTM

@mtrmac mtrmac force-pushed the partial-not-supported branch 2 times, most recently from 03946a8 to ffaffd6 Compare October 24, 2024 13:03
- Diferentiate partial pull errors from other read errors
- Indicate that this is PrepareStagedLayer that is failing

Signed-off-by: Miloslav Trmač <[email protected]>
@mtrmac mtrmac force-pushed the partial-not-supported branch from ffaffd6 to acaf72a Compare October 25, 2024 20:39
@mtrmac mtrmac merged commit 94e0c4f into containers:main Oct 25, 2024
@mtrmac mtrmac deleted the partial-not-supported branch October 25, 2024 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/feature A request for, or a PR adding, new functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants