-
Notifications
You must be signed in to change notification settings - Fork 89
.*: change llm-instance-gateway -> gateway-api-inference-extension #161
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.
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
.*: change llm-instance-gateway -> gateway-api-inference-extension #161
Conversation
/assign @danehans |
There's also https://github.com/kubernetes-sigs/gateway-api-inference-extension/blob/main/PROJECT Considering this was generated at bootstrap time and edited once after the API was renamed, is it okay we edit this manually? |
We can change it manually |
Signed-off-by: Madhav Jivrajani <[email protected]>
3fcd478
to
3e9edf8
Compare
Done @ahg-g, thanks! |
/lgtm Thanks! |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ahg-g, MadhavJivrajani The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…ubernetes-sigs#161) Signed-off-by: Madhav Jivrajani <[email protected]>
…ubernetes-sigs#161) Signed-off-by: Madhav Jivrajani <[email protected]>
This PR renames the main module from
inference.networking.x-k8s.io/llm-instance-gateway
toinference.networking.x-k8s.io/gateway-api-inference-extension
and also changes the name in module imports and some scripts.Note: I've also changed the name in comments that include the old repo name. While this isn't strictly needed since GitHub has redirects in place, I'm not sure if those redirects will always remain.
Fixes #159