-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Deprecate CheckReceiverMetrics functions #12120
base: main
Are you sure you want to change the base?
Deprecate CheckReceiverMetrics functions #12120
Conversation
5583680
to
dbea3be
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #12120 +/- ##
=======================================
Coverage 91.24% 91.24%
=======================================
Files 465 465
Lines 25689 25689
=======================================
Hits 23439 23439
Misses 1837 1837
Partials 413 413 ☔ View full report in Codecov by Sentry. |
ed141cf
to
2af0cce
Compare
2af0cce
to
accc09a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would do this later. I would for the moment move to the exporter metrics which are barely used in contrib.
Description
Deprecating CheckReceiverMetrics functions in the untested
obsreporttest
andotelchecker
go files.cc @bogdandrutu
Testing
n/a
Documentation
n/a