Skip to content

sync gic to latest v1.0.0 release#353

Merged
github-actions[bot] merged 2 commits intollm-d:mainfrom
nirrozenbaum:sync-gie
Sep 16, 2025
Merged

sync gic to latest v1.0.0 release#353
github-actions[bot] merged 2 commits intollm-d:mainfrom
nirrozenbaum:sync-gie

Conversation

@nirrozenbaum
Copy link
Collaborator

No description provided.

Signed-off-by: Nir Rozenbaum <nirro@il.ibm.com>
if test.wantRes2 != nil { // Checking the prefix match in the decode pod.
// make sure prefix plugin stores the prefix hit in cache, so we can test it in the following schedule call
prefixScorer.PreRequest(ctx, test.req, got, 0)
time.Sleep(time.Second)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

this is a temp solution until it's solved appropriately in IGW.
PreRequest is adding the information to the cache asynchronously, so we need to wait.
this should be solved in next version and then we should remove this line from our tests.

Signed-off-by: Nir Rozenbaum <nirro@il.ibm.com>
@shmuelk
Copy link
Collaborator

shmuelk commented Sep 16, 2025

/lgtm

/approve

@github-actions github-actions bot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Sep 16, 2025
@github-actions github-actions bot merged commit 451dd1c into llm-d:main Sep 16, 2025
6 checks passed
@nirrozenbaum nirrozenbaum deleted the sync-gie branch September 16, 2025 14:33
nirrozenbaum added a commit that referenced this pull request Sep 17, 2025
* sync gic to latest v1.0.0 release

Signed-off-by: Nir Rozenbaum <nirro@il.ibm.com>

* missed one more update

Signed-off-by: Nir Rozenbaum <nirro@il.ibm.com>

---------

Signed-off-by: Nir Rozenbaum <nirro@il.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm "Looks good to me", indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants