Skip to content

[CONFIGURATION] Implement declarative configuration (config.yaml) #2518

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 243 commits into
base: main
Choose a base branch
from
Open

[CONFIGURATION] Implement declarative configuration (config.yaml) #2518

wants to merge 243 commits into from

Conversation

marcalff
Copy link
Member

@marcalff marcalff commented Jan 31, 2024 •
edited
Loading

Fixes #2481

Start with examples/yaml/main.cc

Features implemented:

  • https://github.com/open-telemetry/opentelemetry-configuration
    • Full support for propagators
    • Full support for traces, including recursive sampler definitions.
  • Extension points
    • Custom propagators
    • Custom samplers
    • Custom span processors
    • Custom span exporters
    • Custom pull metric exporters
    • Custom push metric exporters
    • Custom log record processors
    • Custom log record exporters

Changes

Please provide a brief description of the changes here.

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 January 31, 2024 00:19
@marcalff marcalff marked this pull request as draft January 31, 2024 00:28
@marcalff marcalff changed the title [IGNORE ME] Poc config yaml [RFC] Poc config yaml Feb 20, 2024
marcalff added a commit that referenced this pull request Jul 27, 2025
@marcalff
Refresh code from PR #2518, to fix review comments.
0b15f90
marcalff added a commit that referenced this pull request Jul 27, 2025
@marcalff
Refresh code from PR #2518, to fix review comments.
8dc2b28
marcalff added a commit that referenced this pull request Jul 27, 2025
@marcalff
Refresh code from PR #2518, to fix review comments.
110159b
marcalff added a commit that referenced this pull request Jul 27, 2025
@marcalff
Refresh code from PR #2518, to fix review comments.
bce4aab
marcalff added a commit that referenced this pull request Aug 4, 2025
@marcalff
Refresh code from PR #2518, to fix review comments.
b3e9155
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dbarker dbarker dbarker left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@esigo esigo

Labels
pr:do-not-merge This PR is not ready to be merged. pr:please-review This PR is ready for review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[CONFIG] Implement file configuration
4 participants
@marcalff @dbarker @esigo @ThomsonTan