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

Remaps http status code tag for otel span for trace metrics #3664

Merged
merged 3 commits into from
May 28, 2024

Conversation

zarirhamza
Copy link
Contributor

Adds http.response.status_code to be remapped to http.status_code for trace metrics for otel spans

@zarirhamza zarirhamza marked this pull request as ready for review May 24, 2024 19:53
@zarirhamza zarirhamza requested a review from a team as a code owner May 24, 2024 19:53
@zarirhamza zarirhamza requested a review from marcotc May 24, 2024 19:54
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.10%. Comparing base (b908262) to head (32eda23).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3664      +/-   ##
==========================================
- Coverage   98.10%   98.10%   -0.01%     
==========================================
  Files        1225     1225              
  Lines       72472    72485      +13     
  Branches     3443     3446       +3     
==========================================
+ Hits        71101    71112      +11     
- Misses       1371     1373       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@zarirhamza zarirhamza merged commit 2aa8bb2 into master May 28, 2024
166 checks passed
@zarirhamza zarirhamza deleted the zarir/otel-stats-remapper branch May 28, 2024 21:12
@github-actions github-actions bot added this to the 2.0.0 milestone May 28, 2024
p-datadog pushed a commit to p-datadog/dd-trace-rb that referenced this pull request Jun 4, 2024
* master: (22 commits)
  Remove experimental warning from OpenTelemetry.md
  Remove latest tag on docker images
  add installer system tests
  Extract trace id format helper method (DataDog#3670)
  Remaps http status code tag for otel span for trace metrics (DataDog#3664)
  Update .github/labeler.yml
  Use 3.20
  rb -> ruby
  Add a note about `use` vs `instrument`
  GitHub: Update labeller bot with new products
  Update lockfiles for release 2.0.0.rc1
  Bump version 2.0.0.beta2 to 2.0.0.rc1
  Add 2.0.0.rc1 to CHANGELOG.md
  chore(lib-injection): update base image to alpine 3.18.6
  Apply suggestions from code review
  Revert "DEBUG-2334 Add Debugger component (DataDog#3640)"
  Implement `latest_major` and `lastest_minor` tag for docker init container (DataDog#3643)
  Build oci package
  Update `rexml`  (DataDog#3656)
  Remove bundler cache in for `setup-ruby` step and pin bundler and rubygem version
  ...
@TonyCTHsu TonyCTHsu mentioned this pull request Jun 6, 2024
p-datadog pushed a commit to p-datadog/dd-trace-rb that referenced this pull request Jun 6, 2024
* master-merge-to-di: (37 commits)
  Revert "Revert "DEBUG-2334 Add Debugger component (DataDog#3640)""
  Update lockfiles for release 2.0.0
  Bump version 2.0.0.rc1 to 2.0.0
  Add 2.0.0 to CHANGELOG.md
  Add missing Apache 2.0 license from gemspec
  Use ffi 1.16.3
  Add platform
  Output gem version
  Output debug logs
  Fix spec to exclude .vscode directory
  Fix docker run command
  Add vscode settings for dockerfile file associations
  Update gem version
  Update gemfiles/*
  Update `ffi` in Gemfile
  Remove experimental warning from OpenTelemetry.md
  Remove latest tag on docker images
  add installer system tests
  Extract trace id format helper method (DataDog#3670)
  Remaps http status code tag for otel span for trace metrics (DataDog#3664)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants