-
Notifications
You must be signed in to change notification settings - Fork 9.8k
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
embed: Can't restart etcd #6042
Comments
Can you provide code snippet to reproduce? Thanks. |
@gyuho Fair enough, I don't have a trivial snippet at the moment, I'll reopen when I get one. Thanks! |
I forget if this is related to: golang/go#4674 or not. |
Apologies for the delay, but I can reliably reproduce this issue, so I will re-open this. I have an easy POC inside of https://github.com/purpleidea/mgmt/ |
Here is the reproducer: start up three members...
tell the ideal cluster size to be three...
check that it is...
add two more clients...
tell the cluster size to be 4...
one more member will be started now...
set it back to three...
make note of who shutdown...
bring it back to 4...
you'll most likely have one that previously started, try to start again... repeat the above 3->4->3 if not.
in the logs of that member:
I believe I am using the embed API correctly, but if there is some additional shutdown/unbind step that I should be performing that I am not, then please let me know. Thanks! |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 21 days if no further activity occurs. Thank you for your contributions. |
hi bot! please stop pinging here |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 21 days if no further activity occurs. Thank you for your contributions. |
Bump, I don't believe this is resolved (IIRC in my integration tests cannot stop and start up embedded etcd again due to this issue). At the least deserves confirmation that's fixed before closing. |
@cretz It's not fixed last I checked, but I'm giving up fighting with the bots, it's very end-user hostile I think. One bump should be enough for the lifetime of the bug. And the bot bugged about eight others today. :/ |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 21 days if no further activity occurs. Thank you for your contributions. |
Stop closing this bot. This is important. |
I believe it might be about (etcd not using) 'SO_REUSEADDR'. Without this setting port is 'locked' for additional 60-120s, That's pretty good article about this: |
Think you can send a patch to fix this?
…On Mon, Oct 5, 2020 at 4:34 AM Piotr Tabor ***@***.***> wrote:
I believe it might be about (etcd not using) 'SO_REUSEADDR'. Without this setting port is 'locked' for additional 60-120s,
to avoid getting left-over communication from previous customers.
That's pretty good article about this:
https://stackoverflow.com/questions/3229860/what-is-the-meaning-of-so-reuseaddr-setsockopt-option-linux/3233022#3233022
and https://hea-www.harvard.edu/~fine/Tech/addrinuse.html
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 21 days if no further activity occurs. Thank you for your contributions. |
bot |
I think / hope it got fixed in: Please test and reopen. |
@ptabor Fantastic news, thanks! I'll test in 3.5 |
Etcd now has an embed API. Sweet :)
I believe there is still a bind problem when starting a server, then stopping it, and then starting it again. On the second start, you'll get a
bind: address already in use
error because the listen isn't closed properly.Opening this issue at the request of @heyitsanthony
I think this may be casually related to #2920
Thanks!
The text was updated successfully, but these errors were encountered: