Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions .github/workflows/configs/weekly_config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -85,8 +85,8 @@ a3:
config_file_path: DeepSeek_V3.2T_MTP3_PD.yaml
config_base_path: tests/e2e/weekly/multi_node/external_dp/config
size: 4
- name: Qwen3.5-122B-A10B-w4a8-128k-1k-TPOT50-90-PD
config_file_path: Qwen3.5-122B-A10B-w4a8-128k-1k-TPOT50-90-PD.yaml
- name: Qwen3.5-122B-A10B-w4a8-128k-1k-PD
config_file_path: Qwen3.5-122B-A10B-w4a8-128k-1k-PD.yaml
config_base_path: tests/e2e/weekly/multi_node/external_dp/config
size: 3
- name: GLM-5.1-W8A8C8-3.5k-1.5k-0-20-PD
Expand Down Expand Up @@ -255,8 +255,8 @@ a3:
config_file_path: Qwen3.5-122B-A10B-w4a8-984k-1k-0-PD.yaml
config_base_path: tests/e2e/weekly/multi_node/external_dp/config
size: 2
- name: Qwen3.5-122B-A10B-w4a8-16k-1k-TPOT50-0-PD
config_file_path: Qwen3.5-122B-A10B-w4a8-16k-1k-TPOT50-0-PD.yaml
- name: Qwen3.5-122B-A10B-w4a8-16k-1k-PD
config_file_path: Qwen3.5-122B-A10B-w4a8-16k-1k-PD.yaml
config_base_path: tests/e2e/weekly/multi_node/external_dp/config
size: 2
single_node:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
test_name: "Qwen3.5-122B-A10B-w4a8-128k-1k-TPOT50-90-PD"
test_name: "Qwen3.5-122B-A10B-w4a8-128k-1k-PD"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

The PR title and summary do not fully adhere to the repository's PR Summary Style Guide. Specifically, the PR title is missing the [Action] tag (e.g., [BugFix] or [Misc]), and the PR summary should be formatted as per the style guide instructions.

Here are the suggested PR Title and PR Summary:

Suggested PR Title:

[CI][BugFix] Modify the test case name and dataset configuration

Suggested PR Summary:

### What this PR does / why we need it?

1. **Qwen3.5-122B Use Case**: Shortens the test case names for Qwen3.5-122B multi-node configurations to prevent Kubernetes scheduling failures. Kubernetes restricts container names (which are formed by concatenating test case names with other characters) to a maximum of 64 characters.
2. **Qwen3.6-35B Use Case**: Corrects and updates the dataset paths in the Qwen3.6-35B single-node configuration to ensure correct dataset configurations are used during testing.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Tested by running the modified end-to-end weekly test cases.
References
  1. PR Title and Summary must follow the specified formats, with the PR Title containing [Branch][Module][Action] prefixes and the PR Summary containing specific markdown sections. (link)

model: "Eco-Tech/Qwen3.5-122B-A10B-w4a8"
num_nodes: 3
npu_per_node: 16
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
test_name: "Qwen3.5-122B-A10B-w4a8-16k-1k-TPOT50-0-PD"
test_name: "Qwen3.5-122B-A10B-w4a8-16k-1k-PD"
model: "Eco-Tech/Qwen3.5-122B-A10B-w4a8"
num_nodes: 2
npu_per_node: 16
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ test_cases:
benchmarks:
perf:
case_type: performance
dataset_path: vllm-ascend/GSM8K_in1007616_bs16_prefix0_qwen
dataset_path: vllm-ascend/GSM8K-in1007616-bs16-prefix0-qwen
request_conf: vllm_api_stream_chat
dataset_conf: gsm8k/gsm8k_gen_0_shot_cot_str_perf
num_prompts: 4
Expand Down Expand Up @@ -165,7 +165,7 @@ test_cases:
benchmarks:
perf:
case_type: performance
dataset_path: vllm-ascend/GSM8K-in1007616-bs16-prefix0-qwen
dataset_path: vllm-ascend/GSM8K_prefix90_in131072_bs100_qwen
request_conf: vllm_api_stream_chat
dataset_conf: gsm8k/gsm8k_gen_0_shot_cot_str_perf
num_prompts: 60
Expand Down
Loading