Skip to content
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

Merged
merged 1 commit into from
Jul 26, 2024

Conversation

mcculls
Copy link
Contributor

@mcculls mcculls commented Jul 25, 2024

Description

The RFC has been revised since #2691

The expected telemetry tags when reporting OTel env-var mapping issues now use snake-case:

 config.datadog       -> config_datadog
 config.opentelemetry -> config_opentelemetry

The expected tag values should also be lower-case.

Reviewer checklist

  • Test coverage seems ok.
  • Appropriate labels assigned.

@pr-commenter
Copy link

pr-commenter bot commented Jul 25, 2024

Benchmarks

Benchmark execution time: 2024-07-25 11:24:47

Comparing candidate commit fbc56c4 in PR branch mcculls/otel-env-var-metric-tag-update with baseline commit d8d904a in branch master.

Found 3 performance improvements and 0 performance regressions! Performance is the same for 175 metrics, 0 unstable metrics.

scenario:PDOBench/benchPDOBaseline

  • 🟩 execution_time [-18.360µs; -14.809µs] or [-9.956%; -8.030%]

scenario:PDOBench/benchPDOOverhead

  • 🟩 execution_time [-18.132µs; -16.355µs] or [-6.319%; -5.700%]

scenario:PDOBench/benchPDOOverheadWithDBM

  • 🟩 execution_time [-18.263µs; -16.221µs] or [-5.877%; -5.220%]

@codecov-commenter
Copy link

codecov-commenter commented Jul 25, 2024

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 78.51%. Comparing base (d8d904a) to head (fbc56c4).

Additional details and impacted files

Impacted file tree graph

@@             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     
Flag Coverage Δ
appsec-extension 69.19% <ø> (?)
tracer-extension 78.60% <0.00%> (ø)
tracer-php 82.13% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
ext/otel_config.c 79.50% <0.00%> (ø)

... and 26 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d8d904a...fbc56c4. Read the comment docs.

@mcculls mcculls force-pushed the mcculls/otel-env-var-metric-tag-update branch from d290a23 to 08a634c Compare July 25, 2024 10:49
…with the latest RFC

  config.datadog       -> config_datadog
  config.opentelemetry -> config_opentelemetry

The expected tag values should also be lower-case.
@mcculls mcculls force-pushed the mcculls/otel-env-var-metric-tag-update branch from 08a634c to fbc56c4 Compare July 25, 2024 10:56
@mcculls mcculls changed the title Updates the expected telemetry tags for OTel env-var mapping Update the expected telemetry tags for OTel env-var mapping Jul 25, 2024
@mcculls mcculls marked this pull request as ready for review July 26, 2024 10:00
@mcculls mcculls requested a review from a team as a code owner July 26, 2024 10:00
Copy link
Collaborator

@bwoebi bwoebi left a 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 :-)

@mcculls mcculls merged commit eff863f into master Jul 26, 2024
675 checks passed
@mcculls mcculls deleted the mcculls/otel-env-var-metric-tag-update branch July 26, 2024 12:18
@github-actions github-actions bot added this to the 1.3.0 milestone Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants