Skip to content

Conversation

@zhangxinyuehfad
Copy link
Contributor

@zhangxinyuehfad zhangxinyuehfad commented Oct 28, 2025

What this PR does / why we need it?

Fix ci test on A3

  1. delete lables
  2. fix filter yaml file name
  3. refactor dist test to e2e full test
  4. skip test_models_distributed_Qwen3_MOE_TP2_WITH_EP & test_models_distributed_Qwen3_MOE_W8A8_WITH_EP because of [Bug]: Qwen_Qwen3-VL-30B-A3B-Instruct aclgraph piecewise模式部署报错 #3895

Does this PR introduce any user-facing change?

How was this patch tested?

@gemini-code-assist
Copy link
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@github-actions
Copy link

👋 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.

@zhangxinyuehfad zhangxinyuehfad force-pushed the zxy_fix_dist branch 4 times, most recently from 1e3867e to f59a428 Compare October 28, 2025 02:35
@MengqingCao
Copy link
Collaborator

LGTM, plz also take a look @wangxiyuan @Yikun This will trigger test on a3 4 cards per pr

@vllm-ascend-ci vllm-ascend-ci added ready-for-test start test by label for PR ready read for review labels Oct 31, 2025
@zhangxinyuehfad zhangxinyuehfad force-pushed the zxy_fix_dist branch 3 times, most recently from 2b1b9e3 to 2673cff Compare November 3, 2025 05:36
@zhangxinyuehfad zhangxinyuehfad changed the title [Fix]Fix ci test on A3 [Fix] Refactor and fix dist test to e2e full test Nov 3, 2025
@github-actions
Copy link

github-actions bot commented Nov 3, 2025

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

@zhangxinyuehfad zhangxinyuehfad force-pushed the zxy_fix_dist branch 2 times, most recently from dc78848 to 71a2c0e Compare November 4, 2025 01:42
@zhangxinyuehfad zhangxinyuehfad force-pushed the zxy_fix_dist branch 3 times, most recently from 6bf7c69 to 973d82a Compare November 4, 2025 03:13
@zhangxinyuehfad zhangxinyuehfad force-pushed the zxy_fix_dist branch 4 times, most recently from 3fa2a05 to 919a7a7 Compare November 7, 2025 01:25
Add accuracy test for qwen3-8b-w8a8

- vLLM version: v0.11.0rc3
- vLLM main:
vllm-project/vllm@c9461e0

Signed-off-by: hfadzxy <starmoon_zhang@163.com>
@wangxiyuan wangxiyuan merged commit fae1c59 into vllm-project:main Nov 11, 2025
22 checks passed
fluctlux pushed a commit to fluctlux/vllm-ascend that referenced this pull request Nov 11, 2025
### What this PR does / why we need it?
Fix ci test on A3
1. delete lables
2. fix filter yaml file name
3. refactor dist test to e2e full test 
4. skip test_models_distributed_Qwen3_MOE_TP2_WITH_EP &
test_models_distributed_Qwen3_MOE_W8A8_WITH_EP because of
vllm-project#3895

- vLLM version: v0.11.0
- vLLM main:
vllm-project/vllm@83f478b

Signed-off-by: hfadzxy <starmoon_zhang@163.com>
luolun pushed a commit to luolun/vllm-ascend that referenced this pull request Nov 19, 2025
### What this PR does / why we need it?
Fix ci test on A3
1. delete lables
2. fix filter yaml file name
3. refactor dist test to e2e full test 
4. skip test_models_distributed_Qwen3_MOE_TP2_WITH_EP &
test_models_distributed_Qwen3_MOE_W8A8_WITH_EP because of
vllm-project#3895

- vLLM version: v0.11.0
- vLLM main:
vllm-project/vllm@83f478b

Signed-off-by: hfadzxy <starmoon_zhang@163.com>
Signed-off-by: luolun <luolun1995@cmbchina.com>
hwhaokun pushed a commit to hwhaokun/vllm-ascend that referenced this pull request Nov 19, 2025
### What this PR does / why we need it?
Fix ci test on A3
1. delete lables
2. fix filter yaml file name
3. refactor dist test to e2e full test
4. skip test_models_distributed_Qwen3_MOE_TP2_WITH_EP &
test_models_distributed_Qwen3_MOE_W8A8_WITH_EP because of
vllm-project#3895

- vLLM version: v0.11.0
- vLLM main:
vllm-project/vllm@83f478b

Signed-off-by: hfadzxy <starmoon_zhang@163.com>
Signed-off-by: hwhaokun <haokun0405@163.com>
NSDie pushed a commit to NSDie/vllm-ascend that referenced this pull request Nov 24, 2025
### What this PR does / why we need it?
Fix ci test on A3
1. delete lables
2. fix filter yaml file name
3. refactor dist test to e2e full test
4. skip test_models_distributed_Qwen3_MOE_TP2_WITH_EP &
test_models_distributed_Qwen3_MOE_W8A8_WITH_EP because of
vllm-project#3895

- vLLM version: v0.11.0
- vLLM main:
vllm-project/vllm@83f478b

Signed-off-by: hfadzxy <starmoon_zhang@163.com>
Signed-off-by: nsdie <yeyifan@huawei.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready read for review ready-for-test start test by label for PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants