-
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
service on docker with Ctrl-C: error stopping tunnel: stopping ssh tunnel: os: process already finished
#8511
Comments
@tstromberg & @medyagh - I am new to this community and looking to learn and contribute. Can I pick this up ? and start experiment if no one else is looking into it. |
Absolutely! You are more than welcome to pick this up. Thanks!
β¦On Thu, Jun 18, 2020 at 7:45 PM Vinodhini Balusamy ***@***.***> wrote:
@tstromberg <https://github.com/tstromberg> & @medyagh
<https://github.com/medyagh> - I am new to this community and looking to
learn and contribute. Can I pick this up ? and start experiment if no one
else is looking into it.
Thanks!
β
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#8511 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAYYMB5SGMWSYAQW765EODRXLGLBANCNFSM4OB54BTQ>
.
|
/assign @vinu2003 |
@tstromberg I have verified this issue in my local macOS setup with docker. But the error that is returned from os.signal function in go seems to be private:
And there is discussion/proposal to export errFinished to ErrFinished. All your concern is about these below lines when SIGINT is sent from. πΏ minikube is exiting due to an error. If the above message is not useful, open an issue: I suggest we can get this in place and handle the specific error case here and exit gracefully in minikube/cmd/service.go if it's the proposal is approved. Keen to know your thoughts on this. |
Proposal to export errFinished - golang/go#39444 seems to be OK. Its in final comment period. |
https://go-review.googlesource.com/c/go/+/242998/ - proposal accepted. Once its in we can update and get similar issues error handling addressed correctly. |
Issues go stale after 90d 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. |
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. |
@vinu2003 are you still interested in fixing this issue? |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-contributor-experience at kubernetes/community. |
Stale issues rot after 30d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-contributor-experience at kubernetes/community. |
/assign |
Please reach out if you have any questions working on this issue. |
I just experienced this issue while going through the Example: Deploying WordPress and MySQL with Persistent Volumes. I know neither Go nor Kubernetes, but I am interested in the latter. So I'm interested in the solution here. I can help test where necessary |
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 |
This is a useless and necessarily scary error message from
minikube service
when Ctrl-C is hit on macOS and the Docker driver:The text was updated successfully, but these errors were encountered: