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 container list: broken flag processing #2489

Closed
edsantiago opened this issue Feb 28, 2019 · 5 comments
Closed

podman container list: broken flag processing #2489

edsantiago opened this issue Feb 28, 2019 · 5 comments
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
Collaborator

#2480 does not include any of the ps flags:

$ podman container list -a
Error: unknown shorthand flag: 'a' in -a
$ podman container list --help
[ does not actually list any flags other than -h ]

Cause: not running cmd/podman/ps.go:init()

Even more fun side effect:

$ podman container list
ERRO[0000] Could not find flag trace
CONTAINER ID  IMAGE  COMMAND  CREATED  STATUS  PORTS  NAMES

(because of the c.Bool("trace") in line 197, added by #1692).

master @ 4a109a7

@edsantiago
Copy link
Collaborator Author

@baude PTAL

@mheon mheon added the kind/bug Categorizes issue or PR as related to a bug. label Feb 28, 2019
@Lagrang
Copy link

Lagrang commented Mar 3, 2019

Also flags processing broken for container command parameters.
For instance, following command podman create $IMAGE_NAME -DsomeParam=value
fails with error Error: unknown shorthand flag: 'D' in -DsomeParam=value

@rhatdan
Copy link
Member

rhatdan commented Mar 4, 2019

@Lagrang Options must come before $IMAGE_NAME

@mheon
Copy link
Member

mheon commented Mar 4, 2019

@rhatdan I think he's trying to pass the flag into the container
@Lagrang If so, 1.1.1 has a fix included

@rhatdan rhatdan closed this as completed Mar 4, 2019
@edsantiago
Copy link
Collaborator Author

Yep, looks fixed in master; thanks.

@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