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

[FEATURE]: Reset Applicator when Gossip drops #316

Closed
1 task done
mvandeberg opened this issue Dec 9, 2024 · 0 comments · Fixed by #319
Closed
1 task done

[FEATURE]: Reset Applicator when Gossip drops #316

mvandeberg opened this issue Dec 9, 2024 · 0 comments · Fixed by #319
Assignees
Labels
enhancement New feature or request

Comments

@mvandeberg
Copy link
Member

Is there an existing issue for this?

  • I have searched the existing issues

New feature

We know that the applicator is getting in to a blocked state and is unable to make progress. We have identified several issues causing this behavior, but are still receiving reports. This is complicated code and will have a long tail to fix every bug.

In the meantime, we should add a 'reset' function on the applicator that is called when gossip is toggled off. This will fix the applicator and allow the node to sync and resume normal function.

When reset is called, the applicator should also dump its internal state so that we can inspect what caused the problem, which will aid in future debugging.

Anything else?

No response

@mvandeberg mvandeberg added the enhancement New feature or request label Dec 9, 2024
@mvandeberg mvandeberg self-assigned this Dec 9, 2024
mvandeberg added a commit that referenced this issue Dec 11, 2024
@mvandeberg mvandeberg mentioned this issue Dec 11, 2024
5 tasks
mvandeberg added a commit that referenced this issue Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant