Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add single command LLM deployment #3209
Add single command LLM deployment #3209
Changes from all commits
c4f968a
08377fe
fbb1f5d
449603d
bdb3f80
5b8801d
6781012
0da0b47
202d137
4161c9d
18016d5
8ab85b8
5bc4914
0009390
c5476fc
e9de819
a69c79c
5eb640f
3122616
2003bd0
de572a0
332fb43
f3508dd
61b8820
286e034
65b6480
3b1f27c
c7fdbf4
8155398
474494a
57de73e
a12dd8b
b49d8d8
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this needed
-e HUGGING_FACE_HUB_TOKEN=$token
Why not directly setexport HUGGING_FACE_HUB_TOKEN= <HUGGINGFACE_HUB_TOKEN>
Also from the security POV, does the existing command print the token?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
AFAIK the docker will not pick up env variables from the calling environment. So you would still have
which is even longer and it comes down to the same process. For a RL deployment the token variable would be set through a secret and so the token would not show up in any of the logs.
This file was deleted.