Skip to content

Commit 9e855b7

Browse files
authored
Adjust concurrency group for each npu workflow (#1068)
### What this PR does / why we need it? Adjust concurrency group for each npu workflow - for pd and benchmarks share the static-08-01, so only one job can runs on - other job one PR/schedule should have only 1 job runs ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? CI passed Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
1 parent afc4c0c commit 9e855b7

File tree

4 files changed

+10
-15
lines changed

4 files changed

+10
-15
lines changed

.github/workflows/accuracy_test.yaml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,8 +62,9 @@ defaults:
6262
run:
6363
shell: bash -el {0}
6464

65+
# only cancel in-progress runs of the same workflow
6566
concurrency:
66-
group: pr-${{ github.event.pull_request.number }}
67+
group: ${{ github.workflow }}-${{ github.ref }}
6768
cancel-in-progress: true
6869

6970
jobs:

.github/workflows/nightly_benchmarks.yaml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,10 @@ defaults:
3333
run:
3434
shell: bash -el {0}
3535

36+
# only 1 job can runs on static-8-01-cards
3637
concurrency:
37-
group: pr-${{ github.event.pull_request.number }}
38-
cancel-in-progress: true
38+
group: static-8-01-cards
39+
cancel-in-progress: false
3940

4041
jobs:
4142
test:

.github/workflows/vllm_ascend_test_long_term.yaml

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,9 @@ defaults:
3030
run:
3131
shell: bash -el {0}
3232

33+
# only cancel in-progress runs of the same workflow
3334
concurrency:
34-
group: pr-${{ github.event.pull_request.number }}
35+
group: ${{ github.workflow }}-${{ github.ref }}
3536
cancel-in-progress: true
3637

3738
jobs:
@@ -43,15 +44,6 @@ jobs:
4344
matrix:
4445
os: [linux-arm64-npu-1, linux-arm64-npu-4]
4546
vllm_version: [main, v0.9.0]
46-
concurrency:
47-
group: >
48-
${{
49-
matrix.os == 'linux-arm64-npu-4'
50-
&& github.event.pull_request.number
51-
&& format('pr-{0}-limit-npu-4-long-term', github.event.pull_request.number)
52-
|| format('job-{0}-{1}-{2}-long-term', matrix.os, matrix.vllm_version, github.event.pull_request.number)
53-
}}
54-
cancel-in-progress: false
5547
name: vLLM Ascend long term test
5648
runs-on: ${{ matrix.os }}
5749
container:

.github/workflows/vllm_ascend_test_pd.yaml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,10 @@ defaults:
3030
run:
3131
shell: bash -el {0}
3232

33+
# only 1 job can runs on static-8-01-cards
3334
concurrency:
34-
group: pr-${{ github.event.pull_request.number }}
35-
cancel-in-progress: true
35+
group: static-8-01-cards
36+
cancel-in-progress: false
3637

3738
jobs:
3839
prefilling-decoding-disaggregation:

0 commit comments

Comments
 (0)