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

v1.17 - Add detailed metrics reporting for packet filtering #1422

Merged
merged 2 commits into from
May 18, 2024

Conversation

jstarry
Copy link

@jstarry jstarry commented May 17, 2024

Problem

We don't report metrics on how many packets are filtered out in banking stage for various reasons like sanitization failures, invalid compute, etc.

Summary of Changes

Report more metrics for different types of filtered packets

Manual backport of #1421

@jstarry jstarry requested a review from a team as a code owner May 17, 2024 22:51
@jstarry jstarry requested a review from apfitzge May 17, 2024 22:51
@jstarry jstarry force-pushed the v1.17-packet-filter-metrics branch from 6a0f612 to c3c46da Compare May 17, 2024 22:57
Copy link

@apfitzge apfitzge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still lgtm - conflict resolution is simple. Useful to have the stats in v17

@codecov-commenter
Copy link

Codecov Report

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

Project coverage is 81.6%. Comparing base (e9a0a1c) to head (c3c46da).

Additional details and impacted files
@@           Coverage Diff           @@
##            v1.17    #1422   +/-   ##
=======================================
  Coverage    81.6%    81.6%           
=======================================
  Files         807      808    +1     
  Lines      219370   219433   +63     
=======================================
+ Hits       179097   179175   +78     
+ Misses      40273    40258   -15     

@jstarry jstarry merged commit f96f10c into anza-xyz:v1.17 May 18, 2024
33 checks passed
@jstarry jstarry deleted the v1.17-packet-filter-metrics branch May 18, 2024 00:14
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.

4 participants