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

Add caching for iso, kic, and preload #46

Merged
merged 3 commits into from
Dec 7, 2022

Conversation

spowelljr
Copy link
Collaborator

Related #16

@spowelljr spowelljr mentioned this pull request Dec 6, 2022
Copy link
Owner

@medyagh medyagh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good a few nits

return cacheHits
}

async function getMinikubeVersion(): Promise<string> {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is not used yet, right ? if so lets comment on top of it (to be used later to cache binary)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is currently being used for adding the version to cache keys.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does that mean if Preload stays same but minikube version changes, it wont use that prelaod from old minikube version in cache ?

Copy link
Collaborator Author

@spowelljr spowelljr Dec 6, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's correct, my one worry is if we bump the preload version (v18) between releases that we don't want to keep using the old version.

The two options I can think of if we don't want to change the preload cache between minikube versions is:
1. Look at the contents of the preload directory to get the version (This won't work as we don't know the preload version before we start minikube)
2. Add a preload version command to minikube that reports the version of the preload

@medyagh medyagh merged commit ec95abe into medyagh:master Dec 7, 2022
@spowelljr spowelljr deleted the addCaching branch December 7, 2022 16:54
@antonioned antonioned mentioned this pull request Mar 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants