-
-
Notifications
You must be signed in to change notification settings - Fork 354
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: handle getDependentRoot() error when importing block #7552
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## unstable #7552 +/- ##
============================================
- Coverage 50.15% 50.14% -0.02%
============================================
Files 603 603
Lines 40454 40454
Branches 2231 2226 -5
============================================
- Hits 20291 20285 -6
- Misses 20123 20129 +6
Partials 40 40 🚀 New features to boost your workflow:
|
Performance Report✔️ no performance regression detected Full benchmark results
|
When did we get the "No block root" error? Is this when we set the unfinalized state at startup? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - we have tested this fix for a while on holesky-rescue branch, see diff
yes it's from that work https://github.com/ChainSafe/lodestar/pull/7509/files#diff-c21a87c055e2b34c02de2fa28ecc52c53837159f567eef5fb22e80ed55827044R231 but it makes sense from code perspective I guess |
🎉 This PR is included in v1.28.0 🎉 |
Motivation
forkchoice.getDependentRoot()
may failed with "No block root" error and break the block import processDescription
try/catch
, this was picked fromholesky-rescue
branch https://github.com/ChainSafe/lodestar/pull/7501/files#diff-c21a87c055e2b34c02de2fa28ecc52c53837159f567eef5fb22e80ed55827044R229