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

Return noop meters once the provider has been shutdown #4154

Merged
merged 8 commits into from
Jun 5, 2023
Merged

Return noop meters once the provider has been shutdown #4154

merged 8 commits into from
Jun 5, 2023

Conversation

dmathieu
Copy link
Member

Closes #4148.

@codecov
Copy link

codecov bot commented May 31, 2023

Codecov Report

Merging #4154 (d68b67b) into main (b4faa3d) will decrease coverage by 0.1%.
The diff coverage is 100.0%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #4154     +/-   ##
=======================================
- Coverage   83.4%   83.3%   -0.1%     
=======================================
  Files        181     181             
  Lines      13928   13940     +12     
=======================================
+ Hits       11617   11625      +8     
- Misses      2090    2094      +4     
  Partials     221     221             
Impacted Files Coverage Δ
sdk/metric/provider.go 100.0% <100.0%> (ø)

... and 2 files with indirect coverage changes

@MrAlias MrAlias added pkg:SDK Related to an SDK package area:metrics Part of OpenTelemetry Metrics labels May 31, 2023
dmathieu and others added 5 commits June 1, 2023 10:02

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
@pellared pellared requested a review from pmcollins June 1, 2023 08:14
@pellared pellared merged commit be82610 into open-telemetry:main Jun 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:metrics Part of OpenTelemetry Metrics pkg:SDK Related to an SDK package
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Return a NoOp Meter after MeterProvider is shut down
5 participants