-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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 save: add --uncompressed
#11678
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: vrothberg The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
You must be a better typer then me. Can we just use --uncompressed. I hate options with embeded "-" and I hate long hard to type options. (I know we have command completion. |
LGTM |
@mtrmac WDYT? |
0775108
to
b33af19
Compare
LGTM |
b33af19
to
4a7b354
Compare
Add an option to `podman save` to allow uncompressed layers when copying OCI images. Do the neccessary plumbing for the remote client, add tests and vendor in the latest commit from c/common to fetch the neccessary changes in libimage. Closes: containers#11613 Signed-off-by: Valentin Rothberg <[email protected]>
4a7b354
to
49c5688
Compare
--oci-accept-uncompressed-layers
--uncompressed
Updated @rhatdan PTAL |
/lgtm |
I have one question, event though I gave you a LGTM? When would a user want --uncompressed=false? |
|
Add an option to
podman save
to allow uncompressed layers whencopying OCI images. Do the neccessary plumbing for the remote client,
add tests and vendor in the latest commit from c/common to fetch
the neccessary changes in libimage.
Closes: #11613
Signed-off-by: Valentin Rothberg [email protected]