opentelemetry-collector icon indicating copy to clipboard operation
opentelemetry-collector copied to clipboard

Use full TracerProvider config from otelconf

Open jade-guiton-dd opened this issue 5 months ago • 1 comments
trafficstars

Description

This PR changes the config struct corresponding to service::telemetry::traces to embed the full TracerProvider config struct from otelconf, instead of including only its Processors field.

Something similar was previously done for the MeterProvider struct in #12433.

This has the effect of letting users enable sampling on internal Collector traces using the sampler section of the declarative config.

Link to tracking issue

I couldn't find any relevant issue.

Testing

I added a basic example of using the sampler to service/telemetry/internal/migration/testdata/v0.3.0_traces.yaml to test that we can unmarshal this section properly.

Documentation

Just a changelog.

jade-guiton-dd avatar Jun 12 '25 16:06 jade-guiton-dd

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.27%. Comparing base (4263e8d) to head (8eda60d). Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #13201      +/-   ##
==========================================
- Coverage   91.27%   91.27%   -0.01%     
==========================================
  Files         514      514              
  Lines       28857    28855       -2     
==========================================
- Hits        26340    26338       -2     
  Misses       1998     1998              
  Partials      519      519              

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

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

codecov[bot] avatar Jun 12 '25 17:06 codecov[bot]