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

Change the mandatory checkpoint to Canopy #1846

Closed
2 tasks
teor2345 opened this issue Mar 5, 2021 · 0 comments · Fixed by #1898
Closed
2 tasks

Change the mandatory checkpoint to Canopy #1846

teor2345 opened this issue Mar 5, 2021 · 0 comments · Fixed by #1898
Assignees
Labels
A-rust Area: Updates to Rust code C-enhancement Category: This is an improvement NU-2 Blossom Network Upgrade: Blossom specific tasks NU-3 Heartwood Network Upgrade: Heartwood specific tasks NU-4 Canopy Network Upgrade: Canopy specific tasks NU-5 Network Upgrade: NU5 specific tasks
Milestone

Comments

@teor2345
Copy link
Contributor

teor2345 commented Mar 5, 2021

Is your feature request related to a problem? Please describe.

Zebra should change its mandatory checkpoint to the Canopy network upgrade, to reduce the work we need to do for NU5 activation.

Checkpointing on Canopy lets us skip the following validation:

  • transfers into the Sprout value pool (rejected at Canopy activation)
  • Founders Reward (finished right before Canopy activation)
  • validation of the PreSapling, Sapling, and ChainHistoryActivation (Heartwood activation) RootHash variants
  • the pre-Blossom block target spacing (finished right before Blossom activation)

Describe the solution you'd like

  • Update the list of Zebra checkpoints for mainnet and testnet
  • Change the Sapling mandatory checkpoint to Canopy

Describe alternatives you've considered

Write a whole lot of validation code that we'll never really use.

@teor2345 teor2345 added A-rust Area: Updates to Rust code NU-3 Heartwood Network Upgrade: Heartwood specific tasks NU-4 Canopy Network Upgrade: Canopy specific tasks NU-2 Blossom Network Upgrade: Blossom specific tasks C-enhancement Category: This is an improvement S-needs-triage Status: A bug report needs triage NU-5 Network Upgrade: NU5 specific tasks P-Medium labels Mar 5, 2021
@mpguerra mpguerra removed the S-needs-triage Status: A bug report needs triage label Mar 8, 2021
@mpguerra mpguerra added this to the 2021 Sprint 5 milestone Mar 8, 2021
@oxarbitrage oxarbitrage self-assigned this Mar 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-rust Area: Updates to Rust code C-enhancement Category: This is an improvement NU-2 Blossom Network Upgrade: Blossom specific tasks NU-3 Heartwood Network Upgrade: Heartwood specific tasks NU-4 Canopy Network Upgrade: Canopy specific tasks NU-5 Network Upgrade: NU5 specific tasks
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants