-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Remove report per cpu config option #1326
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1326 +/- ##
==========================================
+ Coverage 89.60% 89.61% +0.01%
==========================================
Files 215 215
Lines 15185 15179 -6
==========================================
- Hits 13606 13603 -3
+ Misses 1151 1148 -3
Partials 428 428
Continue to review full report at Codecov.
|
Not too much coverage improvements :) looks like testutils is going down. I think AssertDoubleMetricLabelDoesNotExist is not used anywhere now. |
Removed unused functions |
Remove cpu scraper
report_per_cpu
option (this can be removed in metric transform processor instead). Also update tests & improve test coverage