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

Refactor sync checks #169

Closed

Marketen wants to merge 1 commit into main from cfg_with_synccheck

Contributor

Marketen commented Jul 11, 2023

Deleted node sync checks in "NewOnchain()" method, since it wasnt doing anything with the sync status, just logging it.
Added sync check at startup of oracle, using the already implemented method "onchain.AreNodesInSync()", which prints and returns sync information. If not in sync, oracle wont try to load onchain configuration parameters and will try again after 1 minute.

Notice: Execution clients sync status is prone to be misleading on startup. This is not fixed in this PR.


          refactor_sync_checks

3a00005

Marketen requested a review from alrevuelta

July 11, 2023 17:18

Marketen self-assigned this

alrevuelta closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet