Skip to content

Revert "Add nightly b200 test for spec decode eagle correctness (#38577)"#39512

Merged
benchislett merged 1 commit into
vllm-project:mainfrom
CentML:revert-eagle-test
Apr 11, 2026
Merged

Revert "Add nightly b200 test for spec decode eagle correctness (#38577)"#39512
benchislett merged 1 commit into
vllm-project:mainfrom
CentML:revert-eagle-test

Conversation

@benchislett

Copy link
Copy Markdown
Member

This reverts commit adaabb8. Failing in nightly CI.

see: #39441

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

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.

Code Review

This pull request removes several nightly test configurations for speculative decoding on B200 devices from the Buildkite test area specification, including Eagle, Speculators + MTP, and Draft Model tests. I have no feedback to provide.

@mergify mergify Bot added the ci/build label Apr 10, 2026
…-project#38577)"

This reverts commit adaabb8.

Signed-off-by: Benjamin Chislett <bchislett@nvidia.com>
@benchislett benchislett added the ready ONLY add when PR is ready to merge/full CI is needed label Apr 10, 2026
@benchislett benchislett merged commit af661a1 into vllm-project:main Apr 11, 2026
16 checks passed
@benchislett benchislett deleted the revert-eagle-test branch April 11, 2026 00:07
wojciech-wais pushed a commit to wojciech-wais/vllm that referenced this pull request Apr 13, 2026
stecasta added a commit to stecasta/vllm that referenced this pull request Apr 21, 2026
Re-applies the 3 optional nightly B200 buildkite steps originally added
in vllm-project#38577 and reverted in vllm-project#39512. The revert was due
to the Blackwell specdec correctness regression; the preceding commit
in this PR fixes the underlying bug.

Addresses Matthew Bonanni's review ask to re-enable the previously
failing tests and confirm they pass CI.

Co-authored-by: Rishi Puri <riship@nvidia.com>
Signed-off-by: Stefano Castagnetta <scastagnetta@nvidia.com>
stecasta added a commit to stecasta/vllm that referenced this pull request Apr 21, 2026
…m-project#38577)" (vllm-project#39512)

This reverts commit af661a1.

Signed-off-by: Stefano Castagnetta <scastagnetta@nvidia.com>
stecasta added a commit to puririshi98/vllm that referenced this pull request Apr 21, 2026
…m-project#38577)" (vllm-project#39512)

This reverts commit af661a1.

Signed-off-by: Stefano Castagnetta <scastagnetta@nvidia.com>
whk-lab pushed a commit to whk-lab/vllm that referenced this pull request Apr 23, 2026
avinashsingh77 pushed a commit to avinashsingh77/vllm that referenced this pull request Apr 27, 2026
…-project#38577)" (vllm-project#39512)

Signed-off-by: Benjamin Chislett <bchislett@nvidia.com>
Signed-off-by: Avinash Singh <avinashsingh.rcoem@gmail.com>
mystous pushed a commit to mystous/vllm_hybrid that referenced this pull request May 10, 2026
my-other-github-account pushed a commit to my-other-github-account/vllm that referenced this pull request May 15, 2026
my-other-github-account pushed a commit to my-other-github-account/vllm that referenced this pull request May 15, 2026
jhu960213 pushed a commit to jhu960213/vllm that referenced this pull request May 20, 2026
mvanhorn pushed a commit to mvanhorn/vllm that referenced this pull request Jun 4, 2026
…-project#38577)" (vllm-project#39512)

Signed-off-by: Benjamin Chislett <bchislett@nvidia.com>
Signed-off-by: Matt Van Horn <455140+mvanhorn@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/build ready ONLY add when PR is ready to merge/full CI is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants