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

Bump testnet checkpoint and nMinimumChainWork/defaultAssumeValid params #2026

Merged
merged 2 commits into from
Apr 11, 2018

Conversation

UdjinM6
Copy link

@UdjinM6 UdjinM6 commented Apr 11, 2018

New checkpoint should help new (or reindexed) nodes to stay on the right chain. nMinimumChainWork /defaultAssumeValid should make sync a bit faster by skipping a lot of verifications in 2mb spam blocks.

UdjinM6 added 2 commits April 11, 2018 12:17
blockhash: 0000000003aa53e24b6e60ef97642e4193611f2bcb75ea1fa8105f0b5ffd5242
@UdjinM6 UdjinM6 added this to the 12.3 milestone Apr 11, 2018
Copy link

@schinzelh schinzelh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just bumping the checkpoint did not fix it, curious to see if the second part of the patch will work :)

utACK

@UdjinM6
Copy link
Author

UdjinM6 commented Apr 11, 2018

@schinzelh have you tried reindexing after the bump?

@schinzelh
Copy link

@UdjinM6 i think so, but i can try again just in case. The issue is that the nodes stay on the correct chain, but still throw

2018-04-11 09:35:38 ERROR: detected bad peer for initial headers sync, disconnecting nnnnn

and keep on banning themselves from the network.

@UdjinM6
Copy link
Author

UdjinM6 commented Apr 11, 2018

@schinzelh I'm pretty sure reindexing should fix this but if not then that's going to be fixed by another patch I'm working on atm (hopefully) :)

@UdjinM6
Copy link
Author

UdjinM6 commented Apr 11, 2018

Alternatively, you can try to sync from scratch.

@schinzelh
Copy link

@UdjinM6 you are right, reindexing with bumped checkpoint is fixing it.

So utACK --> tested ACK :)

@UdjinM6
Copy link
Author

UdjinM6 commented Apr 11, 2018

Nice, thanks! :)

Copy link

@codablock codablock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@UdjinM6 UdjinM6 merged commit 8b09e77 into dashpay:develop Apr 11, 2018
@UdjinM6 UdjinM6 deleted the fixtestnet branch November 26, 2020 13:26
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 this pull request may close these issues.

3 participants