You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
check that we are at least 200* blocks above the highest checkpoint (detects stalls)
* I picked 200 because 100 non-finalized blocks are excluded from checkpoint generation, and it takes 2 hours to build and run the test. (If mainnet syncs get fast, or we're using testnet.) If we make it any higher, the first time we submit each checkpoint update PR the tests will be slow, because we'll wait to sync newly generated blocks.
Useful to increase coverage:
wait until the mempool has been active for a few minutes before exiting (increases coverage)
Optional:
Only if needed due to CI failures:
Adjust the timeout so that at least 99% of syncs succeed (makes failing tests fail faster)
Here is a refactor I don't think we need to do yet:
Motivation
This is the Rust part of:
Suggested Design
Here's one way to do the Rust part of this change:
Partly done in #3495:
zebrad start
's main loop as a "continuing task", so Zebra exits when it finishesNew work:
3. Create a sync test case with a 6 hour timeout, which uses a "stop at estimated height" config
zebra/zebrad/src/commands/start.rs
Lines 195 to 200 in 1a14baf
Originally posted by @teor2345 in #1592 (comment)
Follow-Up Tasks
Required:
* I picked 200 because 100 non-finalized blocks are excluded from checkpoint generation, and it takes 2 hours to build and run the test. (If mainnet syncs get fast, or we're using testnet.) If we make it any higher, the first time we submit each checkpoint update PR the tests will be slow, because we'll wait to sync newly generated blocks.
Useful to increase coverage:
Optional:
Only if needed due to CI failures:
Here is a refactor I don't think we need to do yet:
zebrad start
main loop #3442 (simplifies similar test code that's currently in the state)The text was updated successfully, but these errors were encountered: