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

Conflict between podman-hpc --gpu flag and new podman --gpus flag #129

Open
danfulton opened this issue Dec 13, 2024 · 0 comments
Open

Conflict between podman-hpc --gpu flag and new podman --gpus flag #129

danfulton opened this issue Dec 13, 2024 · 0 comments
Assignees

Comments

@danfulton
Copy link
Member

We use argparse to validate syntax before passing it to shared run, and to separate out valid arguments for run and exec subcommands. By default argparse allows unique abbreviations of all option flags, e.g.

--foobar
--fooba
--foo
--fo

would all be equivalent if there is only a single option --foobar on a parser. Because of this behaviour, when we construct the validation parse from the podman help page, our --gpu flag is incorrectly detected as a valid abbreviation of the new podman --gpus flag.

The result is podman-hpc run does not work at all with the --gpu flag.

I believe we can tell argparse to disallow abbreviations which should solve this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant