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

Full node stops receiving gossiped blocks when disconnected from sequencer #546

Closed
srene opened this issue Dec 27, 2023 · 4 comments · Fixed by #556
Closed

Full node stops receiving gossiped blocks when disconnected from sequencer #546

srene opened this issue Dec 27, 2023 · 4 comments · Fixed by #556
Assignees

Comments

@srene
Copy link
Contributor

srene commented Dec 27, 2023

When a full node disconnects from the sequencer, it does not reconnect to it unless the node is restarted, when there are only two nodes. Need to check if it happens when there are more nodes.

@srene srene self-assigned this Dec 27, 2023
@srene
Copy link
Contributor Author

srene commented Dec 29, 2023

After a few checks, it seems nodes are reconnecting automatically after approx 2 minutes, but it needs more investigation to see if it happens in all cases and why after 2 minutes.

@srene
Copy link
Contributor Author

srene commented Dec 29, 2023

Kad-dht libp2p implementation runs a periodic check that, if not enough peers on routing tables, it bootstraps again. The process reruns every 2 minute, according to the bootstrap config parameter here:
https://github.com/libp2p/go-libp2p-kad-dht/blob/bdca14419ec5f7ad0790f17856964bb99bee7436/dht_bootstrap.go#L20

it does not seem to be externally configurable.
Therefore it seems in the worse case, a full node will be disconnected during 2 minutes and it will connect again automatically.

@srene
Copy link
Contributor Author

srene commented Dec 29, 2023

@srene
Copy link
Contributor Author

srene commented Jan 2, 2024

PR #556 adds a new function to periodically check if a full node is not connected to other nodes and bootstraps the DHT again to reconnect to sequencer.

@srene srene linked a pull request Jan 2, 2024 that will close this issue
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant