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

Fatal node sync call stops subsequent calls #842

Closed
prd-fox opened this issue Aug 3, 2019 · 1 comment · Fixed by #843
Closed

Fatal node sync call stops subsequent calls #842

prd-fox opened this issue Aug 3, 2019 · 1 comment · Fixed by #843
Assignees
Labels
0.10.0 bug Something isn't working

Comments

@prd-fox
Copy link
Contributor

prd-fox commented Aug 3, 2019

When the PartyInfo poller makes requests to other nodes there are two classes of exceptions that can occur, general connection issues and then other, more serious issues.

A simple connection issue gets logged and passed over, as this is expected to happen from time to time, and all other exception types were presumed to be ephemeral and eventually resolve. We throw this error back up the call stack to handle later.

This means that when this error occurs, all other parties we wanted to sync with after the current do not happen (as the exception stops the loop). Sometimes, the error will never resolve, which effectively stops the node fully syncing with the network.

Any error, whether it is a general connection issue or something more serious, should not stop the processing of other calls.

@prd-fox prd-fox added the bug Something isn't working label Aug 3, 2019
@prd-fox prd-fox self-assigned this Aug 3, 2019
@prd-fox
Copy link
Contributor Author

prd-fox commented Aug 3, 2019

One example when this can occur is inside a Docker network, where the target does not exist.

Generally these scenarios are very rare, and workarounds such as removing the fault peer entry from the config will enable at least a one time sync to the network.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.10.0 bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant