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

Network: Penalize peer/class rankings when peer does not have catchpoint #5959

Merged
merged 1 commit into from Mar 15, 2024

Conversation

gmalouf
Copy link
Contributor

@gmalouf gmalouf commented Mar 15, 2024

Summary

Penalize peer/class rankings during catchpoint catchup initialization when they do not have a catchpoint. This is critical to ensure that the class based peer selector falls back to the next class of peers when too many peers in the first priority class do not have a catchpoint (in turn to prevent blocking the node from doing a fast catchup).

Test Plan

Reproduced catchup against older catchpoints on Betanet getting failing locally, then working properly once the ranking was included. Existing automated tests should all pass.

@gmalouf gmalouf self-assigned this Mar 15, 2024
@gmalouf gmalouf marked this pull request as ready for review March 15, 2024 23:05
Copy link

codecov bot commented Mar 15, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 55.73%. Comparing base (dd22cff) to head (b717438).

Files Patch % Lines
catchup/catchpointService.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5959      +/-   ##
==========================================
+ Coverage   55.71%   55.73%   +0.02%     
==========================================
  Files         489      489              
  Lines       68111    68112       +1     
==========================================
+ Hits        37946    37963      +17     
+ Misses      27590    27573      -17     
- Partials     2575     2576       +1     

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

@gmalouf gmalouf merged commit 98ac36a into algorand:master Mar 15, 2024
20 checks passed
@gmalouf gmalouf deleted the downrank-nodes-no-catchpoint branch March 15, 2024 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants