Skip to content

Conversation

@codeboten
Copy link
Contributor

This adds the internal functions to configure meter provider. No public API yet, part of splitting #8026

Most of this code is copied from v0.3.0, updated to support v1.0.0, and also updated to return more meaningfully typed errors.

Follow up to #8089

@codecov
Copy link

codecov bot commented Oct 29, 2025

Codecov Report

❌ Patch coverage is 87.35294% with 43 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.8%. Comparing base (f5ba341) to head (420cdab).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
otelconf/metric.go 87.3% 32 Missing and 11 partials ⚠️
Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #8094     +/-   ##
=======================================
+ Coverage   80.7%   80.8%   +0.1%     
=======================================
  Files        193     194      +1     
  Lines      12625   12965    +340     
=======================================
+ Hits       10189   10486    +297     
- Misses      2073    2105     +32     
- Partials     363     374     +11     
Files with missing lines Coverage Δ
otelconf/config_common.go 85.7% <ø> (ø)
otelconf/metric.go 87.3% <87.3%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

This adds the internal functions to configure meter provider. No public API yet, part os splitting open-telemetry#8026

Signed-off-by: alex boten <[email protected]>
@codeboten codeboten force-pushed the codeboten/oconf-mprovider branch from 2f6e704 to dcb7aa9 Compare October 30, 2025 20:13
@codeboten codeboten marked this pull request as ready for review October 30, 2025 20:13
@codeboten codeboten requested review from a team and pellared as code owners October 30, 2025 20:13
Signed-off-by: alex boten <[email protected]>
@dmathieu dmathieu added the Skip Changelog Allow PR to succeed without requiring an addition to the CHANGELOG label Oct 31, 2025
@MrAlias MrAlias merged commit 7200b09 into open-telemetry:main Oct 31, 2025
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog Allow PR to succeed without requiring an addition to the CHANGELOG

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants