Skip to content

Commit 33f015b

Browse files
authored
[CI] Introduce a default features to pre-set 'pass' status in the support matrix (#1026)
Signed-off-by: Teresa Chen <boe20211@gmail.com>
1 parent e88f387 commit 33f015b

8 files changed

+82
-206
lines changed

.buildkite/features/Chunked_Prefill.yml

Lines changed: 0 additions & 40 deletions
This file was deleted.

.buildkite/features/JAX-Path_Qxix_Quantization.yml

Lines changed: 19 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,27 +1,27 @@
11
# JAX-Path Qxix Quantization
22
steps:
3-
- label: "Correctness tests for JAX-Path Qxix Quantization"
4-
key: "JAX-Path_Qxix_Quantization_CorrectnessTest"
5-
soft_fail: true
6-
agents:
7-
queue: tpu_v6e_queue
8-
commands:
9-
- echo "covered by performance tests"
10-
- label: "Record correctness test result for JAX-Path Qxix Quantization"
11-
key: "record_JAX-Path_Qxix_Quantization_CorrectnessTest"
12-
depends_on: "JAX-Path_Qxix_Quantization_CorrectnessTest"
13-
env:
14-
CI_TARGET: "JAX-Path Qxix Quantization"
15-
CI_STAGE: "CorrectnessTest"
16-
agents:
17-
queue: cpu
18-
commands:
19-
- |
20-
.buildkite/scripts/record_step_result.sh JAX-Path_Qxix_Quantization_CorrectnessTest
3+
# - label: "Correctness tests for JAX-Path Qxix Quantization"
4+
# key: "JAX-Path_Qxix_Quantization_CorrectnessTest"
5+
# soft_fail: true
6+
# agents:
7+
# queue: tpu_v6e_queue
8+
# commands:
9+
# - echo "covered by performance tests"
10+
# - label: "Record correctness test result for JAX-Path Qxix Quantization"
11+
# key: "record_JAX-Path_Qxix_Quantization_CorrectnessTest"
12+
# depends_on: "JAX-Path_Qxix_Quantization_CorrectnessTest"
13+
# env:
14+
# CI_TARGET: "JAX-Path Qxix Quantization"
15+
# CI_STAGE: "CorrectnessTest"
16+
# agents:
17+
# queue: cpu
18+
# commands:
19+
# - |
20+
# .buildkite/scripts/record_step_result.sh JAX-Path_Qxix_Quantization_CorrectnessTest
2121

2222
- label: "Performance tests for JAX-Path Qxix Quantization"
2323
key: "JAX-Path_Qxix_Quantization_PerformanceTest"
24-
depends_on: "record_JAX-Path_Qxix_Quantization_CorrectnessTest"
24+
# depends_on: "record_JAX-Path_Qxix_Quantization_CorrectnessTest"
2525
soft_fail: true
2626
env:
2727
QUANTIZATION: "True"

.buildkite/features/Prefix_Caching.yml

Lines changed: 0 additions & 40 deletions
This file was deleted.

.buildkite/features/Quantized_Matmul_Attention_and_KV_Cache.yml

Lines changed: 19 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,27 +1,27 @@
11
# Quantized Matmul Attention and KV Cache
22
steps:
3-
- label: "Correctness tests for Quantized Matmul Attention and KV Cache"
4-
key: "Quantized_Matmul_Attention_and_KV_Cache_CorrectnessTest"
5-
soft_fail: true
6-
agents:
7-
queue: cpu
8-
commands:
9-
- echo "covered by performance test"
10-
- label: "Record correctness test result for Quantized Matmul Attention and KV Cache"
11-
key: "record_Quantized_Matmul_Attention_and_KV_Cache_CorrectnessTest"
12-
depends_on: "Quantized_Matmul_Attention_and_KV_Cache_CorrectnessTest"
13-
env:
14-
CI_TARGET: "Quantized Matmul Attention and KV Cache"
15-
CI_STAGE: "CorrectnessTest"
16-
agents:
17-
queue: cpu
18-
commands:
19-
- |
20-
.buildkite/scripts/record_step_result.sh Quantized_Matmul_Attention_and_KV_Cache_CorrectnessTest
3+
# - label: "Correctness tests for Quantized Matmul Attention and KV Cache"
4+
# key: "Quantized_Matmul_Attention_and_KV_Cache_CorrectnessTest"
5+
# soft_fail: true
6+
# agents:
7+
# queue: cpu
8+
# commands:
9+
# - echo "covered by performance test"
10+
# - label: "Record correctness test result for Quantized Matmul Attention and KV Cache"
11+
# key: "record_Quantized_Matmul_Attention_and_KV_Cache_CorrectnessTest"
12+
# depends_on: "Quantized_Matmul_Attention_and_KV_Cache_CorrectnessTest"
13+
# env:
14+
# CI_TARGET: "Quantized Matmul Attention and KV Cache"
15+
# CI_STAGE: "CorrectnessTest"
16+
# agents:
17+
# queue: cpu
18+
# commands:
19+
# - |
20+
# .buildkite/scripts/record_step_result.sh Quantized_Matmul_Attention_and_KV_Cache_CorrectnessTest
2121

2222
- label: "Performance tests for Quantized Matmul Attention and KV Cache"
2323
key: "Quantized_Matmul_Attention_and_KV_Cache_PerformanceTest"
24-
depends_on: "record_Quantized_Matmul_Attention_and_KV_Cache_CorrectnessTest"
24+
# depends_on: "record_Quantized_Matmul_Attention_and_KV_Cache_CorrectnessTest"
2525
soft_fail: true
2626
agents:
2727
queue: tpu_v6e_8_queue

.buildkite/features/Ragged_Paged_Attention_V3.yml

Lines changed: 0 additions & 40 deletions
This file was deleted.

.buildkite/features/Single_Program_Multi_Data.yml

Lines changed: 0 additions & 40 deletions
This file was deleted.
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
Chunked Prefill
2+
Prefix Caching
3+
Ragged Paged Attention V3
4+
Single Program Multi Data

.buildkite/scripts/generate_support_matrices.sh

Lines changed: 40 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,12 @@ ANY_FAILED=false
55

66
MODEL_LIST_KEY="model-list"
77
FEATURE_LIST_KEY="feature-list"
8+
DEFAULT_FEATURES_FILE=".buildkite/features/default_features.txt"
89

910
# Note: This script assumes the metadata keys contain newline-separated lists.
1011
# The `mapfile` command reads these lists into arrays, correctly handling spaces.
1112
mapfile -t model_list < <(buildkite-agent meta-data get "${MODEL_LIST_KEY}" --default "")
12-
mapfile -t feature_list < <(buildkite-agent meta-data get "${FEATURE_LIST_KEY}" --default "")
13+
mapfile -t metadata_feature_list < <(buildkite-agent meta-data get "${FEATURE_LIST_KEY}" --default "")
1314
MODEL_STAGES=("UnitTest" "IntegrationTest" "Benchmark")
1415
FEATURE_STAGES=("CorrectnessTest" "PerformanceTest")
1516

@@ -20,8 +21,18 @@ echo "Model,UnitTest,IntegrationTest,Benchmark" > "$model_support_matrix_csv"
2021
feature_support_matrix_csv="feature_support_matrix.csv"
2122
echo "Feature,CorrectnessTest,PerformanceTest" > "$feature_support_matrix_csv"
2223

24+
# Read the list of default features from the specified file
25+
if [[ -f "${DEFAULT_FEATURES_FILE}" ]]; then
26+
mapfile -t default_feature_list < <(sed 's/\r$//; /^$/d' "${DEFAULT_FEATURES_FILE}")
27+
else
28+
default_feature_list=()
29+
echo "Warning: Default features file not found at ${DEFAULT_FEATURES_FILE}"
30+
fi
31+
2332
process_models() {
24-
for model in "$@"; do
33+
local row
34+
local result
35+
for model in "${model_list[@]:-}"; do
2536
row="\"$model\""
2637
for stage in "${MODEL_STAGES[@]}"; do
2738
result=$(buildkite-agent meta-data get "${model}:${stage}" --default "N/A")
@@ -35,25 +46,46 @@ process_models() {
3546
}
3647

3748
process_features() {
38-
for feature in "$@"; do
39-
row="\"$feature\""
49+
declare -A feature_rows
50+
local result
51+
# Process features from the default list
52+
for feature in "${default_feature_list[@]:-}"; do
53+
if [[ -z "$feature" ]]; then continue; fi
54+
local row="\"$feature\""
55+
for stage in "${FEATURE_STAGES[@]}"; do
56+
result=""
57+
row="$row,$result"
58+
done
59+
feature_rows["$feature"]="$row"
60+
done
61+
# Process features from the metadata list
62+
for feature in "${metadata_feature_list[@]:-}"; do
63+
if [[ -z "$feature" ]]; then continue; fi
64+
if [[ -v feature_rows["$feature"] ]]; then
65+
continue
66+
fi
67+
local row="\"$feature\""
4068
for stage in "${FEATURE_STAGES[@]}"; do
4169
result=$(buildkite-agent meta-data get "${feature}:${stage}" --default "N/A")
4270
row="$row,$result"
4371
if [ "${result}" != "" ] && [ "${result}" != "N/A" ] ; then
4472
ANY_FAILED=true
4573
fi
4674
done
47-
echo "$row" >> "$feature_support_matrix_csv"
75+
feature_rows["$feature"]="$row"
4876
done
77+
# Output all unique rows, sorted, to the CSV file
78+
for row in "${feature_rows[@]}"; do
79+
echo "$row"
80+
done | sort -V >> "$feature_support_matrix_csv"
4981
}
5082

5183
if [ ${#model_list[@]} -gt 0 ]; then
52-
process_models "${model_list[@]}"
84+
process_models
5385
fi
5486

55-
if [ ${#feature_list[@]} -gt 0 ]; then
56-
process_features "${feature_list[@]}"
87+
if [ ${#metadata_feature_list[@]} -gt 0 ] || [ ${#default_feature_list[@]} -gt 0 ]; then
88+
process_features
5789
fi
5890

5991
buildkite-agent meta-data set "CI_TESTS_FAILED" "${ANY_FAILED}"

0 commit comments

Comments
 (0)