Skip to content

[ROCm][DI][CI] Enable WideEP EP16/DP16 tests - #59266

Merged
shen-shanshan merged 20 commits into
vllm-project:mainfrom
lcskrishna:csrikris-internode-di-ci
Oct 21, 2026
Merged

shen-shanshan merged 20 commits into
vllm-project:mainfrom
lcskrishna:csrikris-internode-di-ci

Conversation

@lcskrishna

@lcskrishna lcskrishna commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Purpose

This PR enables ROCm DI WideEP EP16 tests.
Currently the following models are added :

  • DeepSeek-V3
  • DeepSeek-R1-Fp4
  • Kimi K2.6 & 2.5
  • GLM5.2
  • MiniMax M3

Test Plan

  • Run all these models on AMD DI CI.

Test Result

https://buildkite.com/vllm/amd-distributed-inference-ci/builds/118/list


Essential Elements of an Effective PR Description Checklist
  • The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
  • The test plan, such as providing test command.
  • The test results, such as pasting the results comparison before and after, or e2e results
  • (Optional) The necessary documentation update, such as updating supported_models.md and examples for a new model.

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@mergify mergify Bot added ci/build rocm Related to AMD ROCm labels Sep 29, 2026
@lcskrishna
lcskrishna marked this pull request as draft September 29, 2026 18:28
@mergify mergify Bot added deepseek Related to DeepSeek models kv-connector labels Oct 13, 2026
@mergify

mergify Bot commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

This pull request has merge conflicts that must be resolved before it can be
merged. Please rebase the PR, @lcskrishna.

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

@mergify mergify Bot added the needs-rebase label Oct 13, 2026
@lcskrishna
lcskrishna force-pushed the csrikris-internode-di-ci branch from 4d6e2e5 to 3077af5 Compare October 15, 2026 11:58
@mergify mergify Bot removed the needs-rebase label Oct 15, 2026
@mergify

mergify Bot commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

This pull request has merge conflicts that must be resolved before it can be
merged. Please rebase the PR, @lcskrishna.

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

@mergify mergify Bot added the needs-rebase label Oct 16, 2026
@lcskrishna

Copy link
Copy Markdown
Contributor Author

Dependent PRs are merged; Reopening it and is ready for review.

#59596 (FP4 MoE MoRI)
#58585 (GLM HMA changes)
#58989 (Skip all reduce changes)

@lcskrishna
lcskrishna force-pushed the csrikris-internode-di-ci branch from 4890ed0 to 5d30756 Compare October 16, 2026 16:29
@lcskrishna
lcskrishna marked this pull request as ready for review October 16, 2026 16:43

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@mergify mergify Bot removed the needs-rebase label Oct 16, 2026
lcskrishna and others added 3 commits October 21, 2026 05:04
Folds b90eee6, 190c7ac, 74a998d, 9327e85, d1e2300 and 57f33bb. Their conflicts with upstream were resolved in merge 212dddc; this is that merge's diff against df4dbe4.

Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
@lcskrishna
lcskrishna force-pushed the csrikris-internode-di-ci branch from 85c953c to 1504e42 Compare October 21, 2026 05:11
lcskrishna and others added 3 commits October 21, 2026 05:36
Verified on Spur MI350X cluster (4 nodes, nightly + MORI 5c88676c):
- GSM8K flexible: 93.25%
- GSM8K strict: 93.25%

Signed-off-by: Tej Kiran <kiran.tej@amd.com>
Co-authored-by: Tej Kiran <kiran.tej@amd.com>
Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
lcskrishna and others added 12 commits October 21, 2026 05:36
…sts (#27)

* Add MiniMax-M3-MXFP8 and GLM-5.2-FP8 2P2D EP16 WideEP MoRIIO vllm-router CI

Mirror Kimi-K2.5 2P2D EP16 entry (#24). Verified on Spur MI350X
this week: both models PASS on toy proxy and vllm-router (2P2D EP16/DP16).

Co-authored-by: Cursor <cursoragent@cursor.com>

* Add proxy companions for MiniMax and GLM 2P2D EP16 CI steps

Match the proxy + vllm-router pairing used for lower topologies
(e.g. GLM catalog PR pattern).

Co-authored-by: Cursor <cursoragent@cursor.com>

---------

Co-authored-by: avininjamay8 <avpaul@amd.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Avinash Paul <Avinash.Paul@amd.com>
Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
@lcskrishna
lcskrishna force-pushed the csrikris-internode-di-ci branch from 19bdd54 to 4550e5b Compare October 21, 2026 05:37
@shen-shanshan

Copy link
Copy Markdown
Contributor

/ci run

@github-actions

Copy link
Copy Markdown

✅ Triggered Buildkite CI #96360 for commit 01d98cac543a.

@shen-shanshan
shen-shanshan merged commit 64c5f14 into vllm-project:main Oct 21, 2026
20 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in AMD Oct 21, 2026
bantianyue pushed a commit to bantianyue/vllm that referenced this pull request Oct 28, 2026
Signed-off-by: lcskrishna <lollachaitanya@gmail.com>
Signed-off-by: Tej Kiran <kiran.tej@amd.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Tej Kiran <kiran.tej@amd.com>
Co-authored-by: avininjamay8 <avpaul@amd.com>
Co-authored-by: Avinash Paul <Avinash.Paul@amd.com>
Co-authored-by: Shanshan Shen <467638484@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/build deepseek Related to DeepSeek models kv-connector rocm Related to AMD ROCm

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants