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. |
- 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 top-level log_level and attribute_limits in configure_sdk:
log_level sets the SDK internal logger level; attribute_limits are
threaded into the tracer provider as the default span attribute
count/length limits.
530cfd6 to
68471f0
Compare
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