Skip to content

Convert declarative config env var subtitution syntax to ABNF#4448

Merged
jack-berg merged 5 commits intoopen-telemetry:mainfrom
jack-berg:env-var-substitution-abnf
Mar 21, 2025
Merged

Convert declarative config env var subtitution syntax to ABNF#4448
jack-berg merged 5 commits intoopen-telemetry:mainfrom
jack-berg:env-var-substitution-abnf

Conversation

@jack-berg
Copy link
Member

Fixes #4027.

@open-telemetry/configuration-maintainers please take a look!

@jack-berg jack-berg requested review from a team March 12, 2025 16:55
@github-actions
Copy link

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Mar 20, 2025
@MrAlias MrAlias removed the Stale label Mar 20, 2025
Copy link
Member

@reyang reyang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM assuming a changelog entry will be added.

Copy link
Member

@marcalff marcalff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

See comments for a typo.

@jack-berg jack-berg merged commit c05fb1b into open-telemetry:main Mar 21, 2025
6 checks passed
@carlosalberto carlosalberto mentioned this pull request Apr 8, 2025
carlosalberto added a commit that referenced this pull request Apr 15, 2025
### Context

- Add context propagation through Environment Variables specification.

([#4454](#4454))
- On Propagators API, stabilize `GetAll` on the `TextMap` Extract.

([#4472](#4472))

### Traces

- Define sampling threshold field in OpenTelemetry TraceState; define
the behavior
of TraceIdRatioBased sampler in terms of W3C Trace Context Level 2
randomness.

([#4166](#4166))

### Metrics

- Clarify SDK behavior for Instrument Advisory Parameter.

([#4389](#4389))

### Logs

- Add `Enabled` opt-in operation to the `LogRecordProcessor`.

([#4439](#4439))
- Stabilize `Logger.Enabled`.

([#4463](#4463))
- Stabilize `EventName`.

([#4475](#4475))

### Baggage

- Add context (baggage) propagation through Environment Variables
specification.

([#4454](#4454))

### Resource

- Add Datamodel for Entities.

([#4442](#4442))

### SDK Configuration

- Convert declarative config env var substitution syntax to ABNF.

([#4448](#4448))
- List declarative config supported SDK extension plugin interfaces.

([#4452](#4452))

---------

Co-authored-by: Armin Ruech <7052238+arminru@users.noreply.github.com>
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.

Use ABNF syntax instead of PCRE2 in file configuration env var substitution syntax

4 participants