Implement model name virtualization feature#685
Conversation
Add virtual name support to ExternalModel and MaaSModelRef CRDs: - Add backendModelName and modelAliases fields to specs - Add virtualNames and resolvedBackendModelName to status - Implement controller logic for virtual name status updates - Add validation functions for virtual names and backend models - Include comprehensive test coverage - Fix duplicate function declaration by moving equalStringSlices to annotations.go Work in progress - also adding a second PR in the ai-gateway-payload-processing repo to match with this PR in this repo. This PR will be one of two PRs to be merged for the issue opendatahub-io#684 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
|
Skipping CI for Draft Pull Request. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: srampal 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 |
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Central YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
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. |
|
This is a great idea! But we have developed plans to create a plugin for Model Name virtualization external to MaaS |
Work in progress: Not yet ready for review and merge!
Add virtual name support to ExternalModel and MaaSModelRef CRDs:
Work in progress - also adding a second PR in the ai-gateway-payload-processing repo to match with this PR in this repo.
This PR will be one of two PRs to be merged for the issue #684
🤖 Generated with Claude Code
Description
How Has This Been Tested?
Merge criteria: