Remove cancel for main to main check (#4685)
### What this PR does / why we need it? Remove cancel for main to main check. Another choice we set timeout to 4h but I think 2h to get results is more important. Related: https://github.com/vllm-project/vllm-ascend/actions/workflows/vllm_ascend_test_full_vllm_main.yaml ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Can be merged directly if lint passed - vLLM version: v0.12.0 - vLLM main: https://github.com/vllm-project/vllm/commit/v0.12.0 Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
This commit is contained in:
@@ -17,7 +17,7 @@
|
|||||||
name: 'ascend test / vllm main'
|
name: 'ascend test / vllm main'
|
||||||
|
|
||||||
on:
|
on:
|
||||||
# Run 1-card and 2-cards e2e tests per 2h
|
# Run full e2e tests per 2h
|
||||||
schedule:
|
schedule:
|
||||||
- cron: '0 */2 * * *'
|
- cron: '0 */2 * * *'
|
||||||
workflow_dispatch:
|
workflow_dispatch:
|
||||||
@@ -29,12 +29,6 @@ defaults:
|
|||||||
run:
|
run:
|
||||||
shell: bash -el {0}
|
shell: bash -el {0}
|
||||||
|
|
||||||
# only cancel in-progress runs of the same workflow
|
|
||||||
# and ignore the lint / 1 card / 4 cards test type
|
|
||||||
concurrency:
|
|
||||||
group: ${{ github.workflow }}-${{ github.ref }}
|
|
||||||
cancel-in-progress: true
|
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
e2e-test:
|
e2e-test:
|
||||||
uses: ./.github/workflows/_e2e_test.yaml
|
uses: ./.github/workflows/_e2e_test.yaml
|
||||||
|
|||||||
Reference in New Issue
Block a user