Skip to content

Commit 4a664bf

Browse files
Yard1jimpang
authored andcommitted
Add basic correctness 2 GPU tests to 4 GPU pipeline (vllm-project#5518)
1 parent 3914e55 commit 4a664bf

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

.buildkite/test-pipeline.yaml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ steps:
3232
working_dir: "/vllm-workspace/tests"
3333
num_gpus: 2
3434

35-
- label: Distributed Tests
35+
- label: Distributed Tests (2 GPUs)
3636
mirror_hardwares: [amd]
3737
working_dir: "/vllm-workspace/tests"
3838
num_gpus: 2
@@ -50,12 +50,16 @@ steps:
5050
- CUDA_VISIBLE_DEVICES=0,1 pytest -v -s test_sharded_state_loader.py
5151
- CUDA_VISIBLE_DEVICES=0,1 pytest -v -s distributed/test_utils.py
5252

53-
- label: Distributed Tests (Multiple Groups)
53+
- label: Distributed Tests (4 GPUs)
5454
#mirror_hardwares: [amd]
5555
working_dir: "/vllm-workspace/tests"
5656
num_gpus: 4
5757
commands:
5858
- pytest -v -s distributed/test_pynccl.py
59+
# We want to test that models which use 2 GPUs work with 4 GPUs, which is why we duplicate them here.
60+
# See https://github.com/vllm-project/vllm/pull/5473#issuecomment-2166601837 for context.
61+
- TEST_DIST_MODEL=facebook/opt-125m DISTRIBUTED_EXECUTOR_BACKEND=ray pytest -v -s distributed/test_basic_distributed_correctness.py
62+
- TEST_DIST_MODEL=facebook/opt-125m DISTRIBUTED_EXECUTOR_BACKEND=mp pytest -v -s distributed/test_basic_distributed_correctness.py
5963

6064
- label: Engine Test
6165
mirror_hardwares: [amd]

0 commit comments

Comments
 (0)