- 
                Notifications
    
You must be signed in to change notification settings  - Fork 277
 
docs: add docker compose quickstart #181
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
Conversation
Signed-off-by: JaredforReal <[email protected]>
          ✅ Deploy Preview for vllm-semantic-router ready!
 To edit notification comments on pull requests, go to your Netlify project configuration.  | 
    
          👥 vLLM Semantic Team NotificationThe following members have been identified for the changed files in this PR and have been automatically assigned: 📁 
 | 
    
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.
Can you directly move this to docs site
| 
           @Xunzhuo sure  | 
    
Signed-off-by: JaredforReal <[email protected]>
| @@ -0,0 +1,111 @@ | |||
| # Docker Compose Quick Start | |||
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.
Install with Docker Compose
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.
I don't get this. Should I add an "Install Docker Compose V2" section in the doc?
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.
added
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.
sorry i mean rename this title to Install with Docker Compose
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.
Got u now, but I think the "Install Docker Compose V2" section is worth keeping
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.
yes
Signed-off-by: JaredforReal <[email protected]>
Signed-off-by: JaredforReal <[email protected]>
Signed-off-by: JaredforReal <[email protected]>
| 
           /lgtm  | 
    
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.
thanks!
* upgrade docker/README.md with docker compose V2 quick start guide Signed-off-by: JaredforReal <[email protected]> * add docker-quickstart doc Signed-off-by: JaredforReal <[email protected]> * remove docker dir & rerank docker-quickstart in sidebar Signed-off-by: JaredforReal <[email protected]> * add install docker compose v2 section Signed-off-by: JaredforReal <[email protected]> * change title Signed-off-by: JaredforReal <[email protected]> --------- Signed-off-by: JaredforReal <[email protected]>

What type of PR is this?
FIX: Upgrade docker/README.md with docker compose V2 quick start guide
What this PR does / why we need it:
Upgrade Docker Compose V1, which is deprecated, to V2 to avoid unexpected bugs.
Add some troubleshooting based on practice.
Which issue(s) this PR fixes:
Fixes #180