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

Unable to sync the blockchain #900

Closed
b00f opened this issue Jan 3, 2024 · 1 comment · Fixed by #901
Closed

Unable to sync the blockchain #900

b00f opened this issue Jan 3, 2024 · 1 comment · Fixed by #901

Comments

@b00f
Copy link
Collaborator

b00f commented Jan 3, 2024

Description

Unable to sync the blockchain in Testnet at block 35861. Restarting the node won't fix the issue.

WRN committing block failed, removing block from the cache error="invalid public key: public key is not set" _sync="{☍ 30 ⛃ 240}" height=35861

What Happened

Fixing #869 has backfired. Block 35861 contains a Bond transaction for a bootstrap validator. The bootstrap validator has a public key, but at the same time, the LastBondingHeight for the bootstrap validator has been set to zero.

We can fix this by adding a new method to the Sandbox to determine if the Validator is sandboxed or not. However, this will add more complexity to the code. The issue #869 is rare and occurs only once in Testnet. This issue is not permanent; for example, after one day (once the first transaction expires), it will be fixed, or users can broadcast the transaction to other nodes. Fixing both issues will increase the code complexity and make the code harder to understand.

It's better to revert the fix for #869.

@b00f
Copy link
Collaborator Author

b00f commented Jan 3, 2024

We can monitor the Testnet and if the issue #869 occurs more, we can decide for a better fix.

@kehiy kehiy closed this as completed in #901 Jan 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant