From d7cb08c5be3bec309b13942cca02009aafd10ad2 Mon Sep 17 00:00:00 2001 From: Kangyan-Zhou Date: Sat, 29 Nov 2025 12:32:06 -0800 Subject: [PATCH] Always run all stages in cron based PR tests (#14151) --- .github/workflows/pr-test.yml | 78 ++++++++++++++++++++++++++--------- 1 file changed, 59 insertions(+), 19 deletions(-) diff --git a/.github/workflows/pr-test.yml b/.github/workflows/pr-test.yml index 67138caf6..eb9dc37e2 100644 --- a/.github/workflows/pr-test.yml +++ b/.github/workflows/pr-test.yml @@ -337,7 +337,9 @@ jobs: stage-a-test-1: needs: [check-changes, call-gate, sgl-kernel-build-wheels] - if: always() && !failure() && !cancelled() && + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && ((needs.check-changes.outputs.main_package == 'true') || (needs.check-changes.outputs.sgl_kernel == 'true')) runs-on: 1-gpu-runner env: @@ -369,7 +371,10 @@ jobs: multimodal-gen-test-1-gpu: needs: [check-changes, call-gate, sgl-kernel-build-wheels] - if: (always() && !failure() && !cancelled()) && needs.check-changes.outputs.multimodal_gen == 'true' + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && + needs.check-changes.outputs.multimodal_gen == 'true' runs-on: 1-gpu-runner strategy: fail-fast: false @@ -407,7 +412,10 @@ jobs: multimodal-gen-test-2-gpu: needs: [check-changes, call-gate, sgl-kernel-build-wheels] - if: (always() && !failure() && !cancelled()) && needs.check-changes.outputs.multimodal_gen == 'true' + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && + needs.check-changes.outputs.multimodal_gen == 'true' runs-on: 2-gpu-runner strategy: fail-fast: false @@ -445,7 +453,9 @@ jobs: quantization-test: needs: [check-changes, call-gate, stage-a-test-1] - if: always() && !failure() && !cancelled() && + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && ((needs.check-changes.outputs.main_package == 'true') || (needs.check-changes.outputs.sgl_kernel == 'true')) runs-on: 1-gpu-runner steps: @@ -472,7 +482,9 @@ jobs: unit-test-backend-1-gpu: needs: [check-changes, call-gate, stage-a-test-1] - if: always() && !failure() && !cancelled() && + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && ((needs.check-changes.outputs.main_package == 'true') || (needs.check-changes.outputs.sgl_kernel == 'true')) runs-on: 1-gpu-runner env: @@ -506,7 +518,9 @@ jobs: unit-test-backend-2-gpu: needs: [check-changes, call-gate, unit-test-backend-1-gpu] - if: always() && !failure() && !cancelled() && + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && ((needs.check-changes.outputs.main_package == 'true') || (needs.check-changes.outputs.sgl_kernel == 'true')) runs-on: 2-gpu-runner env: @@ -539,7 +553,9 @@ jobs: unit-test-backend-4-gpu: needs: [check-changes, call-gate, unit-test-backend-2-gpu] - if: always() && !failure() && !cancelled() && + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && ((needs.check-changes.outputs.main_package == 'true') || (needs.check-changes.outputs.sgl_kernel == 'true')) runs-on: 4-gpu-h100 env: @@ -572,7 +588,9 @@ jobs: unit-test-backend-8-gpu-h200: needs: [check-changes, call-gate, unit-test-backend-2-gpu] - if: always() && !failure() && !cancelled() && + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && ((needs.check-changes.outputs.main_package == 'true') || (needs.check-changes.outputs.sgl_kernel == 'true')) runs-on: 8-gpu-h200 env: @@ -605,7 +623,9 @@ jobs: unit-test-backend-8-gpu-h20: needs: [check-changes, call-gate, unit-test-backend-2-gpu] - if: always() && !failure() && !cancelled() && + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && ((needs.check-changes.outputs.main_package == 'true') || (needs.check-changes.outputs.sgl_kernel == 'true')) runs-on: 8-gpu-h20 env: @@ -639,7 +659,9 @@ jobs: performance-test-1-gpu-part-1: needs: [check-changes, call-gate, stage-a-test-1] - if: always() && !failure() && !cancelled() && + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && ((needs.check-changes.outputs.main_package == 'true') || (needs.check-changes.outputs.sgl_kernel == 'true')) runs-on: 1-gpu-runner env: @@ -700,7 +722,9 @@ jobs: performance-test-1-gpu-part-2: needs: [check-changes, call-gate, stage-a-test-1] - if: always() && !failure() && !cancelled() && + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && ((needs.check-changes.outputs.main_package == 'true') || (needs.check-changes.outputs.sgl_kernel == 'true')) runs-on: 1-gpu-runner env: @@ -753,7 +777,9 @@ jobs: performance-test-1-gpu-part-3: needs: [check-changes, call-gate, stage-a-test-1] - if: always() && !failure() && !cancelled() && + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && ((needs.check-changes.outputs.main_package == 'true') || (needs.check-changes.outputs.sgl_kernel == 'true')) runs-on: 1-gpu-runner env: @@ -800,7 +826,9 @@ jobs: performance-test-2-gpu: needs: [check-changes, call-gate, unit-test-backend-2-gpu] - if: always() && !failure() && !cancelled() && + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && ((needs.check-changes.outputs.main_package == 'true') || (needs.check-changes.outputs.sgl_kernel == 'true')) runs-on: 2-gpu-runner env: @@ -859,7 +887,9 @@ jobs: accuracy-test-1-gpu: needs: [check-changes, call-gate, stage-a-test-1] - if: always() && !failure() && !cancelled() && + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && ((needs.check-changes.outputs.main_package == 'true') || (needs.check-changes.outputs.sgl_kernel == 'true')) runs-on: 1-gpu-runner env: @@ -891,7 +921,9 @@ jobs: accuracy-test-2-gpu: needs: [check-changes, call-gate, accuracy-test-1-gpu] - if: always() && !failure() && !cancelled() && + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && ((needs.check-changes.outputs.main_package == 'true') || (needs.check-changes.outputs.sgl_kernel == 'true')) runs-on: 2-gpu-runner env: @@ -923,7 +955,9 @@ jobs: unit-test-deepep-4-gpu: needs: [check-changes, call-gate, unit-test-backend-2-gpu] - if: always() && !failure() && !cancelled() && + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && ((needs.check-changes.outputs.main_package == 'true') || (needs.check-changes.outputs.sgl_kernel == 'true')) runs-on: 4-gpu-h100 env: @@ -952,7 +986,9 @@ jobs: unit-test-deepep-8-gpu: needs: [check-changes, call-gate, unit-test-backend-2-gpu] - if: always() && !failure() && !cancelled() && + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && ((needs.check-changes.outputs.main_package == 'true') || (needs.check-changes.outputs.sgl_kernel == 'true')) runs-on: 8-gpu-h200 env: @@ -981,7 +1017,9 @@ jobs: unit-test-backend-4-gpu-b200: needs: [check-changes, call-gate, unit-test-backend-2-gpu] - if: always() && !failure() && !cancelled() && + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && ((needs.check-changes.outputs.main_package == 'true') || (needs.check-changes.outputs.sgl_kernel == 'true')) runs-on: 4-gpu-b200 env: @@ -1015,7 +1053,9 @@ jobs: unit-test-backend-4-gpu-gb200: needs: [check-changes, call-gate, unit-test-backend-2-gpu, sgl-kernel-build-wheels-arm] - if: always() && !failure() && !cancelled() && + if: | + always() && + (github.event_name == 'schedule' || (!failure() && !cancelled())) && ((needs.check-changes.outputs.main_package == 'true') || (needs.check-changes.outputs.sgl_kernel == 'true')) runs-on: 4-gpu-gb200 env: