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

loss of sync peer candidates, max peers reached #2694

Closed
raedah opened this issue Aug 2, 2021 · 3 comments · Fixed by #3251
Closed

loss of sync peer candidates, max peers reached #2694

raedah opened this issue Aug 2, 2021 · 3 comments · Fixed by #3251
Labels
Milestone

Comments

@raedah
Copy link
Contributor

raedah commented Aug 2, 2021

[WRN] BMGR: No sync peer candidates available

[INF] SRVR: Max peers reached [125] - disconnecting peer x.x.x.x:x (inbound)

Not sure what triggered the above conditions, but these both show in the log and dcrd was not able to recover until manually restarted.

@davecgh
Copy link
Member

davecgh commented Aug 2, 2021

Thanks for the report. It's the first time I've seen it in nearly a decade, but I do understand how it can theoretically happen.

@davecgh davecgh modified the milestone: 1.8.0 Sep 15, 2021
@davecgh davecgh added the bug label Sep 15, 2021
@ukane-philemon
Copy link

@davecgh pls can you throw more light on how it can theoretically happen?

@davecgh
Copy link
Member

davecgh commented Oct 16, 2021

Essentially, there isn't explicit handling to 100% prevent it from happening, so that makes it possible even though the probability is extremely low such that it essentially almost never happens. I should also note that it eventually will recover when some of the existing peers disconnect, but that could take a while since peers are not ordinarily disconnected without an explicit reason which could make it seem like it couldn't recover automatically.

All of that said, the final outstanding part of #1145 gets rid of the entire notion of a sync peer and thus will resolve it indirectly too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants