Skip to content

[BUILD] Create the configuration_core library for programmatic config#4311

Open
dbarker wants to merge 5 commits into
open-telemetry:mainfrom
dbarker:cmake_create_configuration_core
Open

[BUILD] Create the configuration_core library for programmatic config#4311
dbarker wants to merge 5 commits into
open-telemetry:mainfrom
dbarker:cmake_create_configuration_core

Conversation

@dbarker

@dbarker dbarker commented Jul 26, 2026

Copy link
Copy Markdown
Member

Contributes to #4134 and follow up from ##3655 (review)

This PR creates a new library (configuration_core) to contain the SDK configuration components that do not require external dependencies. This library is built unconditionally with the SDK and available in the sdk CMake component. The YAML configuration component (and ryml dependency) remain with the configuration library and publicly link to the configuration_core library.

Changes

  • Creates the opentelemetry-cpp::configuration_core CMake target for all configuration components that don't require external dependencies
    • The opentelemetry-cpp::configuration CMake target now just includes the ryml dependent components (ryml document, yaml parser).
  • Creates the configuration_core library for Bazel builds
  • Updates the INSTALL guide to cover these two targets

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

@codecov

codecov Bot commented Jul 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.25%. Comparing base (aa60678) to head (473c3b5).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #4311      +/-   ##
==========================================
+ Coverage   81.24%   81.25%   +0.02%     
==========================================
  Files         446      446              
  Lines       18872    18872              
==========================================
+ Hits        15330    15332       +2     
+ Misses       3542     3540       -2     
Files with missing lines Coverage Δ
sdk/src/configuration/registry.cc 60.18% <ø> (ø)

... 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.

@dbarker
dbarker marked this pull request as ready for review July 26, 2026 19:52
@dbarker
dbarker requested a review from a team as a code owner July 26, 2026 19:52
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.

1 participant