-
Notifications
You must be signed in to change notification settings - Fork 42
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
feat: node recovery strategies #2076
Comments
From discussion:
|
ping @vpavlin @hackyguru for perspective on the issue as we are not sure if it is needed |
I am a bit confused by
I'd expect there are basically 2 methods - "find new nodes" and "hard reset" which can potentially be used in any health state by the app dev in case their app notices anything weird? And then automating it on waku level by setting Or maybe this is what has been said?:) |
@vpavlin there could be two methods or just one depending how we find it better. but the question here is more - do we need it? Have you noticed before such a need when talked to people that use |
As we don't have enough evidence it would give a lot of improvement for developers - iceboxing for now. |
From https://github.com/waku-org/support/issues/2
Ideally |
This is a feature request
Problem
Prerequisite: #2070
Once we can understand in what state a light node is we should provide clear strategies for consumers to recover from undesirable states.
Proposed Solutions
For each node health state we should develop a method that should be triggered to recover from it.
Additionally we should introduce option
networkRecovery: boolean
that would make triggering of such states automatic.This behavior should be
off
by default and tested before making iton
by default.Node health state:
hard reset
operation that would re-establish connection to bootstrap nodes and will start all over again;networkRecovery
option was provided;Note:
hard reset
operation won't help in case if node is offline (from Internet) and we should be clear about it in our API / documentation / behavior.The text was updated successfully, but these errors were encountered: