feat: sync missing el blocks from client db

sapinb commented 4 weeks ago

Description

If reth db is missing blocks expected by the client chainstate, sync the missing blocks.

Type of Change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature/Enhancement (non-breaking change which adds functionality or enhances an existing one)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Refactor
[ ] New or updated tests

Checklist

[ ] I have performed a self-review of my code.
[ ] I have commented my code where necessary.
[ ] I have updated the documentation if needed.
[ ] My changes do not introduce new warnings.
[ ] I have added tests that prove my changes are effective or that my feature works.
[ ] New and existing tests pass with my changes.

Related Issues

storopoli commented 4 weeks ago

Does this also needs to be PR'd to main as well?

Rajil1213 commented 4 weeks ago

Does this also needs to be PR'd to main as well?

That'll have a bunch of conflicts plus some additional changes that we may not want since this branch builds upon other changes in the release branch. I think the plan is to periodically merge some of the changes from release to main. But not sure what that is going to be like.

codecov[bot] commented 4 weeks ago

Codecov Report

Attention: Patch coverage is 49.25373% with 34 lines in your changes missing coverage. Please review.

Project coverage is 57.16%. Comparing base (a14d926) to head (68cdafe). Report is 2 commits behind head on releases/0.1.0.

Files with missing lines	Patch %	Lines
bin/strata-client/src/el_sync.rs	50.76%	32 Missing :warning:
bin/strata-client/src/main.rs	0.00%	2 Missing :warning:

@@                Coverage Diff                 @@
##           releases/0.1.0     #417      +/-   ##
==================================================
- Coverage           57.16%   57.16%   -0.01%     
==================================================
  Files                 256      257       +1     
  Lines               27048    27113      +65     
==================================================
+ Hits                15462    15499      +37     
- Misses              11586    11614      +28

Files with missing lines	Coverage Δ
crates/reth/exex/src/prover_exex.rs	`0.00% <ø> (ø)`
bin/strata-client/src/main.rs	`0.00% <0.00%> (ø)`
bin/strata-client/src/el_sync.rs	`50.76% <50.76%> (ø)`

... and 6 files with indirect coverage changes

alpenlabs / strata