-
Notifications
You must be signed in to change notification settings - Fork 63
Can we have a kustomizev3 manifest which can deploy KFP 0.5.1 with Tekton backend #27
Comments
@adrian555 Do you know what's the best method to deploy kubeflow manifest using kustomizev3? The KFP team has the v3 manifest ready but they don't have it integrate with kfdef. |
if it is a kustomize v3 manifest, then you can just run |
ok - i was thinking more in context of larger kubeflow deployment on ibmcloud a version of kfctl/kfdef/kustomize which deploys our implementation of kubeflow pipelines with tekton. We then ensure @shawnzhu is recommending that version as well. |
Will this be a standalone kfp+tekton installation or part of the kubeflow (ie. part of the kfctl_ibm.yaml)? If it is the latter case, we will need some modification in the But likely should confirm @Tomcli whether the v3 kfp manifests contain kfp0.5.1 or else. |
I'm wondering is there a single kustomization.yaml for deploying all the Kubeflow component? With the v3 manifest in kubeflow right now, we don't have an easy way to maintain all our custom configuration in a single place. Also can kfdef apply the overlays on v3 kustomization base? if so then we can keep using kfdef as our single entrypoint. Here is the pipeline v3 kustomization. |
The v3 kustomization base should work for kfp 0.4 and above. Right now their v3 base image is kfp 1.0-rc2 |
It does not exist in the manifests repo but you can write one simply by adding all the components as the resources to a kustomization.yaml.
The purpose of kustomize v3 update includes removing the parameters and overlays fields from the kfdef. But the source code processing the overlays etc is still there. So in theory, it can. However, if we still need such ability, why not we just stick with the pre-v3 configuration? Seriously with the recent trouble we encountered trying to support dex version of kfctl_ibm.yaml, it makes me think we should reconsider whether continue to depend on kfdef and kfctl. |
done through #37 |
cc @adrian555 @Tomcli
The text was updated successfully, but these errors were encountered: