Skip to content
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 load: confusing command-line usability #2667

Closed
edsantiago opened this issue Mar 15, 2019 · 5 comments
Closed

podman load: confusing command-line usability #2667

edsantiago opened this issue Mar 15, 2019 · 5 comments
Assignees
Labels
kind/bug Categorizes issue or PR as related to a bug. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.

Comments

@edsantiago
Copy link
Member

There's a confusing and (AFAICT) unused PATH option for podman load:

$ podman load --help
Usage:
  podman load [flags] [PATH]

The man page describes the argument differently, and non-optional:

   podman load name[:tag|@digest]

However, the option seems to be completely ignored. This is especially misleading when a naïve user types podman load foo.tar and it just hangs (waiting for input on stdin).

This one is beyond my ability because I don't know the intention behind this argument.

While you're looking into it, could you also add an isTerminal() check?

@mheon mheon added the kind/bug Categorizes issue or PR as related to a bug. label Mar 15, 2019
@rhatdan
Copy link
Member

rhatdan commented Mar 15, 2019

Load an image from a tar archive or STDIN

@kunalkushwaha
Copy link
Collaborator

kunalkushwaha commented Mar 18, 2019

Ideally the behaviour of podman load should report in both cases.

  • podman load <tar-file> i.e. argument without any flag should report error.
  • podman load with input from terminal, It should report error.

I can work on this.

/assign

@edsantiago
Copy link
Member Author

@kunalkushwaha Dan is working on it; #2674. Thank you!

@kunalkushwaha
Copy link
Collaborator

ohh okay thanks for update :)

@rhatdan
Copy link
Member

rhatdan commented Apr 13, 2019

seems like this is fixed

@rhatdan rhatdan closed this as completed Apr 13, 2019
@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 24, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/bug Categorizes issue or PR as related to a bug. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.
Projects
None yet
Development

No branches or pull requests

4 participants