Skip to content

[CI/Build][CPU] Update CPU CI test cases#31466

Merged
DarkLight1337 merged 1 commit intovllm-project:mainfrom
bigPYJ1151:fix_lora
Dec 29, 2025
Merged

[CI/Build][CPU] Update CPU CI test cases#31466
DarkLight1337 merged 1 commit intovllm-project:mainfrom
bigPYJ1151:fix_lora

Conversation

@bigPYJ1151
Copy link
Copy Markdown
Member

@bigPYJ1151 bigPYJ1151 commented Dec 29, 2025

Purpose

Update changed test cases in CPU CI

Test Plan

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.

Signed-off-by: jiang1.li <jiang1.li@intel.com>
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 updates the path to a test file within a CPU CI script. The change from tests/lora/test_qwen2vl.py to tests/lora/test_qwenvl.py aligns with the renaming of the test file, which now contains tests for multiple Qwen-VL models. The update is correct and necessary to ensure the CI pipeline executes the appropriate tests. The change is straightforward and I have no issues to report.

@mergify mergify bot added ci/build cpu Related to CPU backends labels Dec 29, 2025
@bigPYJ1151 bigPYJ1151 added ready ONLY add when PR is ready to merge/full CI is needed and removed ci/build cpu Related to CPU backends labels Dec 29, 2025
@mergify mergify bot added ci/build cpu Related to CPU backends labels Dec 29, 2025
Copy link
Copy Markdown
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.

Actually wait, where is this file?

@bigPYJ1151
Copy link
Copy Markdown
Member Author

@DarkLight1337
Copy link
Copy Markdown
Member

Ok found it, I forgot to pull from main 😅

@DarkLight1337 DarkLight1337 merged commit 17347da into vllm-project:main Dec 29, 2025
21 of 24 checks passed
yiliu30 pushed a commit to yiliu30/vllm-fork that referenced this pull request Dec 30, 2025
Signed-off-by: jiang1.li <jiang1.li@intel.com>
akh64bit pushed a commit to akh64bit/vllm that referenced this pull request Jan 16, 2026
Signed-off-by: jiang1.li <jiang1.li@intel.com>
dsuhinin pushed a commit to dsuhinin/vllm that referenced this pull request Jan 21, 2026
Signed-off-by: jiang1.li <jiang1.li@intel.com>
Signed-off-by: dsuhinin <suhinin.dmitriy@gmail.com>
ItzDEXX pushed a commit to ItzDEXX/vllm that referenced this pull request Feb 19, 2026
Signed-off-by: jiang1.li <jiang1.li@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/build cpu Related to CPU backends 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.

2 participants