-
Notifications
You must be signed in to change notification settings - Fork 151
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
Update the expected telemetry tags for OTel env-var mapping #2775
Conversation
BenchmarksBenchmark execution time: 2024-07-25 11:24:47 Comparing candidate commit fbc56c4 in PR branch Found 3 performance improvements and 0 performance regressions! Performance is the same for 175 metrics, 0 unstable metrics. scenario:PDOBench/benchPDOBaseline
scenario:PDOBench/benchPDOOverhead
scenario:PDOBench/benchPDOOverheadWithDBM
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2775 +/- ##
============================================
- Coverage 80.09% 78.51% -1.58%
Complexity 2459 2459
============================================
Files 204 230 +26
Lines 23769 27794 +4025
Branches 0 989 +989
============================================
+ Hits 19038 21823 +2785
- Misses 4731 5445 +714
- Partials 0 526 +526
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 26 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
d290a23
to
08a634c
Compare
…with the latest RFC config.datadog -> config_datadog config.opentelemetry -> config_opentelemetry The expected tag values should also be lower-case.
08a634c
to
fbc56c4
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.
Looks simple enough to me, thanks :-)
Description
The RFC has been revised since #2691
The expected telemetry tags when reporting OTel env-var mapping issues now use snake-case:
The expected tag values should also be lower-case.
Reviewer checklist