-
Notifications
You must be signed in to change notification settings - Fork 21
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
Move from odh-manifests to Kserve source repo #72
Comments
Hey, @zdtsw Is I've heard that it may be going away. |
Correct! |
@zdtsw A (basic?) question: For operator v1, we still needed older kustomize format. So, I understand that for per-repo manifests, which I'm assuming it is only for operator v2, we no longer have that restriction and we can use the very latest kustomize v5 format. Is this right? Or do we still need to support the older format? |
yes, you are correct in this case ( till we do not release new version for v1): manifests from kserve (and all components) need to stay with the old syntax. we will have another "epic" to follow-up the upgrade "manifests" for all components. |
if, you already migrated "manifests" in "kserve" repo for v5 e.g: |
sorry, must correct myself. in v1 operator, kserve is not supported, only modelmesh is. |
@israel-hdez could you please attach the PRs updating the SHA tags once they are ready and then close the issue? |
All done. |
/kind feature
Describe the solution you'd like
As part of the transition from central odh-manifests to component repo work, it would be good to have a placeholder file
params.env
inkserve/config
folder to host kserve images.this has previously done in https://github.com/opendatahub-io/odh-manifests/blob/master/kserve/base/params.env
Anything else you would like to add:
Ensure all manifests required by kserve in https://github.com/opendatahub-io/odh-manifests/tree/master/kserve can be mapped to https://github.com/opendatahub-io/kserve/tree/master/config
i.e running
kustmoize build
inkserve/config
will produce no less function as run fromodh-manifets/kserve/base
The text was updated successfully, but these errors were encountered: