From 0146692cc9e3a5351642b0cc699151e5dba2938b Mon Sep 17 00:00:00 2001 From: Liangsheng Yin Date: Fri, 19 Jun 2026 02:07:52 -0700 Subject: [PATCH] [CI] Fail fast on empty install_script in Rerun Test workflow (#28721) --- .github/workflows/rerun-test.yml | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/.github/workflows/rerun-test.yml b/.github/workflows/rerun-test.yml index 92287e1a6..3441d2807 100644 --- a/.github/workflows/rerun-test.yml +++ b/.github/workflows/rerun-test.yml @@ -112,6 +112,16 @@ jobs: if [[ "${{ inputs.runs_on }}" == "1-gpu-5090" ]]; then source /etc/profile.d/sglang-ci.sh fi + # Fail fast on an empty install_script. `bash ` is a silent + # no-op that "succeeds" without installing sglang, so the test step + # then dies with `ModuleNotFoundError: No module named 'sglang'`. + # /rerun-test resolves install_script from runner_configs.yml; a manual + # workflow_dispatch must pass it explicitly (it can't be derived from + # runs_on, which is shared across configs, e.g. default vs deepep). + if [ -z "${{ inputs.install_script }}" ]; then + echo "::error::install_script is required for cuda mode (empty would silently skip installing sglang). Pass e.g. scripts/ci/cuda/ci_install_dependency.sh" + exit 1 + fi bash ${{ inputs.install_script }} - name: Run test