-
-
Notifications
You must be signed in to change notification settings - Fork 5k
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
CI: Add ROCm Docker Build #2886
Merged
Merged
Changes from 29 commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
cfba4cd
CI: Add ROCm Docker Build
simon-mo d237ff6
fix name
simon-mo 03aba6b
move
simon-mo e226c59
Add sanity test
simon-mo 51d2d75
Add sanity test
simon-mo 5a70ad7
remove only the rocm container
simon-mo 2156f9e
Merge branch 'main' of github.com:vllm-project/vllm into rcom-ci
simon-mo cd910dc
migrate to buildkite
simon-mo 1c872cb
change to shell script
simon-mo e1d886f
fix typo
simon-mo 4d87cec
fix typo
simon-mo 456b76b
fix docker command
simon-mo cb7989a
fix docker command
simon-mo 7a60114
fix docker command
simon-mo 7367304
run docker sync
simon-mo ff78d36
add cmd
simon-mo 2c5277e
Merge branch 'main' of github.com:vllm-project/vllm into rcom-ci
simon-mo 21b0dbb
add outlines
simon-mo fe983cc
Revert "Dynamically configure shared memory size for moe_align_block_…
simon-mo 822bffd
add debug info
simon-mo 0125268
use test server due to outlines issue
simon-mo 5e3180d
actually use gpus
simon-mo fbc07a9
use rocm docker
simon-mo 83447e4
use better healthcheck
simon-mo 1117e46
use curl on host instead
simon-mo 1d8911f
fix curl
simon-mo ec2d7ec
reset tests
simon-mo 94fa91d
newline
simon-mo c5850ad
add comments
simon-mo 4a88632
Revert "Revert "Dynamically configure shared memory size for moe_alig…
simon-mo 06f9ccf
Merge branch 'main' of github.com:vllm-project/vllm into rcom-ci
simon-mo File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,38 @@ | ||
# This script build the ROCm docker image and run the API server inside the container. | ||
# It serves a sanity check for compilation and basic model usage. | ||
set -ex | ||
|
||
# Print ROCm version | ||
rocminfo | ||
|
||
# Try building the docker image | ||
docker build -t rocm -f Dockerfile.rocm . | ||
|
||
# Setup cleanup | ||
remove_docker_container() { docker rm -f rocm || true; } | ||
trap remove_docker_container EXIT | ||
remove_docker_container | ||
|
||
# Run the image | ||
docker run --device /dev/kfd --device /dev/dri --network host --name rocm rocm python3 -m vllm.entrypoints.api_server & | ||
|
||
# Wait for the server to start | ||
wait_for_server_to_start() { | ||
timeout=300 | ||
counter=0 | ||
|
||
while [ "$(curl -s -o /dev/null -w ''%{http_code}'' localhost:8000/health)" != "200" ]; do | ||
sleep 1 | ||
counter=$((counter + 1)) | ||
if [ $counter -ge $timeout ]; then | ||
echo "Timeout after $timeout seconds" | ||
break | ||
fi | ||
done | ||
} | ||
wait_for_server_to_start | ||
|
||
# Test a simple prompt | ||
curl -X POST -H "Content-Type: application/json" \ | ||
localhost:8000/generate \ | ||
-d '{"prompt": "San Francisco is a"}' |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
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.
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.
Why is this file reverted?
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.
Oh I just saw the PR description. I believe this can be easily fixed.
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.
@simon-mo Do you mind if I directly push a change to fix the kernel compilation error?
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.
go ahead!
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.
Oh just created a new PR #3470 to get reviewed.
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 merged #3470 Can you please rebase the PR with the current main branch?
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.
done!