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

[Doc] Add note for docker user #4340

Merged
merged 2 commits into from
Apr 24, 2024
Merged

Conversation

youkaichao
Copy link
Member

#4312 reported that /root/.config/vllm/nccl/cu12/libnccl.so.2.18.1 is not accessible by other users.

I tried to enable it by default, through vllm-nccl-cu12 library, but that is also dangerous, becuase people might not want their directory be readable by other users.

The final solution is to document it in doc. If a user wants the library to be read by others, they can change the permission themselves.

docs/source/serving/deploying_with_docker.rst Outdated Show resolved Hide resolved
Co-authored-by: Simon Mo <simon.mo@hey.com>
@youkaichao youkaichao enabled auto-merge (squash) April 24, 2024 20:47
@youkaichao youkaichao merged commit 2768884 into vllm-project:main Apr 24, 2024
48 checks passed
@youkaichao youkaichao deleted the docker_user branch April 24, 2024 21:20
xjpang pushed a commit to xjpang/vllm that referenced this pull request Apr 25, 2024
Co-authored-by: Simon Mo <simon.mo@hey.com>
robertgshaw2-neuralmagic pushed a commit to neuralmagic/nm-vllm that referenced this pull request Apr 26, 2024
Co-authored-by: Simon Mo <simon.mo@hey.com>
alexeykondrat pushed a commit to alexeykondrat/ci-vllm that referenced this pull request May 1, 2024
Co-authored-by: Simon Mo <simon.mo@hey.com>
z103cb pushed a commit to z103cb/opendatahub_vllm that referenced this pull request May 7, 2024
Co-authored-by: Simon Mo <simon.mo@hey.com>
Temirulan pushed a commit to Temirulan/vllm-whisper that referenced this pull request Sep 6, 2024
Co-authored-by: Simon Mo <simon.mo@hey.com>
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

Successfully merging this pull request may close these issues.

2 participants