-
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: run docker-archive: ... must be a docker reference #7116
Comments
@edsantiago this error is actually correct. it may not be clear enough, but it is correct. I'm worried that trying to catch this in the remote client (as opposed to the API), will bloat the remote client by bring in containers/image stuff. wdyt? |
So |
@baude I'm totally fine with podman-remote not accepting this usage; I just think the error message may not be helpful to users. Loath as I am to suggest this, could the remote client be hacked to intercept the message and make it palatable? Maybe add a suggestion like "You need to |
A friendly reminder that this issue had no activity for 30 days. |
@edsantiago if we remove the stutter, does the message become more clear? ./bin/podman-remote run docker-archive:/tmp/alpine.tar date |
The goal is to improve errors when users use the wrong transport in certain cases we stutter, in other cases we don't give enough information. Remove stutters when failing to pull remote images, because of lack of support. Fix errors returned by reference.Parse to wrap in image that was being checked. Fixes: containers#7116 Signed-off-by: Daniel J Walsh <[email protected]>
TBH no, not really. It doesn't help me, the end user, figure out what's wrong. Would it be possible to intercept this error and offer a suggestion?
...or something along those lines, assuming that one of those would work |
In window 1:
$ ./bin/podman system service --timeout=0
In window 2:
Expected:
master @ 288ebec , root & rootless
The text was updated successfully, but these errors were encountered: