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

Setup the correct meter provider if telemetry.useOtelForInternalMetrics featuregate enabled #5146

Merged
merged 1 commit into from
Apr 4, 2022

Conversation

bogdandrutu
Copy link
Member

This PR also removes the set of the global MeterProvider, similarly with #5138.

Signed-off-by: Bogdan Drutu [email protected]

@bogdandrutu bogdandrutu requested review from a team and mx-psi April 4, 2022 18:38
@codecov
Copy link

codecov bot commented Apr 4, 2022

Codecov Report

Merging #5146 (ff3305f) into main (1c3be5d) will decrease coverage by 0.02%.
The diff coverage is 70.00%.

@@            Coverage Diff             @@
##             main    #5146      +/-   ##
==========================================
- Coverage   90.33%   90.30%   -0.03%     
==========================================
  Files         182      182              
  Lines       11032    11031       -1     
==========================================
- Hits         9966     9962       -4     
- Misses        841      843       +2     
- Partials      225      226       +1     
Impacted Files Coverage Δ
service/collector.go 77.01% <57.14%> (+0.47%) ⬆️
service/telemetry.go 76.25% <100.00%> (ø)
model/internal/pdata/common.go 93.54% <0.00%> (-0.72%) ⬇️

Continue to review full report at Codecov.

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

…cs featuregate enabled

This PR also removes the set of the global MeterProvider, similarly with open-telemetry#5138.

Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu bogdandrutu merged commit caa4f27 into open-telemetry:main Apr 4, 2022
@bogdandrutu bogdandrutu deleted the fixmetrics branch April 4, 2022 21:41
Nicholaswang pushed a commit to Nicholaswang/opentelemetry-collector that referenced this pull request Jun 7, 2022
…cs featuregate enabled (open-telemetry#5146)

This PR also removes the set of the global MeterProvider, similarly with open-telemetry#5138.

Signed-off-by: Bogdan Drutu <[email protected]>
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