-
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-remote send name and tag #7220
Conversation
@cevich i think you were looking for a failure like i am seeing? |
LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes LGTM.
Before we merge: @edsantiago is there a system test we can enable for this one? I started to become pedantic for fixes without a test change.
$ ack 7124 test/system/*.bats
test/system/120-load.bats
80: skip_if_remote "FIXME: pending #7124" |
It's not working, though: $ ./bin/podman-remote pull alpine
a24bb4013296f61e89ba57005a7b3e52274d8edd3ae2077d04395f806b63d83e
$ ./bin/podman-remote save a24bb401 -o /tmp/alpine.tar
$ ./bin/podman-remote rmi a24bb401
Untagged: docker.io/library/alpine:latest
Deleted: a24bb4013296f61e89ba57005a7b3e52274d8edd3ae2077d04395f806b63d83e
$ ./bin/podman-remote load -i /tmp/alpine.tar hellothere
Error: invalid reference format
$ echo $?
125 |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: baude 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 |
when loading an image with podman-remote load, we need to send a name and a tag to the endpoint Fixes: containers#7124 Signed-off-by: Brent Baude <[email protected]>
Weird... looks like
(full log). I'm assuming it's a flake, and have restarted the test. I have a feeling I'll be filing an issue about this soon. |
/lgtm |
when loading an image with podman-remote load, we need to send a name and a tag to the endpoint
Fixes: #7124
Signed-off-by: Brent Baude [email protected]