Skip to content

[CONFIGURATION] Upgrade to configuration model 1.2.0 - #4734

Open
marcalff wants to merge 5 commits into
open-telemetry:mainfrom
marcalff:fix_config_yaml_1.2.0
Open

marcalff wants to merge 5 commits into
open-telemetry:mainfrom
marcalff:fix_config_yaml_1.2.0

Conversation

@marcalff

@marcalff marcalff commented Oct 25, 2026 •

Copy link
Copy Markdown
Member

Fixes # (issue)

Changes

Please provide a brief description of the changes here.

  • Upgrade the yaml configuration to use schema 1.2.0
    • Add attribute_value_depth_limit to attribute, span, and log record limits
    • Add experimental meter_provider.view_matching_mode/development property
    • Add attribute_value_depth_limit to attribute, span, and log record limits
    • Add max_request_size and max_response_size to OTLP exporters

References:

For significant contributions please make sure you have completed the following items:

  • CHANGELOG.md updated for non-trivial changes
  • Unit tests have been added
  • Changes in public API reviewed

@marcalff
marcalff requested a review from a team as a code owner October 25, 2026 12:23
@codecov

codecov Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 59.25926% with 44 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.57%. Comparing base (e854d29) to head (d187245).

Files with missing lines Patch % Lines
sdk/src/configuration/configuration_parser.cc 59.16% 29 Missing ⚠️
sdk/src/configuration/trace_builder_utils.cc 0.00% 10 Missing ⚠️
sdk/src/configuration/trace_builders.cc 70.00% 3 Missing ⚠️
...include/opentelemetry/sdk/configuration/registry.h 0.00% 2 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #4734      +/-   ##
==========================================
- Coverage   86.69%   86.57%   -0.11%     
==========================================
  Files         527      529       +2     
  Lines       20902    20996      +94     
==========================================
+ Hits        18119    18176      +57     
- Misses       2783     2820      +37     
Files with missing lines Coverage Δ
...ude/opentelemetry/exporters/otlp/otlp_recordable.h 100.00% <ø> (ø)
exporters/otlp/src/otlp_recordable.cc 91.41% <100.00%> (+0.07%) ⬆️
.../sdk/configuration/always_record_sampler_builder.h 100.00% <100.00%> (ø)
...onfiguration/always_record_sampler_configuration.h 100.00% <100.00%> (ø)
...tion/otlp_grpc_log_record_exporter_configuration.h 100.00% <ø> (ø)
...ion/otlp_grpc_push_metric_exporter_configuration.h 100.00% <ø> (ø)
...figuration/otlp_grpc_span_exporter_configuration.h 100.00% <ø> (ø)
...tion/otlp_http_log_record_exporter_configuration.h 100.00% <ø> (ø)
...ion/otlp_http_push_metric_exporter_configuration.h 100.00% <ø> (ø)
...figuration/otlp_http_span_exporter_configuration.h 100.00% <ø> (ø)
... and 11 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@marcalff marcalff added the pr:please-review This PR is ready for review label Oct 25, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:please-review This PR is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants