Skip to content

Commit cfdfd54

Browse files
khluurasmith
authored andcommitted
[perf-benchmark] Fix dependency for steps in benchmark pipeline (vllm-project#11710)
1 parent e52baf6 commit cfdfd54

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

.buildkite/nightly-benchmarks/benchmark-pipeline.yaml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
steps:
22
- label: "Wait for container to be ready"
3+
key: wait-for-container-image
34
agents:
45
queue: A100
56
plugins:
@@ -10,12 +11,11 @@ steps:
1011
command:
1112
- sh .buildkite/nightly-benchmarks/scripts/wait-for-image.sh
1213

13-
- wait
14-
1514
- label: "A100"
1615
# skip: "use this flag to conditionally skip the benchmark step, useful for PR testing"
1716
agents:
1817
queue: A100
18+
depends_on: wait-for-container-image
1919
plugins:
2020
- kubernetes:
2121
podSpec:
@@ -49,6 +49,7 @@ steps:
4949
# skip: "use this flag to conditionally skip the benchmark step, useful for PR testing"
5050
agents:
5151
queue: H200
52+
depends_on: wait-for-container-image
5253
plugins:
5354
- docker#v5.12.0:
5455
image: public.ecr.aws/q9t5s3a7/vllm-ci-postmerge-repo:$BUILDKITE_COMMIT
@@ -73,7 +74,7 @@ steps:
7374
# skip: "use this flag to conditionally skip the benchmark step, useful for PR testing"
7475
agents:
7576
queue: H100
76-
depends_on: ~
77+
depends_on: wait-for-container-image
7778
plugins:
7879
- docker#v5.12.0:
7980
image: public.ecr.aws/q9t5s3a7/vllm-ci-postmerge-repo:$BUILDKITE_COMMIT

0 commit comments

Comments
 (0)