test: Scale up and down the model server during an end to end test#354
Merged
shmuelk merged 8 commits intollm-d:mainfrom Nov 10, 2025
Merged
test: Scale up and down the model server during an end to end test#354shmuelk merged 8 commits intollm-d:mainfrom
shmuelk merged 8 commits intollm-d:mainfrom
Conversation
shmuelk
force-pushed
the
scale-up-down-test
branch
from
8f159b3 to
22c5fa8
Compare
shmuelk
force-pushed
the
scale-up-down-test
branch
from
cd7e869 to
793c1de
Compare
elevran
requested changes
Oct 20, 2025
Collaborator
|
@shmuelk - please resolve conflicts |
Signed-off-by: Shmuel Kallner <kallner@il.ibm.com>
Signed-off-by: Shmuel Kallner <kallner@il.ibm.com>
Signed-off-by: Shmuel Kallner <kallner@il.ibm.com>
Signed-off-by: Shmuel Kallner <kallner@il.ibm.com>
Signed-off-by: Shmuel Kallner <kallner@il.ibm.com>
Signed-off-by: Shmuel Kallner <kallner@il.ibm.com>
Signed-off-by: Shmuel Kallner <kallner@il.ibm.com>
Signed-off-by: Shmuel Kallner <kallner@il.ibm.com>
shmuelk
force-pushed
the
scale-up-down-test
branch
from
793c1de to
22efbb6
Compare
Collaborator
Author
|
/unhold |
Collaborator
|
/lgtm |
github-actions
bot
added
the
lgtm
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
elevran
approved these changes
Nov 10, 2025
github-project-automation
bot
moved this from In review
to Done
in llm-d-inference-scheduler
lionelvillard
pushed a commit
to lionelvillard/llm-d-inference-scheduler
that referenced
this pull request
Nov 10, 2025
…lm-d#354) * Added a helper to scale up/down deployments Signed-off-by: Shmuel Kallner <kallner@il.ibm.com> * Added a test in which the model server is scaled up and down Signed-off-by: Shmuel Kallner <kallner@il.ibm.com> * Fixed typo Signed-off-by: Shmuel Kallner <kallner@il.ibm.com> * Fixed lint issue Signed-off-by: Shmuel Kallner <kallner@il.ibm.com> * Restored code commented out for debugging Signed-off-by: Shmuel Kallner <kallner@il.ibm.com> * Updates due to review comments Signed-off-by: Shmuel Kallner <kallner@il.ibm.com> * Added comments for clarification Signed-off-by: Shmuel Kallner <kallner@il.ibm.com> * Changes due to refactoring done in prior PRs Signed-off-by: Shmuel Kallner <kallner@il.ibm.com> --------- Signed-off-by: Shmuel Kallner <kallner@il.ibm.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds a test to the end to tests.
In particular it adds a test in which:
Fixes: #347