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
Archival Support: Remove configurable support for catching up from "archivers" #5920
Conversation
… for archiver catchup that utilized it.
…arying settings for the EnableGossipService flag.
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #5920 +/- ##
==========================================
- Coverage 56.02% 55.94% -0.09%
==========================================
Files 478 478
Lines 67613 67561 -52
==========================================
- Hits 37878 37794 -84
- Misses 27176 27203 +27
- Partials 2559 2564 +5 ☔ View full report in Codecov by Sentry. |
…nd falling back/redirecting to them within the block service. Refactored blockservice redirect tests to primarily leverage 'getNextCustomFallbackEndpoint' for retrieving a peer list (empty by default.
Co-authored-by: John Jannotti <jannotti@gmail.com>
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.
I have no objections. The caveat is that I don't really understand the config json stuff.
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, need to ensure (manual or auto test) that peer selector does not stuck when where is no archive DNS.
Summary
This PR removes configurable support for catching up from or (in the case of block serving) falling back to "archivers"; a concept that was never utilized on mainnet nor is currently on testnet. This fits with the larger effort to support first class non-archival relays and dedicated archival nodes for catchup/catchpoint purposes.
A concurrency issue was addressed around read/write access to
blockService.fallbackEndpoints.lastUsed
along the way.Test Plan
No longer relevant test scenarios are removed, a few cases were added in catchup service to reflect needing EnableGossipService set to be a relay, existing tests should otherwise pass.