-
Notifications
You must be signed in to change notification settings - Fork 485
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
[whitelist] metrics improvement #2517
[whitelist] metrics improvement #2517
Conversation
…e check node debug we can make a pointer to node containing whitelist
…de we already know about
… whitelisted by ips/cidrs
alternative solution
…init looking for feedback
@LaurenceJJones: There are no 'kind' label on this PR. You need a 'kind' label to generate the release automatically.
DetailsI am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the BirthdayResearch/oss-governance-bot repository. |
@LaurenceJJones: There are no area labels on this PR. You can add as many areas as you see fit.
DetailsI am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the BirthdayResearch/oss-governance-bot repository. |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #2517 +/- ##
==========================================
- Coverage 57.16% 56.97% -0.20%
==========================================
Files 190 187 -3
Lines 26156 25845 -311
==========================================
- Hits 14953 14726 -227
+ Misses 9633 9573 -60
+ Partials 1570 1546 -24
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
/kind enhancement |
Hi @LaurenceJJones what's the state on this? |
Closed in favor of #2813 |
Fix #2462
Based on #2422
We split out whitelist metrics from parsers, and provide detail breakdown of each expression.
Mostly looking for feedback as I know on the metrics side, we still need to merge the above before moving forward with this one.
Example of metrics;
Currently I am using the actual index but for users we may just want to increase the digit +1 so they know if was the first expression that passed for example.