Skip to content

otelconf: add sampler configuration support#7148

Merged
MrAlias merged 8 commits intoopen-telemetry:mainfrom
codeboten:codeboten/add-sampler
Apr 8, 2025
Merged

otelconf: add sampler configuration support#7148
MrAlias merged 8 commits intoopen-telemetry:mainfrom
codeboten:codeboten/add-sampler

Conversation

@codeboten
Copy link
Contributor

This PR adds parsing and configuration for sampler configuration.

This PR adds parsing and configuration for sampler configuration.

Signed-off-by: Alex Boten <[email protected]>
@codeboten codeboten requested review from a team and pellared as code owners April 3, 2025 22:16
Signed-off-by: Alex Boten <[email protected]>
@codecov
Copy link

codecov bot commented Apr 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.9%. Comparing base (0fea947) to head (ce6b755).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #7148     +/-   ##
=======================================
+ Coverage   75.7%   75.9%   +0.1%     
=======================================
  Files        206     206             
  Lines      19240   19310     +70     
=======================================
+ Hits       14581   14667     +86     
+ Misses      4226    4212     -14     
+ Partials     433     431      -2     
Files with missing lines Coverage Δ
otelconf/v0.3.0/trace.go 99.5% <100.0%> (+2.0%) ⬆️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@codeboten
Copy link
Contributor Author

@open-telemetry/go-approvers Anything I can help with to move this PR forward?

@MrAlias MrAlias merged commit 8422458 into open-telemetry:main Apr 8, 2025
26 checks passed
@MrAlias MrAlias added this to the v1.36.0 milestone Apr 11, 2025
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