Skip to content

Add TransportFromImageName procedure#631

Merged
mtrmac merged 1 commit intocontainers:masterfrom
TristanCacqueray:master
May 18, 2019
Merged

Add TransportFromImageName procedure#631
mtrmac merged 1 commit intocontainers:masterfrom
TristanCacqueray:master

Conversation

@TristanCacqueray
Copy link
Contributor

This change adds a new transports.TransportFromImageName procedure
to get the transport's name of an image without parsing the store
reference.

TristanCacqueray added a commit to TristanCacqueray/skopeo that referenced this pull request May 17, 2019
This change fixes skopeo usage in restricted environment such as
bubblewrap where it doesn't need extra capabilities or user namespace
to perform its action.

Close containers#649
Depends-On: containers/image#631
Signed-off-by: Tristan Cacqueray <tdecacqu@redhat.com>
@TristanCacqueray TristanCacqueray force-pushed the master branch 2 times, most recently from 53b44b0 to dee4fe1 Compare May 18, 2019 00:07
This change adds a new alltransports.TransportFromImageName procedure
to get the transport's name of an image without parsing the store
reference.

Signed-off-by: Tristan Cacqueray <tdecacqu@redhat.com>
Copy link
Collaborator

@mtrmac mtrmac left a comment

Choose a reason for hiding this comment

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

Thanks! LGTM pending tests.

TristanCacqueray added a commit to TristanCacqueray/skopeo that referenced this pull request May 18, 2019
Depends-On: containers/image#631
Signed-off-by: Tristan Cacqueray <tdecacqu@redhat.com>
@mtrmac mtrmac merged commit 2c0349c into containers:master May 18, 2019
TristanCacqueray added a commit to TristanCacqueray/skopeo that referenced this pull request May 18, 2019
Depends-On: containers/image#631
Signed-off-by: Tristan Cacqueray <tdecacqu@redhat.com>
TristanCacqueray added a commit to TristanCacqueray/skopeo that referenced this pull request May 18, 2019
Depends-On: containers/image#631
Signed-off-by: Tristan Cacqueray <tdecacqu@redhat.com>
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.

2 participants