Skip to content

[Model Runner V2] Add KV Connector support#32742

Merged
WoosukKwon merged 7 commits intovllm-project:mainfrom
njhill:mrv2-kvconnectors
Jan 23, 2026
Merged

[Model Runner V2] Add KV Connector support#32742
WoosukKwon merged 7 commits intovllm-project:mainfrom
njhill:mrv2-kvconnectors

Conversation

@njhill
Copy link
Copy Markdown
Member

@njhill njhill commented Jan 21, 2026

Tested with cpu offloading and NIXL P/D.

@mergify mergify bot added the v1 label Jan 21, 2026
@njhill njhill force-pushed the mrv2-kvconnectors branch from 64bce72 to f687265 Compare January 21, 2026 00:52
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 introduces support for KV Connector, which facilitates Key-Value cache transfers. The changes involve modifying the init_kv_cache function to return the initialized KV caches, updating the ModelRunner to register these caches with the KV Connector, and refactoring preemption handling into a dedicated method within a new mixin. The execute_model_state and ModelRunnerOutput are also updated to propagate KV Connector specific outputs. Overall, the changes are well-structured and integrate the new functionality effectively.

@mergify
Copy link
Copy Markdown

mergify bot commented Jan 21, 2026

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

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

Signed-off-by: Nick Hill <nickhill123@gmail.com>
Signed-off-by: Nick Hill <nickhill123@gmail.com>
@njhill njhill force-pushed the mrv2-kvconnectors branch 2 times, most recently from 3f7c1cb to d398473 Compare January 21, 2026 23:45
@mergify mergify bot removed the needs-rebase label Jan 21, 2026
Signed-off-by: Nick Hill <nickhill123@gmail.com>
@njhill njhill force-pushed the mrv2-kvconnectors branch from d398473 to 9196aaf Compare January 21, 2026 23:53
Signed-off-by: Nick Hill <nickhill123@gmail.com>
Signed-off-by: Nick Hill <nickhill123@gmail.com>
Signed-off-by: Nick Hill <nickhill123@gmail.com>
@njhill njhill marked this pull request as ready for review January 22, 2026 06:19
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

Comment @cursor review or bugbot run to trigger another review on this PR

@njhill njhill added the ready ONLY add when PR is ready to merge/full CI is needed label Jan 22, 2026
@WoosukKwon WoosukKwon merged commit 8518b30 into vllm-project:main Jan 23, 2026
51 of 52 checks passed
@njhill njhill deleted the mrv2-kvconnectors branch January 23, 2026 23:44
cwazai pushed a commit to cwazai/vllm that referenced this pull request Jan 25, 2026
Signed-off-by: Nick Hill <nickhill123@gmail.com>
Signed-off-by: 陈建华 <1647430658@qq.com>
lapy pushed a commit to lapy/vllm that referenced this pull request Jan 27, 2026
Signed-off-by: Nick Hill <nickhill123@gmail.com>
ItzDEXX pushed a commit to ItzDEXX/vllm that referenced this pull request Feb 19, 2026
Signed-off-by: Nick Hill <nickhill123@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kv-connector ready ONLY add when PR is ready to merge/full CI is needed v1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants