containers / podman

Podman: A tool for managing OCI containers and pods.
https://podman.io
Apache License 2.0
23.77k stars 2.42k forks source link

podman load: confusing command-line usability #2667

Closed edsantiago closed 5 years ago

edsantiago commented 5 years ago

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?

rhatdan commented 5 years ago

Load an image from a tar archive or STDIN

kunalkushwaha commented 5 years ago

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

I can work on this.

/assign

edsantiago commented 5 years ago

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

kunalkushwaha commented 5 years ago

ohh okay thanks for update :)

rhatdan commented 5 years ago

seems like this is fixed