openvinotoolkit / openvinotoolkit/docker_ci
Modify the Linux docker run command for systems with 2+ GPUs
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 70
- Forks
- 65
- Avg merge
- 16h 59m
- Merged PRs (30d)
- 3
Description
The docker run command line for Linux in https://github.com/openvinotoolkit/docker_ci/blob/master/docs/accelerators.md>, docker run -it --device /dev/dri --group-add=$(stat -c \"%g\" /dev/dri/render* ) $IMAGE ./samples/cpp/samples_bin/hello_query_device, leads to docker: invalid reference format error.
The reason being stat -c \"%g\" /dev/dri/render* prints more than 1 line of output and the second line of IDs is interpreted as a docker image name.
I suggest the following modification for --group-add to generalize the command line for such cases:
--group-add=$(for g in $(stat -c "%g" /dev/dri/render* | sort -u); do echo $g; done).
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Open docs/accelerators.md and inspect the Linux docker run command for systems with multiple GPUs. Update the --group-add expression so multiple render-device group IDs are handled without producing an invalid Docker image reference, then verify the documented command's shell and Docker syntax.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- docker, shell
- Domain
- devops, documentation
- Issue type
- Documentation
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 68/100