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

Record the current nodes locktower votes from the bank#3502

Merged
aeyakovenko merged 2 commits intosolana-labs:masterfrom
aeyakovenko:observed_locktower_stats
Mar 26, 2019
Merged

Record the current nodes locktower votes from the bank#3502
aeyakovenko merged 2 commits intosolana-labs:masterfrom
aeyakovenko:observed_locktower_stats

Conversation

@aeyakovenko
Copy link
Copy Markdown
Member

Problem

Lack of observed actual data on lockout votes from the bank for the current node.

Summary of Changes

Add counters. These may be out of order, but should trend in the right direction.

Fixes #

@aeyakovenko aeyakovenko requested a review from carllin March 26, 2019 17:27
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 26, 2019

Codecov Report

Merging #3502 into master will decrease coverage by <.1%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master   #3502     +/-   ##
========================================
- Coverage    79.3%   79.3%   -0.1%     
========================================
  Files         151     151             
  Lines       24172   24180      +8     
========================================
+ Hits        19182   19188      +6     
- Misses       4990    4992      +2

@aeyakovenko aeyakovenko merged commit 420cbc4 into solana-labs:master Mar 26, 2019
@aeyakovenko
Copy link
Copy Markdown
Member Author

@pgarg66 can you cherry-pick this to beacons

@pgarg66
Copy link
Copy Markdown
Contributor

pgarg66 commented Mar 26, 2019

will do

pgarg66 referenced this pull request in pgarg66/solana Mar 26, 2019
* observed_locktower_stats

* fixup! observed_locktower_stats
solana-grimes pushed a commit that referenced this pull request Mar 26, 2019
* observed_locktower_stats

* fixup! observed_locktower_stats
joncinque pushed a commit to joncinque/agave that referenced this pull request Nov 7, 2024
…bs#3502)

* Store epoch in MaxAge (solana-labs#3485)

(cherry picked from commit 5a6f518)

# Conflicts:
#	core/src/banking_stage/scheduler_messages.rs
#	core/src/banking_stage/transaction_scheduler/transaction_state.rs
#	core/src/banking_stage/transaction_scheduler/transaction_state_container.rs

* resolve conflicts

---------

Co-authored-by: Andrew Fitzgerald <apfitzge@gmail.com>
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