Skip to content

[XPU] fix_test_worker_memory_snapshot_for_xpu - #50097

Merged
jikunshang merged 4 commits into
vllm-project:mainfrom
mayuyuace:qiming/fix_test_worker_memory_snapshot_for_xpu
Sep 16, 2026
Merged

jikunshang merged 4 commits into
vllm-project:mainfrom
mayuyuace:qiming/fix_test_worker_memory_snapshot_for_xpu

Conversation

@mayuyuace

Copy link
Copy Markdown
Contributor

Dynamically select worker class ([XPUWorker] vs [Worker]) and patch target based on [current_platform.is_xpu()], so the test works on both CUDA and XPU.
Rename label "nccl_all_reduce" to "dist_all_reduce", since XPU uses XCCL, not NCCL; the label should be backend-agnostic.

Signed-off-by: mayuyuace <qiming1.zhang@intel.com>

@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 intel-gpu Related to Intel GPU v1 labels Jul 28, 2026
@mayuyuace

Copy link
Copy Markdown
Contributor Author

@jikunshang

@jikunshang

Copy link
Copy Markdown
Member

/ci run

@github-actions

Copy link
Copy Markdown

✅ Triggered Buildkite CI #89220 for commit 1468005cd1be.

@jikunshang
jikunshang merged commit 4ace5cd into vllm-project:main Sep 16, 2026
39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

intel-gpu Related to Intel GPU v1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants