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

app/eth2wrap: add CLI support for fallback beacon node #3459

Merged
merged 9 commits into from
Jan 22, 2025

Conversation

DiogoSantoss
Copy link
Contributor

Add --fallback-beacon-node-endpoints CLI flag to specify a list of beacon nodes to use if the primary list (specified using --beacon-node-endpoints) fails.

This is the continuation of a previous PR (#3342) which added the fallback logic.

category: feature
ticket: #3328

@DiogoSantoss DiogoSantoss requested review from gsora, pinebit and KaloyanTanev and removed request for gsora and pinebit January 14, 2025 12:52
Copy link

codecov bot commented Jan 14, 2025

Codecov Report

Attention: Patch coverage is 52.76074% with 77 lines in your changes missing coverage. Please review.

Project coverage is 57.40%. Comparing base (6dacdd3) to head (65a2d7f).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
app/eth2wrap/eth2wrap_gen.go 24.24% 50 Missing ⚠️
app/app.go 0.00% 16 Missing ⚠️
app/eth2wrap/eth2wrap.go 83.33% 7 Missing and 1 partial ⚠️
cmd/exit.go 66.66% 2 Missing ⚠️
app/eth2wrap/multi.go 95.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3459      +/-   ##
==========================================
+ Coverage   57.05%   57.40%   +0.35%     
==========================================
  Files         218      217       -1     
  Lines       32634    32249     -385     
==========================================
- Hits        18618    18513     -105     
+ Misses      12105    11824     -281     
- Partials     1911     1912       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@DiogoSantoss DiogoSantoss force-pushed the diogo/fallback-beacon-node branch from 3f8d183 to fe366ec Compare January 20, 2025 16:40
@DiogoSantoss DiogoSantoss force-pushed the diogo/fallback-beacon-node branch from ebd1e42 to f780477 Compare January 21, 2025 17:00
app/eth2wrap/eth2wrap_test.go Outdated Show resolved Hide resolved
@DiogoSantoss DiogoSantoss added the merge when ready Indicates bulldozer bot may merge when all checks pass label Jan 22, 2025
@obol-bulldozer obol-bulldozer bot merged commit 06233ce into main Jan 22, 2025
13 checks passed
@obol-bulldozer obol-bulldozer bot deleted the diogo/fallback-beacon-node branch January 22, 2025 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge when ready Indicates bulldozer bot may merge when all checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants