oss_26_declarative_config_env_substitution - #36
Conversation
|
🔒 Internal (dash0) — not for upstream.
|
|
📣 Public-facing draft — to be used for the upstream PR in Title: Fix declarative config env substitution and top-level fields What
WhySpec-conformance gaps: SpecReference grammar (optional TestsNew env-substitution tests (prefixed form; prefixed + default; prefixed missing raises; default applied when empty; empty-no-default substitutes empty; prefixed empty + default) and LimitationThe SDK |
|
This PR has been automatically marked as stale because it has not had any activity for 14 days. It will be closed if no further activity occurs within 14 days of this comment. |
530cfd6 to
68471f0
Compare
- Support the ${env:VAR} prefixed environment variable reference form.
- Apply ${VAR:-default} defaults when the referenced variable is
set-but-empty, not only when it is unset (per the data-model spec).
- Honor the top-level attribute_limits field in configure_sdk: thread it
into the tracer provider as the default span attribute count/length
limits, overridden by the tracer provider's own limits.
The top-level log_level wiring is now handled upstream, so it is no
longer part of this change.
68471f0 to
71a1d41
Compare
|
🔒 Internal (dash0) — reworked after upstream advanced. Rebased onto current |
Closes #32
Declarative config env substitution + top-level fields (Linear OSS-26, findings L5/L6/L7): support
${env:VAR}; apply:-defaultwhen the var is null/empty/undefined; wire top-levellog_levelandattribute_limitsinto configure_sdk (attribute_limits applied to spans; logs/metrics not programmatically supported → warns).Validation: scope contained to opentelemetry-configuration + changelog; package suite 374 passed.
Linear issue: https://linear.app/dash0/issue/OSS-26/l5-l6-l7-declarative-config-env-substitution-top-level-attribute