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

podman load doesn't work on xz compressed files #472

Closed
vrothberg opened this issue Mar 12, 2018 · 7 comments
Closed

podman load doesn't work on xz compressed files #472

vrothberg opened this issue Mar 12, 2018 · 7 comments
Assignees
Labels
locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.

Comments

@vrothberg
Copy link
Member

/kind bug

Description

podman load on xz compressed files fails.

Steps to reproduce the issue:

  1. Perform a podman load on an xz compressed file.

Describe the results you received:
open container-feeder471485113/manifest.json: not a directory

There are two things related to this issue. The first one is that the fall-through logic in PullImage is masking the actual bug, which made debugging the issue harder. I guess we could improve the situation by being more verbose when falling through the various transports. The second one is that containers/image currently doesn't support xz compressed images and compressed layers. @cyphar has fixed the latter with containers/image#427. I tested containers/image#427 locally with libpod by updating the vendored image library, which required just a minor tweak in the code to reflect the new tar API.

Describe the results you expected:
A successful image load.

@mheon
Copy link
Member

mheon commented Mar 12, 2018

Sounds like a nice, easy improvement. We can track the PR in c/image and merge once that's ready.

@baude
Copy link
Member

baude commented Mar 15, 2018

@umohnani8 you did the gz, how hard would it be to add this ?

@umohnani8
Copy link
Member

@baude If it is fixed in containers/image, just a matter of vendoring it in.

@cyphar
Copy link

cyphar commented Mar 16, 2018

I've implemented it in containers/image#427. Just going through review now.

@rhatdan
Copy link
Member

rhatdan commented Jun 4, 2018

@umohnani8 Could you follow up on this one?

@umohnani8
Copy link
Member

@rhatdan ACK. Will keep an eye on the containers/image PR for this.

@umohnani8
Copy link
Member

Fixed in #1137.

@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 24, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.
Projects
None yet
Development

No branches or pull requests

7 participants