From 20e42d9cf075bfe2b21b717deeedd531c003cbef Mon Sep 17 00:00:00 2001 From: local Date: Mon, 29 Sep 2025 12:04:20 -0700 Subject: [PATCH 1/2] skip hardware tests on docs-only changes; add docs_only detection in template Signed-off-by: Reza Barazesh --- buildkite/test-template-ci.j2 | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/buildkite/test-template-ci.j2 b/buildkite/test-template-ci.j2 index 206b50a0..0c283721 100644 --- a/buildkite/test-template-ci.j2 +++ b/buildkite/test-template-ci.j2 @@ -17,6 +17,12 @@ {% set hf_home_fsx = "/fsx/hf_cache" %} {% set list_file_diff = list_file_diff | split("|") %} +{% set docs_only = (list_file_diff | length > 0) %} +{% for file in list_file_diff %} +{% if file[:5] != 'docs/' %} +{% set docs_only = false %} +{% endif %} +{% endfor %} {% macro add_pytest_coverage(cmd, coverage_file) %} {% if "pytest " in cmd %} COVERAGE_FILE={{ coverage_file }} {{ cmd | replace("pytest ", "pytest --cov=vllm --cov-report= --cov-append --durations=0 ") }} || true @@ -529,6 +535,7 @@ steps: {% endif %} {% endfor %} +{% if not docs_only %} - label: "Neuron Test" depends_on: ~ agents: @@ -701,6 +708,7 @@ steps: - "vllm#tpu-ci-notifications" YAML fi +{% endif %} {% endif %} {% if cov_enabled %} From bc0413948cc82f1cbf2862152dd98d6832a3f500 Mon Sep 17 00:00:00 2001 From: local Date: Mon, 29 Sep 2025 12:27:01 -0700 Subject: [PATCH 2/2] ci(buildkite): robust docs_only calculation; avoid length on iterator Signed-off-by: local --- buildkite/test-template-ci.j2 | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/buildkite/test-template-ci.j2 b/buildkite/test-template-ci.j2 index 0c283721..46780935 100644 --- a/buildkite/test-template-ci.j2 +++ b/buildkite/test-template-ci.j2 @@ -17,12 +17,14 @@ {% set hf_home_fsx = "/fsx/hf_cache" %} {% set list_file_diff = list_file_diff | split("|") %} -{% set docs_only = (list_file_diff | length > 0) %} +{% set docs_check = namespace(all_docs=true, found_any=false) %} {% for file in list_file_diff %} +{% set docs_check.found_any = true %} {% if file[:5] != 'docs/' %} -{% set docs_only = false %} +{% set docs_check.all_docs = false %} {% endif %} {% endfor %} +{% set docs_only = (docs_check.all_docs and docs_check.found_any) %} {% macro add_pytest_coverage(cmd, coverage_file) %} {% if "pytest " in cmd %} COVERAGE_FILE={{ coverage_file }} {{ cmd | replace("pytest ", "pytest --cov=vllm --cov-report= --cov-append --durations=0 ") }} || true