Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Lower vote-only-mode to 400 slots#22210

Merged
sakridge merged 1 commit intosolana-labs:masterfrom
sakridge:vote-only-mode
Jan 4, 2022
Merged

Lower vote-only-mode to 400 slots#22210
sakridge merged 1 commit intosolana-labs:masterfrom
sakridge:vote-only-mode

Conversation

@sakridge
Copy link
Copy Markdown
Contributor

Problem

500 slots is a bit close and allows for 800-1000 slot root distance

Summary of Changes

Reduce to 400 slot height difference to trigger vote-mode to keep the cluster in check.

Fixes #

@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 31, 2021

Codecov Report

Merging #22210 (022eeb1) into master (0b1b36f) will increase coverage by 0.0%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #22210   +/-   ##
=======================================
  Coverage    81.0%    81.0%           
=======================================
  Files         523      523           
  Lines      146497   146497           
=======================================
+ Hits       118739   118748    +9     
+ Misses      27758    27749    -9     

@sakridge sakridge requested a review from t-nelson January 1, 2022 08:16
@sakridge sakridge merged commit 2486e21 into solana-labs:master Jan 4, 2022
@sakridge sakridge deleted the vote-only-mode branch January 4, 2022 11:49
mergify Bot pushed a commit that referenced this pull request Jan 4, 2022
(cherry picked from commit 2486e21)
mergify Bot pushed a commit that referenced this pull request Jan 4, 2022
(cherry picked from commit 2486e21)
mergify Bot added a commit that referenced this pull request Jan 4, 2022
(cherry picked from commit 2486e21)

Co-authored-by: sakridge <sakridge@gmail.com>
mergify Bot added a commit that referenced this pull request Jan 4, 2022
(cherry picked from commit 2486e21)

Co-authored-by: sakridge <sakridge@gmail.com>
@brooksprumo brooksprumo mentioned this pull request Jan 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants