Another step in the Ext proc refactoring#2560
Another step in the Ext proc refactoring#2560shmuelk wants to merge 4 commits intokubernetes-sigs:mainfrom
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: shmuelk The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
✅ Deploy Preview for gateway-api-inference-extension ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
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>
2ef0c71 to
65a64fb
Compare
|
PR needs rebase. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
Are you planning to rebase and merge the PR in this repo? or close and work on this after the migration? |
|
I'm not sure this PR is relevent, as the EPP is moving to the llm-d-inference-scheduler and the bbr is moving to a different repo. Opinions? |
+1 on closing |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Both the Endpoint Picker (EPP) and the Body Based Router (BBR) are Envoy ExtProc filters. There is much code that has been copy and pasted from one to the other.
This PR is in a series of PRs that will refactor out the common Ext Proc processing code into a set of reusable packages. This is being done to try and keep the PRs somewhat smaller.
In particular this PR refactors packages, some code is moved to the common package, and the StreamingServer is renamed.
Which issue(s) this PR fixes:
Ref: #2427
Does this PR introduce a user-facing change?: