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

History root doesn't match real history root #2347

Open
viquezclaudio opened this issue Apr 4, 2024 · 1 comment · May be fixed by #2533
Open

History root doesn't match real history root #2347

viquezclaudio opened this issue Apr 4, 2024 · 1 comment · May be fixed by #2533

Comments

@viquezclaudio
Copy link
Contributor

This was observed when a full node was trying to synchronize to the testnet.
It happens after a second validity window synchronization process is started and the existing and some existing history is removed

log4.log

@viquezclaudio
Copy link
Contributor Author

viquezclaudio commented Apr 10, 2024

This problem is caused by a node that connects and has a newer macro head than the one we have, and we are currently syncing to the tip of the chain and sufficiently close for live sync but state sync is not completed yet.

This scenario should have moved the peer to live sync instead of pushing the macro block, but since our accounts is not complete, then we are doing macro sync with this peer.

I.e.: the following condition needs to be revisited:

    if peer_head_upper_bound.saturating_sub(our_head) <= self.full_sync_threshold
                        && blockchain.accounts_complete()

@styppo styppo added this to the Nimiq PoS Mainnet milestone May 20, 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.

2 participants