Skip to content
This repository has been archived by the owner on Jan 22, 2024. It is now read-only.

Make'd nvidia-docker doesn't parse single-character options with additional parameters correctly #46

Closed
Multihuntr opened this issue Feb 9, 2016 · 6 comments
Labels

Comments

@Multihuntr
Copy link

I have just pulled the repository and tried the sudo make install created nvidia-docker launcher. I have been using Nvidia-docker on other computers using the old script launcher.

I just installed Ubuntu on this computer, installed the NVIDIA drivers for my graphics card, CUDA, Docker, Sublime-Text and now this, so I feel confident that this is not a conflict with something else.

At the moment, if I run
nvidia-docker run -v /home/me:/home/me my_image
It tells me: Error parsing reference: "/home/me:/home/me" is not a valid repository/tag
However, if I run
nvidia-docker run --volume /home/me:/home/me my_image
It starts the image correctly.
Additionally:

  • docker run -v /home/me:/home/me my_image runs fine, and
  • I get the same error as above when I run the same command without the 'my_image' at the end.

Something similar happens for the '-p' command.

It seems to be parsing single-character options as if they have no additional parameters, and so parses the next thing passed as the image to start, and the rest as commands for the image

@3XX0
Copy link
Member

3XX0 commented Feb 9, 2016

Let me guess, you run Docker 1.10 ?
I just checked it looks like they changed their usage in the new version.

@3XX0 3XX0 added the bug label Feb 9, 2016
@Multihuntr
Copy link
Author

Yes. The latest version of Docker.

@anibali
Copy link

anibali commented Feb 16, 2016

I can confirm that upgrading from Docker 1.9 to 1.10 introduced this problem.

@3XX0
Copy link
Member

3XX0 commented Feb 19, 2016

This Docker PR broke the way we were parsing flags.

I'm not really sure how we're going to fix that. One way would be to hardcode all the boolean Docker flags. We could also rely on their package but this would effectively break some Docker versions.

@flx42
Copy link
Member

flx42 commented Feb 21, 2016

Also see this issue: moby/moby#18797

@flx42
Copy link
Member

flx42 commented Feb 27, 2016

I created a PR for Docker 1.10 (#56), I will test it over the week end with weird commands before merging it.
Please test it if you can :)

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

No branches or pull requests

4 participants