-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Enable auto-update #375
Enable auto-update #375
Conversation
"github.com/spf13/viper" | ||
"k8s.io/minikube/pkg/minikube/config" | ||
"k8s.io/minikube/pkg/minikube/constants" | ||
"k8s.io/minikube/pkg/version" | ||
) | ||
|
||
const updateLinkPrefix = "https://github.com/kubernetes/minikube/releases/tag/v" | ||
const downloadLinkFormat = "https://github.com/kubernetes/minikube/releases/download/v%s/%s" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've had trouble hitting github rate limits when querying URLs like this. Have you seen anything like that yet? What do you think about extending the releases.json file to contain the SHA instead of hitting Github?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I forgot about that. I always use API tokens so don't hit those limits but most users probably won't do that.
Yeah updating release json makes a lot of sense. Need to automate the release process to add this in but will document it for now.
High-level question, how do you see this working with multiple binaries (when we start bundling the drivers, for example)? |
Not sure, we could do it with a self extracting binary, using the same go-bindata we use for localkube? |
Would you mind holding onto this until we figure out the packaging story a little more? I want to make sure whatever auto-update we get in place works once we have OS-specific installers (brew, .msi, .deb/.rpm, etc.) that bundle several binaries. |
100% agree. If we're using OS-specific installers then I wouldn't want to do autoupdate, we can leave that up to the package manager, can't we? |
I'd like to have this in |
Issues go stale after 90d of inactivity. Prevent issues from auto-closing with an If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or |
Stale issues rot after 30d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
Rotten issues close after 30d of inactivity. Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
Fixe #373