copy: add --dest-compress-format and --dest-compress-level #671
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mtrmac
reviewed
Jun 11, 2019
giuseppe
force-pushed
the
zstd
branch
3 times, most recently
from
4fffa42
to
d518171
Compare
giuseppe
force-pushed
the
zstd
branch
7 times, most recently
from
5b5b4ab
to
855608d
Compare
giuseppe
force-pushed
the
zstd
branch
6 times, most recently
from
e9be01d
to
32915d0
Compare
|
rebased |
giuseppe
changed the title
giuseppe
force-pushed
the
zstd
branch
2 times, most recently
from
5d17f85
to
8956b88
Compare
I've added a test to verify the compressed layer is a .zstd file |
rhatdan
reviewed
Aug 21, 2019
mtrmac
reviewed
Aug 21, 2019
giuseppe
force-pushed
the
zstd
branch
2 times, most recently
from
fe54879
to
638dba1
Compare
|
@mtrmac does it look fine now? |
Signed-off-by: Giuseppe Scrivano <[email protected]>
|
can this be reviewed? |
vrothberg
reviewed
Sep 2, 2019
vrothberg
reviewed
Sep 2, 2019
docs/skopeo-copy.1.md
Outdated
| @@ -58,6 +58,10 @@ If the authorization state is not found there, $HOME/.docker/config.json is chec | |||
|
|
|||
| Existing signatures, if any, are preserved as well. | |||
|
|
|||
| **--dest-compress-format** _format_ Specifies the compression format to use. Supported values are: `gzip` and `zstd`. | |||
|
|
|||
| **--dest-compress-level** _format_ Specifies the compression level to use. The value is specific to the compression algorithm used, e.g. for zstd the accepted values are in the range 1-20 (inclusive), while for gzip it is 1-9 (inclusive). Algorithms that don't support the setting, will ignore it. | |||
There was a problem hiding this comment.
The last sentence is now redundant since we only support gzip and zstd.
add the possibility to specify the format and the level to use when compressing blobs. Signed-off-by: Giuseppe Scrivano <[email protected]>
|
LGTM |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
add the possibility to specify the format and the level to use when
compressing blobs.
Signed-off-by: Giuseppe Scrivano [email protected]