Skip to content
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

Closed
zdtsw opened this issue Aug 29, 2023 · 9 comments · Fixed by #107
Closed

Move from odh-manifests to Kserve source repo #72

zdtsw opened this issue Aug 29, 2023 · 9 comments · Fixed by #107
Assignees
Labels

Comments

@zdtsw
Copy link
Member

zdtsw commented Aug 29, 2023

/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 in kserve/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 in kserve/config will produce no less function as run from odh-manifets/kserve/base

@openshift-ci openshift-ci bot added the kind/feature New feature label Aug 29, 2023
@zdtsw zdtsw changed the title Move from odh-manifests to Kserve source reop Move from odh-manifests to Kserve source repo Aug 29, 2023
@israel-hdez
Copy link

Hey, @zdtsw

Is params.env still going to be the preferred method to parametrize images?

I've heard that it may be going away.

@zdtsw
Copy link
Member Author

zdtsw commented Sep 21, 2023

Hey, @zdtsw

Is params.env still going to be the preferred method to parametrize images?

I've heard that it may be going away.

Correct!
I've not heard anything that we plan to move away from params.env
for ODH it is not important to have params.env, but downstream still will rely on this file.
it is up to team to decide if to keep a placeholder params.env in ODH or not.

@israel-hdez
Copy link

@zdtsw A (basic?) question:

For operator v1, we still needed older kustomize format. So, odh-manifests needed to be compatible with the older format.

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?

@zdtsw
Copy link
Member Author

zdtsw commented Sep 22, 2023

yes, you are correct
we will still support v1 release for a couple of rounds. and v1 operator need old format of manifests to work with pre-v5 kustomize build.

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.

@zdtsw
Copy link
Member Author

zdtsw commented Sep 22, 2023

if, you already migrated "manifests" in "kserve" repo for v5 kustomize
there is a workaround for v1 operator, but that will require you to keep updating/using "odh-manifests" repo.

e.g:
in "kserve" repo, the manifests are in the new format, this will be used by v2 operator
in "odh-manifests/kserve", you generate v3 compatible manifests for kserve, this will be used by v1 operator.
i would not recommand to do this way, but if that is the case, we can work it out together.

@zdtsw
Copy link
Member Author

zdtsw commented Sep 22, 2023

sorry, must correct myself.

in v1 operator, kserve is not supported, only modelmesh is.
so if you migrate manifests in kserve repo for v5 kustomize, it should be fine for v2 operator.

@israel-hdez israel-hdez self-assigned this Oct 12, 2023
@israel-hdez israel-hdez linked a pull request Oct 13, 2023 that will close this issue
1 task
@israel-hdez israel-hdez removed a link to a pull request Oct 17, 2023
1 task
@heyselbi heyselbi linked a pull request Oct 18, 2023 that will close this issue
1 task
@heyselbi
Copy link

@israel-hdez could you please attach the PRs updating the SHA tags once they are ready and then close the issue?

@heyselbi
Copy link

@israel-hdez
Copy link

All done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Status: No status
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants