pipelines icon indicating copy to clipboard operation
pipelines copied to clipboard

[feature] Should be able to set _DEFAULT_LAUNCHER_IMAGE using environment variable in notebook pod

Open typhoonzero opened this issue 3 years ago • 6 comments

Feature Area

What feature would you like to see?

Currently the default launcher image is set using a constant: sdk/python/kfp/compiler/v2_compat.py:_DEFAULT_LAUNCHER_IMAGE = "gcr.io/ml-pipeline/kfp-launcher:1.8.7", however, if we want to use a private registry when launching a pipeline in Kubeflow notebook, we may need to use an environment variable to set this, so that we can just configure that in the Kubeflow notebook YAML file.

Is there a workaround currently?

I can set it directly in the function call here: https://kubeflow-pipelines.readthedocs.io/en/latest/source/kfp.client.html#kfp.Client.create_run_from_pipeline_func, but I'd like to configure a default for all users.


Love this idea? Give it a 👍. We prioritize fulfilling features with the most 👍.

typhoonzero avatar Jan 27 '22 07:01 typhoonzero

Hi @typhoonzero, v2 compatible mode is being deprecated in the next SDK release: https://github.com/kubeflow/pipelines/issues/6829

We would have a similar launcher/driver image for v2. And we plan to version it the same way as we version the rest KFP backend images. That being said, I'm still interested in why would you want to use use private registry for the launcher image. Are you making custom images? Can you share some details to help us understand your use case? Thanks!

chensun avatar Feb 03 '22 23:02 chensun

Thanks for the reply @chensun , my case is simple, I can't access gcr.io in China, so I'm hoping to host the images in a private registry.

typhoonzero avatar Feb 08 '22 01:02 typhoonzero

any update on this?

dbg-raghulkrishna avatar Oct 05 '22 11:10 dbg-raghulkrishna

+1

wang-mask avatar Dec 23 '23 07:12 wang-mask

how to set the image in create_run_from_pipeline_func?

wang-mask avatar Dec 23 '23 07:12 wang-mask

https://github.com/kubeflow/pipelines/pull/10269#issuecomment-1916254578

xin053 avatar Jan 30 '24 08:01 xin053

Closing this issue as it was implemented. Feel free to reopen it if issue persists.

/close

rimolive avatar Mar 12 '24 08:03 rimolive

@rimolive: Closing this issue.

In response to this:

Closing this issue as it was implemented. Feel free to reopen it if issue persists.

/close

Instructions 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/test-infra repository.

google-oss-prow[bot] avatar Mar 12 '24 08:03 google-oss-prow[bot]