kubeflow / pipelines

Machine Learning Pipelines for Kubeflow
https://www.kubeflow.org/docs/components/pipelines/
Apache License 2.0
3.59k stars 1.62k forks source link

[feature] Option to disable Caching in V2 at the KFP, Pipeline, and Run level #10839

Open HumairAK opened 4 months ago

HumairAK commented 4 months ago

Feature Area

/area frontend /area backend

What feature would you like to see?

Allow flexibility in being able to enable/disable caching at multiple levels in KFP. A user should be able to:

The user should be able to do this VIA the UI, or API. I don't think this requires changes from SDK side, but if it does, we should point that out.

As part of this acceptance criteria, the implementer should also update the configurations to clearly indicate where the old v1 caching behavior is for v1 only and not for v2. Example here.

What is the use case or pain point?

Currently the only way to disable caching in KFP V2 is via the SDK , by setting .set_caching_options.

This can be annoying to do for every single step, every single time. In some cases a user may not be satisfied with the current caching capabilities and want to opt to not use it, the user should be easily enabled to do so.

This feature seems like it used to exist with kfp v1 cache server to some degree, but no longer exists in kfpv2.

Is there a workaround currently?

The only workaround is to disable caching via sdk on a per component basis.


Love this idea? Give it a 👍.

HumairAK commented 4 months ago

It looks like there was a lot of work on this done by @TobiasGoerke in the past here: https://github.com/kubeflow/pipelines/pull/8177

It seems like it didn't end up getting merged, but could serve as a useful reference.

HumairAK commented 4 months ago

Related: https://github.com/kubeflow/pipelines/issues/8104

gregsheremeta commented 3 months ago

~I will begin working on this~

edit: scratch that, I'm not going to work on (all) of this quite yet

HumairAK commented 3 months ago

/assign gregsheremeta

gregsheremeta commented 1 month ago

partial duplicate of #6578

gregsheremeta commented 1 month ago

/unassign @gregsheremeta