From c60952d9330091b0db5b11b2cdbc00fd3063a935 Mon Sep 17 00:00:00 2001 From: hhhh1252023 <139246842+hhhh1252023@users.noreply.github.com> Date: Tue, 18 Aug 2026 23:48:48 +0800 Subject: [PATCH] Exclude multimodal-gen NPU jobs from fast-fail cascade (#35238) --- .github/workflows/_npu-pr-test-stage.yml | 5 +++++ .github/workflows/_npu-single-node-test-stage.yml | 5 +++++ .github/workflows/pr-test-npu.yml | 2 +- 3 files changed, 11 insertions(+), 1 deletion(-) diff --git a/.github/workflows/_npu-pr-test-stage.yml b/.github/workflows/_npu-pr-test-stage.yml index 79bb16954..d36933d8d 100644 --- a/.github/workflows/_npu-pr-test-stage.yml +++ b/.github/workflows/_npu-pr-test-stage.yml @@ -164,6 +164,11 @@ jobs: core.info(`[health-check] Filtered out h20 job: ${j.name}`); return false; } + // multimodal-gen NPU tests should not cascade fast-fail to perf/accuracy stages. + if (baseName === 'multimodal-gen-test-1-npu-a3' || baseName === 'multimodal-gen-test-2-npu-a3') { + core.info(`[health-check] Filtered out multimodal-gen NPU job: ${j.name}`); + return false; + } // If the failing step is the health check, it's a cascade — skip it const failedStep = (j.steps || []).find(s => s.conclusion === 'failure'); if (failedStep && (failedStep.name.includes('check-pr-test-health') || failedStep.name.includes('Check PR test health'))) { diff --git a/.github/workflows/_npu-single-node-test-stage.yml b/.github/workflows/_npu-single-node-test-stage.yml index 1674e444d..abcb568bb 100644 --- a/.github/workflows/_npu-single-node-test-stage.yml +++ b/.github/workflows/_npu-single-node-test-stage.yml @@ -166,6 +166,11 @@ jobs: core.info(`[health-check] Filtered out h20 job: ${j.name}`); return false; } + // multimodal-gen NPU tests should not cascade fast-fail to perf/accuracy stages. + if (baseName === 'multimodal-gen-test-1-npu-a3' || baseName === 'multimodal-gen-test-2-npu-a3') { + core.info(`[health-check] Filtered out multimodal-gen NPU job: ${j.name}`); + return false; + } // If the failing step is the health check, it's a cascade — skip it const failedStep = (j.steps || []).find(s => s.conclusion === 'failure'); if (failedStep && (failedStep.name.includes('check-pr-test-health') || failedStep.name.includes('Check PR test health'))) { diff --git a/.github/workflows/pr-test-npu.yml b/.github/workflows/pr-test-npu.yml index e4b3cde63..214b8a332 100644 --- a/.github/workflows/pr-test-npu.yml +++ b/.github/workflows/pr-test-npu.yml @@ -194,7 +194,7 @@ jobs: bash scripts/ci/npu/npu_ci_install_dependency.sh a3 # copy required file from our daily cache cp ~/.cache/modelscope/hub/datasets/otavia/ShareGPT_Vicuna_unfiltered/ShareGPT_V3_unfiltered_cleaned_split.json /tmp - # copy gsm8k dataset + # copy gsm8k dataset from cache cp ~/.cache/modelscope/hub/datasets/tmp/test.jsonl /tmp - name: Run test