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.

Sign up for GitHub

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

Jump to bottom

Refresh removed nodes by maintainer #138

Closed
herr-seppia opened this issue Mar 7, 2024 · 0 comments · Fixed by #141
Closed

Refresh removed nodes by maintainer #138

herr-seppia opened this issue Mar 7, 2024 · 0 comments · Fixed by #141
Labels
audit

Comments

@herr-seppia
Copy link
Member

Summary

During maintanance, idle nodes are deleted with a full override of the eviction process.

Possible solution design or implementation

  • Modify 'remove_idle_nodes()' to return the list of removed nodes.
  • Immediately send a PING to the removed nodes.

This aims to ensure that if the removed peers are indeed alive, they would be reinserted promptly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
audit
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Change maintainer to ping nodes while removal dusk-network/kadcast
1 participant
@herr-seppia