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

modified otel-http-server-duration from microseconds to milliseconds #970

Merged
merged 1 commit into from
Nov 3, 2021

Conversation

rltoSD
Copy link

@rltoSD rltoSD commented Nov 3, 2021

Description:
This PR is to modify otel http server duration from microseconds to milliseconds as per the specs mentioned in OpenTelemetry Go.

Link to tracking Issue:
issue # 1335

@codecov-commenter
Copy link

Codecov Report

Merging #970 (b23dca3) into main (6c86403) will not change coverage.
The diff coverage is 100.0%.

Impacted file tree graph

@@          Coverage Diff          @@
##            main    #970   +/-   ##
=====================================
  Coverage   69.3%   69.3%           
=====================================
  Files        127     127           
  Lines       5496    5496           
=====================================
  Hits        3810    3810           
  Misses      1538    1538           
  Partials     148     148           
Impacted Files Coverage Δ
instrumentation/net/http/otelhttp/handler.go 78.1% <100.0%> (ø)

@rltoSD rltoSD merged commit 6c76dcd into main Nov 3, 2021
@rltoSD
Copy link
Author

rltoSD commented Dec 21, 2021

please refer to upstream PR # 1537

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