-
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
minikube delete should not get stuck, if Docker Desktop is stuck or responding too slow #12846
Comments
/assign |
the least it should do is, if we get "Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running?" error we should out.Warn to the user, this might take a long time,...and tell them we are stuck at Docker part so they dont be bored waiting for nothing |
I guess we can always check the existence of /var/run/docker.sock before running each docker command. But I'm not sure if this is sufficient since it's specific to docker. OTOH, there is already functionality to warn the user of a slow command: https://github.com/kubernetes/minikube/blob/master/pkg/drivers/kic/oci/cli_runner.go#L148-L152. I can just enable it by default. The caveat is that enabling warning also enables timeout though. Otherwise, since warnSlow doesn't seem to be used by anything yet, we can repurpose warnSlow to killSlow, which is off by default, and then enable warning by default. WDYT? |
we could do it in a verity of ways ! I would for dissussion if u make a PR to see a POC and I beleive it would be a cheap call to check if driver is Docker in oci.Cli Runner that pasess the ociBin to it |
@yayaha are u still working on this |
Hi @medyagh, sorry for the delay. No, I'm not working on this anymore. |
/unassign |
Hi @chungjin, this is an issue you can work on if you're interested. |
/assign |
Hi @medyagh , when I running it locally, I never faced this issue, do you know how often we may face this situation? Maybe it depends on the docker version? If it is rare, maybe we don't need to deal with it? If we must process it, maybe we can save the stderr to a tmp file and analysis the log, but i am not sure the overhead of it. |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
I took so long that I Ctrl-Ced it and I realized Docker Desktop itself is taking SUPER SUPER longtime to respond
I re-ran it with --alsologtostderr
the right thing to do is minikube should inform the user that Docker Desktop is taking a long time (if it takes more than 10 seconds and suggest them to either keep waiting or restart docker
The text was updated successfully, but these errors were encountered: