Skip to content

P/D Launch Branch #84

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
Closed
wants to merge 27 commits into from
Closed

P/D Launch Branch #84

wants to merge 27 commits into from

Conversation

robertgshaw2-redhat
Copy link
Collaborator

@robertgshaw2-redhat robertgshaw2-redhat commented May 14, 2025 •
edited by github-actions bot
Loading

SUMMARY:

  • pull down latest main
  • cherry-pick missing upstream commits

mgoin and others added 25 commits May 3, 2025 08:42
@mgoin @njhill
[V1] Support multiple kv connectors
3a70eda 
Signed-off-by: mgoin <mgoin64@gmail.com>
Signed-off-by: Nick Hill <nhill@redhat.com>
@mgoin @njhill
Example script
89a5788 
Signed-off-by: mgoin <mgoin64@gmail.com>
@mgoin @njhill
.
4fa62d4 
Signed-off-by: mgoin <mgoin64@gmail.com>
@mgoin @njhill
Add test
15ca542 
Signed-off-by: mgoin <mgoin64@gmail.com>
@mgoin @njhill
make mypy happy
cd5af12 
Signed-off-by: mgoin <mgoin64@gmail.com>
@njhill
move MultiKVConnectorMetadata to multi_connector.py
5a9a314 
Signed-off-by: Nick Hill <nhill@redhat.com>
@njhill
minor simplifications
2f5e537 
Signed-off-by: Nick Hill <nhill@redhat.com>
@mgoin @njhill
Remove script
014cb2c 
Signed-off-by: mgoin <mgoin64@gmail.com>
@njhill
michael inprogress
7370d83 
Signed-off-by: Nick Hill <nhill@redhat.com>
@mgoin @njhill
Make sure we pop requests from connector dict
df0a82b 
Signed-off-by: mgoin <mgoin64@gmail.com>
@mgoin @njhill
req_id -> request_id
8423135 
Signed-off-by: mgoin <mgoin64@gmail.com>
@mgoin
Update with better test
551fff1 
Signed-off-by: mgoin <mgoin64@gmail.com>
@njhill
add comment to test
10a26b6 
Signed-off-by: Nick Hill <nhill@redhat.com>
@njhill
Handle other new methods and latest API changes
35f3748 
Signed-off-by: Nick Hill <nhill@redhat.com>
@njhill
update get_finished() method with new arg
ff4e083 
Signed-off-by: Nick Hill <nhill@redhat.com>
@mgoin
Merge branch 'main' into multi-kv-connectors
a0f7136 
Signed-off-by: mgoin <mgoin64@gmail.com>
@mgoin
Move test to v1/kv_connector/unit
fc65a18 
Signed-off-by: mgoin <mgoin64@gmail.com>
@njhill
update KVTransferParams typing
e5e1191 
Signed-off-by: Nick Hill <nhill@redhat.com>
@mgoin
Fix typing issue for get_finished
8537d93 
Signed-off-by: mgoin <mgoin64@gmail.com>
@njhill
remove @DataClass from KVConnectorMetadata
9a767d1 
This was there originally but inadvertently dropped

Signed-off-by: Nick Hill <nhill@redhat.com>
@mgoin
Merge branch 'main' into multi-kv-connectors
59ba66f
@tlrmchlsmth @robertgshaw2-redhat
[P/D Disagg][Benchmarking] One request at a time benchmarking for P/D (…
9459f0a 
…vllm-project#79)

* Benchmark one concurrent req

Signed-off-by: Tyler Michael Smith <tyler@neuralmagic.com>

* Updates

Signed-off-by: Tyler Michael Smith <tyler@neuralmagic.com>

* restore

Signed-off-by: Tyler Michael Smith <tyler@neuralmagic.com>

* Improve random requests, switch up initial test

Signed-off-by: Tyler Michael Smith <tyler@neuralmagic.com>

---------

Signed-off-by: Tyler Michael Smith <tyler@neuralmagic.com>
Signed-off-by: Nick Hill <nhill@redhat.com>
@njhill @robertgshaw2-redhat
add LMCache async save support
3a63220 
Requires version of LMCache with the corresponding changes

Signed-off-by: Nick Hill <nhill@redhat.com>
@njhill @robertgshaw2-redhat
handle disabled lmcache async save
6b8a8f8 
Signed-off-by: Nick Hill <nhill@redhat.com>
@njhill @robertgshaw2-redhat
fix
f5bc748 
Signed-off-by: Nick Hill <nhill@redhat.com>
@github-actions GitHub Actions
Copy link

👋 Hi! Thank you for contributing to the vLLM project.

💬 Join our developer Slack at https://slack.vllm.ai to discuss your PR in #pr-reviews, coordinate on features in #feat- channels, or join special interest groups in #sig- channels.

Just a reminder: PRs would not trigger full CI run by default. Instead, it would only run fastcheck CI which starts running only a small and essential subset of CI tests to quickly catch errors. You can run other CI tests on top of those by going to your fastcheck build on Buildkite UI (linked in the PR checks section) and unblock them. If you do not have permission to unblock, ping simon-mo or khluu to add you in our Buildkite org.

Once the PR is approved and ready to go, your PR reviewer(s) can run CI to test the changes comprehensively before merging.

To run CI, PR reviewers can either: Add ready label to the PR or enable auto-merge.

🚀

@robertgshaw2-redhat robertgshaw2-redhat changed the title Llm d launch branch [llm-d] llm-d launch branch May 14, 2025
@robertgshaw2-redhat robertgshaw2-redhat marked this pull request as ready for review May 14, 2025 17:30
@robertgshaw2-redhat robertgshaw2-redhat changed the title [llm-d] llm-d launch branch P/D Disagg Launch branch May 14, 2025
@robertgshaw2-redhat
ifxed
ed1af74 
Signed-off-by: rshaw@neuralmagic.com <robertgshaw2@gmail.com>
@robertgshaw2-redhat robertgshaw2-redhat changed the title P/D Disagg Launch branch P/D Launch Branch May 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@robertgshaw2-redhat @mgoin @njhill @tlrmchlsmth