-
-
Notifications
You must be signed in to change notification settings - Fork 299
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 outOfRangeData when range sync Deneb #7249
Conversation
…_of_range_blob_block
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## unstable #7249 +/- ##
============================================
- Coverage 48.51% 48.51% -0.01%
============================================
Files 600 600
Lines 40121 40122 +1
Branches 2054 2054
============================================
Hits 19466 19466
- Misses 20617 20618 +1
Partials 38 38 |
not a complete fix, got
the reason is all
|
Performance Report✔️ no performance regression detected Full benchmark results
|
was able to sync with this branch from genesis (in progress, not finish yet), the error is gone
|
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.
Looks good to me, without these changing we can't sync a node post-deneb if blobs are expired already without using checkpoint sync.
The sepolia node is syncing post-deneb without errors while on unstable branch it was just stuck on deneb fork transition.
info: Syncing - 4.2 days left - 5.77 slots/s - slot: 6413947 - head: 4308799 0x1ff2…e8c9 - exec-block: valid(5248314 0xe48b…) - finalized: 0xf150…04d4:134648 - peers: 15
info: Syncing - 4.3 days left - 5.65 slots/s - slot: 6413948 - head: 4308831 0xaf60…7e6b - exec-block: valid(5248342 0xa31d…) - finalized: 0xac66…0bf3:134649 - peers: 16
info: Syncing - 4.6 days left - 5.30 slots/s - slot: 6413949 - head: 4308863 0x42c8…7d97 - exec-block: valid(5248374 0x6b22…) - finalized: 0xfb0a…8d20:134650 - peers: 16
The Error processing finalized checkpoint
error is no longer happening.
Motivation
Cannot sync lodestar from mekong genesis due to this error
Cannot sync blobs outside of blobs prune window
Description
Beacon node should return
BlockInputType.outOfRangeData
block input instead of throwing that errorCloses #7248