RHDHBUGS-2501: openshift-ai connector allow for 'rhdh.modelcatalog.io/model-name' override #1683
+12
−9
Conversation
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
…/model-name' override
Contributor
Author
|
to any reviewers, I don't seem to have permission to add labels per the PR instructions |
Collaborator
|
Updated preview: https://redhat-developer.github.io/red-hat-developers-documentation-rhdh/pr-1683/ @ 01/19/26 19:19:41 |
Contributor
Author
|
From the perview: |
Contributor
Author
|
also |
...nshift-ai-connector-for-rhdh/proc-setting-up-openshift-ai-connector-for-rhdh-with-rhoai.adoc
Outdated
Show resolved
Hide resolved
...nshift-ai-connector-for-rhdh/proc-setting-up-openshift-ai-connector-for-rhdh-with-rhoai.adoc
Outdated
Show resolved
Hide resolved
modules/openshift-ai-connector-for-rhdh/ref-enrich-ai-model-metadata.adoc
Outdated
Show resolved
Hide resolved
Member
|
@deerskindoll, thank you for reviewing this! |
Member
you're welcome. feel free to add your suggestions since you were in touch with Gabe and know more about this than I do. |
…ift-ai-connector-for-rhdh-with-rhoai.adoc Co-authored-by: Jana Vrbkova <[email protected]>
…ift-ai-connector-for-rhdh-with-rhoai.adoc Co-authored-by: Jana Vrbkova <[email protected]>
…tadata.adoc Co-authored-by: Jana Vrbkova <[email protected]>
|
Contributor
Author
|
thanks @deerskindoll @pabel-rh - I've pulled in all your suggestions PTAL |
pabel-rh
reviewed
Jan 20, 2026
| |`rhdh.modelcatalog.io/model-name` | ||
| |Annotations | ||
| |A key piece of metadata is the name of the model used when communicating with the model server's REST API. {openshift-ai-connector-name} stores this name in the `rhdh.modelcatalog.io/model-name` annotation on the `Resource` entity, and will default this annotation's value to the combined names of the `ResourceModel` and `ModelVersion` in the model registry, or the `KServe InferenceService` name if the model registry is not used, as those often line up with the model name used when communicating with the model server's REST API. But the names are not guaranteed to match. When the names do not match, provide the correct model name for model REST API invocations. | ||
| |A name of the model used when communicating with the model server's REST API. {openshift-ai-connector-name} stores this name in the `rhdh.modelcatalog.io/model-name` annotation on the `Resource` entity, and defaults this annotation's value to the combined names of the `ResourceModel` and `ModelVersion` in the model registry. |
Member
There was a problem hiding this comment.
Suggested change
| |A name of the model used when communicating with the model server's REST API. {openshift-ai-connector-name} stores this name in the `rhdh.modelcatalog.io/model-name` annotation on the `Resource` entity, and defaults this annotation's value to the combined names of the `ResourceModel` and `ModelVersion` in the model registry. | |
| |A name of the model used when communicating with the model server's REST API. {openshift-ai-connector-name} stores this name in the `rhdh.modelcatalog.io/model-name` annotation on the `Resource` entity, and defaults this annotation's value to the combined names of the `ResourceModel` and `ModelVersion` in the model registry. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 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.
Update list of property overrides to describe how to change the
rhdh.modelcatalog.io/model-nameannotation on anyResourceentities imported by the openshift ai connectorthis doc update includes both
@pabel-rh - this is the change we discussed in https://redhat-internal.slack.com/archives/C04CUSD4JSG/p1768421642057969
IMPORTANT: Do Not Merge - To be merged by Docs Team Only
Version(s):
1.9 for sure
1.8 if we are making any more 1.8 updates to https://docs.redhat.com/en/documentation/red_hat_developer_hub/1.8/
Issue:
https://issues.redhat.com/browse/RHDHBUGS-2501
Preview:
Files changed:
modules/openshift-ai-connector-for-rhdh/proc-setting-up-openshift-ai-connector-for-rhdh-with-rhoai.adoc
modules/openshift-ai-connector-for-rhdh/ref-enrich-ai-model-metadata.adoc