Skip to content

[CI] Move applicable tests to CPU #24080

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 9 commits into from
Sep 30, 2025
Merged

[CI] Move applicable tests to CPU #24080

merged 9 commits into from
Sep 30, 2025

Conversation

@rzabarazesh
Copy link
Collaborator

@rzabarazesh rzabarazesh commented Sep 2, 2025 •
edited by github-actions bot
Loading

Purpose

Moving tests to CPU as part of #23589

Test Plan

CI

Test Result

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.
  • (Optional) Release notes update. If your change is user facing, please update the release notes draft in the Google Doc.

@mergify mergify bot added the ci/build label Sep 2, 2025
@rzabarazesh rzabarazesh added the ready ONLY add when PR is ready to merge/full CI is needed label Sep 2, 2025
@rzabarazesh rzabarazesh marked this pull request as ready for review September 2, 2025 13:24
@mergify mergify bot added documentation Improvements or additions to documentation frontend multi-modality Related to multi-modality (#4194) new-model Requests to new models labels Sep 2, 2025
@rzabarazesh
Move V1 tests to CPU
d6a2456 
Signed-off-by: Reza Barazesh <[email protected]>

Re-arrange tests

Signed-off-by: Reza Barazesh <[email protected]>

Move tokenization tests to cpu

Signed-off-by: Reza Barazesh <[email protected]>

Move Basic model tests to cpu

Signed-off-by: Reza Barazesh <[email protected]>

Move Multimodal tests to CPU

Signed-off-by: Reza Barazesh <[email protected]>

Move tool-use tests to cpu

Signed-off-by: Reza Barazesh <[email protected]>

Fix Basic model tests

Signed-off-by: Reza Barazesh <[email protected]>

Move entrypoint tests to CPU

Signed-off-by: Reza Barazesh <[email protected]>

Move Core tests to cpu

Signed-off-by: Reza Barazesh <[email protected]>

Move v1/engine tests to CPU

Signed-off-by: Reza Barazesh <[email protected]>

Move remaining tests to cpu

Signed-off-by: Reza Barazesh <[email protected]>

Revert failing tests

Signed-off-by: Reza Barazesh <[email protected]>

Update test configurations for CPU pipeline execution
hmellor
hmellor requested changes Sep 30, 2025
View reviewed changes
@mergify
Copy link

mergify bot commented Sep 30, 2025

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

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 Sep 30, 2025
@mergify mergify bot removed the needs-rebase label Sep 30, 2025
@hmellor
Fix V1 test others
588b76f 
Signed-off-by: Harry Mellor <[email protected]>
@hmellor
Fix basic models tests other
bcc13b7 
Signed-off-by: Harry Mellor <[email protected]>
@hmellor
Move transformers_utils to CPU job
5648491 
Signed-off-by: Harry Mellor <[email protected]>
@hmellor
Remove marks from non-test files
1fe5b1c 
Signed-off-by: Harry Mellor <[email protected]>
@hmellor
Use single pytest command
7137c7e 
Signed-off-by: Harry Mellor <[email protected]>
@hmellor hmellor enabled auto-merge (squash) September 30, 2025 09:37
DarkLight1337
DarkLight1337 reviewed Sep 30, 2025
View reviewed changes
Copy link
Member

@DarkLight1337 DarkLight1337 left a comment

Choose a reason for hiding this comment

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

Nice, this should keep our CI cost down

@hmellor hmellor disabled auto-merge September 30, 2025 13:45
@hmellor hmellor merged commit bc546f7 into vllm-project:main Sep 30, 2025
28 checks passed
@rzabarazesh rzabarazesh deleted the cpu-pipeline branch October 1, 2025 05:09
pdasigi pushed a commit to pdasigi/vllm that referenced this pull request Oct 2, 2025
@rzabarazesh @hmellor @pdasigi
[CI] Move applicable tests to CPU (vllm-project#24080)
8291c88 
Signed-off-by: Harry Mellor <[email protected]>
Co-authored-by: Harry Mellor <[email protected]>
yewentao256 pushed a commit that referenced this pull request Oct 3, 2025
@rzabarazesh @hmellor @yewentao256
[CI] Move applicable tests to CPU (#24080)
8914d52 
Signed-off-by: Harry Mellor <[email protected]>
Co-authored-by: Harry Mellor <[email protected]>
Signed-off-by: yewentao256 <[email protected]>
tomeras91 pushed a commit to tomeras91/vllm that referenced this pull request Oct 6, 2025
@rzabarazesh @hmellor @tomeras91
[CI] Move applicable tests to CPU (vllm-project#24080)
f540576 
Signed-off-by: Harry Mellor <[email protected]>
Co-authored-by: Harry Mellor <[email protected]>
Signed-off-by: Tomer Asida <[email protected]>
xuebwang-amd pushed a commit to xuebwang-amd/vllm that referenced this pull request Oct 10, 2025
@rzabarazesh @hmellor @xuebwang-amd
[CI] Move applicable tests to CPU (vllm-project#24080)
5aebe41 
Signed-off-by: Harry Mellor <[email protected]>
Co-authored-by: Harry Mellor <[email protected]>
Signed-off-by: xuebwang-amd <[email protected]>
lywa1998 pushed a commit to lywa1998/vllm that referenced this pull request Oct 20, 2025
@rzabarazesh @hmellor @lywa1998
[CI] Move applicable tests to CPU (vllm-project#24080)
031b8d1 
Signed-off-by: Harry Mellor <[email protected]>
Co-authored-by: Harry Mellor <[email protected]>
alhridoy pushed a commit to alhridoy/vllm that referenced this pull request Oct 24, 2025
@rzabarazesh @hmellor @alhridoy
[CI] Move applicable tests to CPU (vllm-project#24080)
19485dc 
Signed-off-by: Harry Mellor <[email protected]>
Co-authored-by: Harry Mellor <[email protected]>
xuebwang-amd pushed a commit to xuebwang-amd/vllm that referenced this pull request Oct 24, 2025
@rzabarazesh @hmellor @xuebwang-amd
[CI] Move applicable tests to CPU (vllm-project#24080)
250d8ba 
Signed-off-by: Harry Mellor <[email protected]>
Co-authored-by: Harry Mellor <[email protected]>
Signed-off-by: xuebwang-amd <[email protected]>
rtourgeman pushed a commit to rtourgeman/vllm that referenced this pull request Nov 10, 2025
@rzabarazesh @hmellor @rtourgeman
[CI] Move applicable tests to CPU (vllm-project#24080)
2354a3e 
Signed-off-by: Harry Mellor <[email protected]>
Co-authored-by: Harry Mellor <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@houseroad houseroad houseroad left review comments

@yewentao256 yewentao256 yewentao256 left review comments

@DarkLight1337 DarkLight1337 DarkLight1337 left review comments

@hmellor hmellor hmellor approved these changes

@bigPYJ1151 bigPYJ1151 Awaiting requested review from bigPYJ1151 bigPYJ1151 is a code owner

@jikunshang jikunshang Awaiting requested review from jikunshang

@tdoublep tdoublep Awaiting requested review from tdoublep

@jeejeelee jeejeelee Awaiting requested review from jeejeelee

@ywang96 ywang96 Awaiting requested review from ywang96 ywang96 is a code owner

@tlrmchlsmth tlrmchlsmth Awaiting requested review from tlrmchlsmth

@WoosukKwon WoosukKwon Awaiting requested review from WoosukKwon WoosukKwon is a code owner

@robertgshaw2-redhat robertgshaw2-redhat Awaiting requested review from robertgshaw2-redhat robertgshaw2-redhat is a code owner

@simon-mo simon-mo Awaiting requested review from simon-mo

@aarnphm aarnphm Awaiting requested review from aarnphm aarnphm is a code owner

@njhill njhill Awaiting requested review from njhill njhill is a code owner

@comaniac comaniac Awaiting requested review from comaniac

@alexm-redhat alexm-redhat Awaiting requested review from alexm-redhat alexm-redhat is a code owner

@youkaichao youkaichao Awaiting requested review from youkaichao

@mgoin mgoin Awaiting requested review from mgoin mgoin is a code owner

@ProExpertProg ProExpertProg Awaiting requested review from ProExpertProg

@zhuohan123 zhuohan123 Awaiting requested review from zhuohan123

@sighingnow sighingnow Awaiting requested review from sighingnow

@heheda12345 heheda12345 Awaiting requested review from heheda12345 heheda12345 is a code owner

@russellb russellb Awaiting requested review from russellb russellb is a code owner

@NickLucche NickLucche Awaiting requested review from NickLucche NickLucche is a code owner

@ApostaC ApostaC Awaiting requested review from ApostaC ApostaC is a code owner

+1 more reviewer

@afeldman-nm afeldman-nm afeldman-nm approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

ci/build kv-connector multi-modality Related to multi-modality (#4194) qwen Related to Qwen models ready ONLY add when PR is ready to merge/full CI is needed structured-output tool-calling v1

Projects

Structured Output
Status: Done
Tool Calling
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@rzabarazesh @robertgshaw2-redhat @njhill @hmellor @houseroad @yewentao256 @DarkLight1337 @afeldman-nm