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

fix(consensus): update Zebra's hard-coded blockchain checkpoint lists #3606

Merged
merged 3 commits into from
Feb 22, 2022

Conversation

teor2345
Copy link
Contributor

Motivation

Updating the checkpoints should speed up the full sync tests.

Solution

This updates the checkpoint lists as at 2022-02-22:

This is part of #2368, but I didn't check off any tasks, because we'll need to update the lists again closer to NU5.

Review

@gustavovalverde can review this PR.

Reviewer Checklist

  • Existing tests pass

Follow Up Work

Update the checkpoints again before NU5 activation:

This updates the checkpoint lists as at 2022-02-22:
- mainnet: to the latest finalized tip
- testnet: to the latest Canopy checkpoint,
  because we expect a NU5 testnet rollback
@teor2345 teor2345 added C-enhancement Category: This is an improvement NU-5 Network Upgrade: NU5 specific tasks P-High 🔥 labels Feb 22, 2022
@teor2345 teor2345 self-assigned this Feb 22, 2022
@teor2345 teor2345 mentioned this pull request Feb 22, 2022
3 tasks
@codecov
Copy link

codecov bot commented Feb 22, 2022

Codecov Report

Merging #3606 (c03f763) into main (499ae89) will increase coverage by 1.83%.
The diff coverage is 75.85%.

@@            Coverage Diff             @@
##             main    #3606      +/-   ##
==========================================
+ Coverage   78.34%   80.17%   +1.83%     
==========================================
  Files         267      276       +9     
  Lines       31526    32420     +894     
==========================================
+ Hits        24698    25993    +1295     
+ Misses       6828     6427     -401     

@gustavovalverde gustavovalverde merged commit 7e585b0 into main Feb 22, 2022
@gustavovalverde gustavovalverde deleted the update-pre-nu5-checkpoints branch February 22, 2022 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: This is an improvement NU-5 Network Upgrade: NU5 specific tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants