-
Notifications
You must be signed in to change notification settings - Fork 492
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
Reduce various info logs to debug #2700
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2700 +/- ##
=====================================
Coverage 79.3% 79.3%
=====================================
Files 123 123
Lines 22657 22657
=====================================
+ Hits 17970 17974 +4
+ Misses 4687 4683 -4 ☔ View full report in Codecov by Sentry. |
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.
Looks good except one place : https://github.com/open-telemetry/opentelemetry-rust/pull/2700/files#r1966362700
Once this is resolved, we are good to merge.
(Also, there is merge conflict due to changelog. Please resolve that too to merge)
2d33cd0
to
616b2d0
Compare
Fixes #
Design discussion issue (if applicable) #2697
Changes
Reduce various recently introduced info level logs to debug.
Many of these are lifecycle related and unlikely to be interesting to most users.
Merge requirement checklist
CHANGELOG.md
files updated for non-trivial, user-facing changes