Skip to content

[CI]test qwen3-235B and qwen3-vl-235b nightly#5442

Closed
MrZ20 wants to merge 5 commits intovllm-project:mainfrom
MrZ20:qwen3-vl-235b-nightly
Closed

[CI]test qwen3-235B and qwen3-vl-235b nightly#5442
MrZ20 wants to merge 5 commits intovllm-project:mainfrom
MrZ20:qwen3-vl-235b-nightly

Conversation

@MrZ20
Copy link
Copy Markdown
Contributor

@MrZ20 MrZ20 commented Dec 27, 2025 •
edited by github-actions bot
Loading

What this PR does / why we need it?

[CI]Add nightly ci test for qwen3-vl-235b-a22b

Does this PR introduce any user-facing change?

How was this patch tested?

gemini-code-assist[bot]
gemini-code-assist bot reviewed Dec 27, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds a new nightly CI test for the Qwen3-VL-235B-A22B model. While adding more test coverage is valuable, I've found several issues that compromise the stability and maintainability of this new test. There are critical errors in the test configuration, such as an incorrect model name that will prevent the test from running correctly. Additionally, the Kubernetes configuration includes hardcoded IP addresses, and the execution script relies on a temporary hack to check out a specific pull request, which will inevitably break the CI pipeline in the near future. These issues need to be addressed before this PR can be merged.

Comment thread tests/e2e/nightly/multi_node/config/models/Qwen3-VL-235B-A22B.yaml Outdated
Comment thread tests/e2e/nightly/multi_node/config/models/Qwen3-VL-235B-A22B.yaml Outdated
Comment thread tests/e2e/nightly/multi_node/scripts/run.sh
Comment thread tests/e2e/nightly/multi_node/scripts/lws.yaml.jinja2 Outdated
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 27, 2025

👋 Hi! Thank you for contributing to the vLLM Ascend project. The following points will speed up your PR merge:‌‌

  • A PR should do only one thing, smaller PRs enable faster reviews.
  • Every PR should include unit tests and end-to-end tests ‌to ensure it works and is not broken by other future PRs.
  • Write the commit message by fulfilling the PR description to help reviewer and future developers understand.

If CI fails, you can run linting and testing checks locally according Contributing and Testing.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 27, 2025

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@MrZ20 MrZ20 force-pushed the qwen3-vl-235b-nightly branch from 1f057a3 to bf7bfb6 Compare December 29, 2025 03:14
@MrZ20 MrZ20 force-pushed the qwen3-vl-235b-nightly branch 6 times, most recently from 655de33 to 86287e4 Compare December 29, 2025 07:03
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 30, 2025

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@MrZ20 MrZ20 force-pushed the qwen3-vl-235b-nightly branch 2 times, most recently from c3b2d51 to 393c45f Compare January 5, 2026 06:29
@MrZ20 MrZ20 force-pushed the qwen3-vl-235b-nightly branch 4 times, most recently from d1dac5e to 0279639 Compare January 6, 2026 01:29
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 6, 2026

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@MrZ20 MrZ20 closed this Jan 7, 2026
@MrZ20 MrZ20 reopened this Jan 9, 2026
@MrZ20 MrZ20 changed the title [CI]Add nightly ci test for qwen3-vl-235b-a22b [CI]test qwen3-235B and qwen3-vl-235b nightly Jan 9, 2026
MrZ20 added 2 commits January 9, 2026 10:58
@MrZ20
add qwen3-vl-235b-a22b multi_node test
0d041b4 
Signed-off-by: MrZ20 <2609716663@qq.com>
@MrZ20
add pr messege
5697327 
Signed-off-by: MrZ20 <2609716663@qq.com>
MrZ20 added 2 commits January 9, 2026 10:59
@MrZ20
modify
4921c3d 
Signed-off-by: MrZ20 <2609716663@qq.com>

modify

Signed-off-by: MrZ20 <2609716663@qq.com>

modify config para

Signed-off-by: MrZ20 <2609716663@qq.com>

modify

Signed-off-by: MrZ20 <2609716663@qq.com>
@MrZ20
end test
30eb7b1 
Signed-off-by: MrZ20 <2609716663@qq.com>
@MrZ20 MrZ20 force-pushed the qwen3-vl-235b-nightly branch from 0279639 to 5ce18cd Compare January 9, 2026 03:00
@MrZ20 MrZ20 force-pushed the qwen3-vl-235b-nightly branch from 5ce18cd to a3aba3c Compare January 9, 2026 03:15
@MrZ20
test
2c0ffa0 
Signed-off-by: MrZ20 <2609716663@qq.com>
@MrZ20 MrZ20 force-pushed the qwen3-vl-235b-nightly branch from a3aba3c to 2c0ffa0 Compare January 9, 2026 03:29
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 9, 2026

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@MrZ20 MrZ20 closed this Jan 9, 2026
@MrZ20 MrZ20 deleted the qwen3-vl-235b-nightly branch January 13, 2026 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@gemini-code-assist gemini-code-assist[bot] gemini-code-assist[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

ci/build merge-conflicts module:tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@MrZ20