Skip to content

make oras-py behave the same way as oras-go for deciding whether to unpack a layer#170

Closed
MichaelKopfMkf wants to merge 2 commits into
oras-project:mainfrom
MichaelKopfMkf:use_same_logic_for_unpacking_as_oras_go
Closed

make oras-py behave the same way as oras-go for deciding whether to unpack a layer#170
MichaelKopfMkf wants to merge 2 commits into
oras-project:mainfrom
MichaelKopfMkf:use_same_logic_for_unpacking_as_oras_go

Conversation

@MichaelKopfMkf
Copy link
Copy Markdown
Contributor

resolves #119

use the same logic as oras-go and cli for determining whether to unpack a tar layer or not.

With this implementation oras-py does not anymore use the layer's media type, it uses the annotation io.deis.oras.content.unpack.

that way artifacts published by oras-cli can be consumed the same way with oras-py and vice versa.

in addition, a skip_unpack param was added to the pull command in favor of #119

…npack a tar layer or not

Signed-off-by: Michael Kopf <michael.kopf@vector.com>
Signed-off-by: Michael Kopf <michael.kopf@vector.com>
@vsoch
Copy link
Copy Markdown
Contributor

vsoch commented Dec 3, 2024

This is great! I think I might have been involved with at least requesting / discussing the feature: oras-project/oras-go#644

I'm waiting for #171 to finish, then likely we will want to merge and rebase off of that. If they work together they can go into the same release (and we can just update the changelog to indicate that).

@vsoch
Copy link
Copy Markdown
Contributor

vsoch commented Dec 3, 2024

okay merged #171 - please rebase here and then we will test.

@stefansli
Copy link
Copy Markdown

hi @vsoch
@MichaelKopfMkf is currently not available. If this is okay for you, I can rebase his changes and create a new PR. Sadly I don't have access to his fork.

@stefansli
Copy link
Copy Markdown

stefansli commented Dec 10, 2024

Please see #179. Do you want a rebase or is regular pull+merge okay?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Disable tgz extraction on pull

3 participants