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.

Sign up for GitHub

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

Jump to bottom

[v2compat] caching design #5726

Closed
Bobgy opened this issue May 24, 2021 · 3 comments
Closed

[v2compat] caching design #5726

Bobgy opened this issue May 24, 2021 · 3 comments
Assignees
@Bobgy @capri-xiyue
Labels
size/L

Comments

@Bobgy
Copy link
Contributor

Bobgy commented May 24, 2021 •
edited
Loading

Part of #5667
@Bobgy Bobgy added the size/L label May 24, 2021
@Bobgy Bobgy changed the title KFP data model based caching design [v2compat] caching design May 24, 2021
@Bobgy Bobgy mentioned this issue May 24, 2021
Closed
7 tasks
@juliusvonkohout
Copy link
Member

@Bobgy please keep in mind that cpu and memory ressource requests must be set for compatibility with kubeflow profile quotas. I tried something but could not get it working yet #5743

@Bobgy
Copy link
Contributor Author

Bobgy commented Jun 10, 2021 •
edited
Loading

Hi @juliusvonkohout , the problem you mentioned is with KFP v1 caching.

In v2 compatible caching design, we won't change the Pod spec when it's cached, so this will no longer be a problem.
In v2 design, we will skip the Pod entirely, so this won't be a problem either.

@capri-xiyue
Copy link
Contributor

Design Doc finished

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Bobgy Bobgy

@capri-xiyue capri-xiyue

Labels
size/L
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Bobgy @juliusvonkohout @capri-xiyue