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

Add banking metrics for buffered and dropped packets #19902

Merged
merged 1 commit into from
Sep 15, 2021

Conversation

jstarry
Copy link
Member

@jstarry jstarry commented Sep 15, 2021

Problem

Buffered packet metric and dropped packet metrics only report the number of "packet batches" and it's not possible to know how many packets are in each batch (could be at most 128)

Summary of Changes

Add extra metrics for the actual number of packets dropped and currently buffered

Fixes #

@codecov
Copy link

codecov bot commented Sep 15, 2021

Codecov Report

Merging #19902 (1ed53fb) into master (691bea8) will decrease coverage by 0.0%.
The diff coverage is 100.0%.

@@            Coverage Diff            @@
##           master   #19902     +/-   ##
=========================================
- Coverage    82.6%    82.6%   -0.1%     
=========================================
  Files         478      478             
  Lines      133443   133460     +17     
=========================================
- Hits       110334   110325      -9     
- Misses      23109    23135     +26     

@jstarry jstarry requested a review from sakridge September 15, 2021 16:21
@jstarry jstarry merged commit ca3f147 into solana-labs:master Sep 15, 2021
@jstarry jstarry deleted the more-banking-metrics branch September 15, 2021 20:53
mergify bot pushed a commit that referenced this pull request Sep 15, 2021
mergify bot pushed a commit that referenced this pull request Sep 15, 2021
mergify bot added a commit that referenced this pull request Sep 16, 2021
mergify bot added a commit that referenced this pull request Sep 16, 2021
dankelleher pushed a commit to identity-com/solana that referenced this pull request Nov 24, 2021
frits-metalogix added a commit to identity-com/solana that referenced this pull request Nov 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant