feat(config): Add LoggerProvider support for declarative config by MikeGoldsmith · Pull Request #4990 · open-telemetry/opentelemetry-python

MikeGoldsmith · 2026-03-17T14:42:46Z

Description

Implements `create_logger_provider()` and `configure_logger_provider()` for the declarative configuration pipeline, as part of the ongoing work tracked in the following issue:

Add declarative configuration prototype #3631

Note

This PR is based on #4979 (Resource & Propagator creation) which must be merged first. The extra commits at the base of this branch will be dropped once that PR lands.

What's included

`_logger_provider.py`: creates an SDK `LoggerProvider` from declarative config
Supports `BatchLogRecordProcessor` with schedule_delay/export_timeout/max_queue_size/max_export_batch_size config
Supports `SimpleLogRecordProcessor`
Supports OTLP HTTP, OTLP gRPC, and Console log exporters
Lazy imports for optional OTLP packages with a `ConfigurationError` if not installed
`configure_logger_provider(None)` is a no-op per spec/Java/JS behavior
`otlp_file_development` raises `ConfigurationError` (experimental; not yet supported)
`log_record_limits` logs a warning if specified — Python SDK `LoggerProvider` constructor does not accept limits
28 new tests covering all paths including env-var suppression

Env-var suppression

Python's `BatchLogRecordProcessor` reads `OTEL_BLRP_*` env vars when params are `None`. To match the spec's "what you see is what you get" semantics, we always pass explicit values — defaulting to spec defaults. Notably, the Python SDK's default for `OTEL_BLRP_SCHEDULE_DELAY` is 5000ms (incorrect per spec); we pass 1000ms per the OTel spec.

Type of change

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update

How Has This Been Tested?

28 unit tests in `tests/_configuration/test_logger_provider.py`
Includes explicit test that `OTEL_BLRP_SCHEDULE_DELAY` env var is not read when config is used

Does This PR Require a Contrib Repo Change?

No.

Checklist

My code follows the style guidelines of this project
I have performed a self-review of my own code
I have commented my code, particularly in hard-to-understand areas
My changes generate no new warnings
I have added tests that prove my fix is effective or that my feature works
New and existing unit tests pass locally with my changes

Implements create_resource() and create_propagator()/configure_propagator() for the declarative file configuration. Resource creation does not read OTEL_RESOURCE_ATTRIBUTES or run any detectors (matches Java/JS SDK behavior). Propagator configuration always calls set_global_textmap to override Python's default tracecontext+baggage, setting a noop CompositePropagator when no propagator is configured. Assisted-by: Claude Sonnet 4.6

Assisted-by: Claude Sonnet 4.6

- _resource.py: refactor _coerce_attribute_value to dispatch table to avoid too-many-return-statements; fix short variable names k/v -> attr_key/attr_val; fix return type of _sdk_default_attributes to dict[str, str] to satisfy pyright - _propagator.py: rename short variable names e -> exc, p -> propagator - test_resource.py: move imports to top level; split TestCreateResource (25 methods) into three focused classes to satisfy too-many-public-methods - test_propagator.py: add pylint disable for protected-access Assisted-by: Claude Sonnet 4.6

- replace _sdk_default_attributes() with _DEFAULT_RESOURCE from resources module - move _coerce_bool into dispatch tables for both scalar and array bool types, fixing a bug where bool_array with string values like "false" would coerce incorrectly via plain bool() (non-empty string -> True) - add test for bool_array with string values to cover the bug Assisted-by: Claude Sonnet 4.6

… into mike/config-resource-propagator

…erge - collapse _SCALAR_COERCIONS and _ARRAY_COERCIONS into a single _COERCIONS dict using an _array() factory, reducing _coerce_attribute_value to two lines - process attributes_list before attributes so explicit attributes naturally overwrite list entries without needing an explicit guard Assisted-by: Claude Sonnet 4.6

…metry#4990) Assisted-by: Claude Sonnet 4.6

Assisted-by: Claude Sonnet 4.6

…e-propagator

Adds _run_detectors() stub and _filter_attributes() to create_resource(), providing the shared scaffolding for detector PRs to build on. Detectors are opt-in: nothing runs unless explicitly listed under detection_development.detectors in the config. The include/exclude attribute filter mirrors other SDK behaviour. Assisted-by: Claude Sonnet 4.6

Merges service.name=unknown_service into base before running detectors, so detectors (e.g. service) can override it. Previously it was added to config_attrs and merged last, which would have silently overridden any detector-provided service.name. Assisted-by: Claude Sonnet 4.6

…metry#4990) Assisted-by: Claude Sonnet 4.6

Assisted-by: Claude Sonnet 4.6

…config Implements LoggerProvider instantiation from declarative config files, following the same env-var-suppression pattern as create_meter_provider. BatchLogRecordProcessor defaults use spec values (1000ms schedule_delay), overriding the Python SDK's incorrect 5000ms env-var default. Assisted-by: Claude Sonnet 4.6

…metry#4990) Assisted-by: Claude Sonnet 4.6

…ropagator Assisted-by: Claude Sonnet 4.6

Import directly from _exceptions.py since this is new code with no existing dependents on the _loader module path. Assisted-by: Claude Sonnet 4.6

Assisted-by: Claude Sonnet 4.6

…-telemetry#4990) * config: add resource and propagator creation from declarative config Implements create_resource() and create_propagator()/configure_propagator() for the declarative file configuration. Resource creation does not read OTEL_RESOURCE_ATTRIBUTES or run any detectors (matches Java/JS SDK behavior). Propagator configuration always calls set_global_textmap to override Python's default tracecontext+baggage, setting a noop CompositePropagator when no propagator is configured. Assisted-by: Claude Sonnet 4.6 * update changelog with PR number Assisted-by: Claude Sonnet 4.6 * fix pylint, pyright and ruff errors in resource/propagator config - _resource.py: refactor _coerce_attribute_value to dispatch table to avoid too-many-return-statements; fix short variable names k/v -> attr_key/attr_val; fix return type of _sdk_default_attributes to dict[str, str] to satisfy pyright - _propagator.py: rename short variable names e -> exc, p -> propagator - test_resource.py: move imports to top level; split TestCreateResource (25 methods) into three focused classes to satisfy too-many-public-methods - test_propagator.py: add pylint disable for protected-access Assisted-by: Claude Sonnet 4.6 * address review feedback: use _DEFAULT_RESOURCE, fix bool_array coercion - replace _sdk_default_attributes() with _DEFAULT_RESOURCE from resources module - move _coerce_bool into dispatch tables for both scalar and array bool types, fixing a bug where bool_array with string values like "false" would coerce incorrectly via plain bool() (non-empty string -> True) - add test for bool_array with string values to cover the bug Assisted-by: Claude Sonnet 4.6 * fix linter * address review feedback: single coercion table, simplify attributes merge - collapse _SCALAR_COERCIONS and _ARRAY_COERCIONS into a single _COERCIONS dict using an _array() factory, reducing _coerce_attribute_value to two lines - process attributes_list before attributes so explicit attributes naturally overwrite list entries without needing an explicit guard Assisted-by: Claude Sonnet 4.6 * use Callable type annotation on _array helper Assisted-by: Claude Sonnet 4.6 * add detection infrastructure foundations for resource detectors Adds _run_detectors() stub and _filter_attributes() to create_resource(), providing the shared scaffolding for detector PRs to build on. Detectors are opt-in: nothing runs unless explicitly listed under detection_development.detectors in the config. The include/exclude attribute filter mirrors other SDK behaviour. Assisted-by: Claude Sonnet 4.6 * move service.name default into base resource Merges service.name=unknown_service into base before running detectors, so detectors (e.g. service) can override it. Previously it was added to config_attrs and merged last, which would have silently overridden any detector-provided service.name. Assisted-by: Claude Sonnet 4.6 * remove unused logging import from _propagator.py Assisted-by: Claude Sonnet 4.6 * add create_logger_provider/configure_logger_provider for declarative config Implements LoggerProvider instantiation from declarative config files, following the same env-var-suppression pattern as create_meter_provider. BatchLogRecordProcessor defaults use spec values (1000ms schedule_delay), overriding the Python SDK's incorrect 5000ms env-var default. Assisted-by: Claude Sonnet 4.6 * add changelog entry for logger provider declarative config (open-telemetry#4990) Assisted-by: Claude Sonnet 4.6 * fix linter errors * add test verifying OTEL_PROPAGATORS env var is ignored by configure_propagator Assisted-by: Claude Sonnet 4.6 * remove backwards compat re-export of ConfigurationError from _loader.py Import directly from _exceptions.py since this is new code with no existing dependents on the _loader module path. Assisted-by: Claude Sonnet 4.6 * address review feedback: simplify resource filter and propagator loading Assisted-by: Claude Sonnet 4.6 * fix ruff formatting Assisted-by: Claude Sonnet 4.6 * fix pyright: wrap EntryPoints in iter() for next() compatibility Assisted-by: Claude Sonnet 4.6 * remove stale SDK bug note fixed in v1.41.0 Assisted-by: Claude Sonnet 4.6 --------- Co-authored-by: Emídio Neto <9735060+emdneto@users.noreply.github.com> Co-authored-by: Riccardo Magliocchetti <riccardo.magliocchetti@gmail.com>

MikeGoldsmith added 7 commits March 13, 2026 13:04

update changelog with PR number

8232012

Assisted-by: Claude Sonnet 4.6

fix linter

8232d48

Merge branch 'main' of github.com:open-telemetry/opentelemetry-python…

6ed3425

… into mike/config-resource-propagator

MikeGoldsmith added a commit to MikeGoldsmith/opentelemetry-python that referenced this pull request Mar 17, 2026

add changelog entry for logger provider declarative config (open-tele…

1b93f64

…metry#4990) Assisted-by: Claude Sonnet 4.6

MikeGoldsmith changed the title ~~feat(config): add create_logger_provider/configure_logger_provider for declarative config~~ feat(config): Add LoggerProvider support for declarative config Mar 18, 2026

xrmx mentioned this pull request Mar 20, 2026

Add declarative configuration prototype #3631

Open

22 tasks

MikeGoldsmith added 4 commits March 20, 2026 14:18

use Callable type annotation on _array helper

8ba91d8

Assisted-by: Claude Sonnet 4.6

Merge remote-tracking branch 'upstream/main' into mike/config-resourc…

516aecc

…e-propagator

MikeGoldsmith added a commit to MikeGoldsmith/opentelemetry-python that referenced this pull request Mar 20, 2026

add changelog entry for logger provider declarative config (open-tele…

21041ec

…metry#4990) Assisted-by: Claude Sonnet 4.6

MikeGoldsmith force-pushed the mike/config-logger-provider branch from 90ab88b to d149d79 Compare March 20, 2026 14:30

MikeGoldsmith added 4 commits March 20, 2026 18:22

remove unused logging import from _propagator.py

7f51034

Assisted-by: Claude Sonnet 4.6

add changelog entry for logger provider declarative config (open-tele…

da00c92

…metry#4990) Assisted-by: Claude Sonnet 4.6

fix linter errors

31933ef

MikeGoldsmith force-pushed the mike/config-logger-provider branch from d149d79 to 31933ef Compare March 20, 2026 19:40

MikeGoldsmith added 2 commits March 24, 2026 22:38

add test verifying OTEL_PROPAGATORS env var is ignored by configure_p…

16b89b1

…ropagator Assisted-by: Claude Sonnet 4.6

remove backwards compat re-export of ConfigurationError from _loader.py

a171842

Import directly from _exceptions.py since this is new code with no existing dependents on the _loader module path. Assisted-by: Claude Sonnet 4.6

MikeGoldsmith mentioned this pull request Mar 29, 2026

feat(config): add shared _parse_headers helper for declarative config exporters #5021

Merged

6 tasks

herin049 approved these changes Apr 1, 2026

View reviewed changes

Comment thread opentelemetry-sdk/src/opentelemetry/sdk/_configuration/_propagator.py Outdated

Comment thread opentelemetry-sdk/src/opentelemetry/sdk/_configuration/_propagator.py Outdated

MikeGoldsmith added 4 commits April 1, 2026 20:12

merge upstream/main and use shared _parse_headers from _common

d7b4616

Assisted-by: Claude Sonnet 4.6

address review feedback: simplify resource filter and propagator loading

14ba572

Assisted-by: Claude Sonnet 4.6

fix ruff formatting

14b1c9b

Assisted-by: Claude Sonnet 4.6

fix pyright: wrap EntryPoints in iter() for next() compatibility

c0906d4

Assisted-by: Claude Sonnet 4.6

MikeGoldsmith marked this pull request as ready for review April 3, 2026 15:24

MikeGoldsmith requested a review from a team as a code owner April 3, 2026 15:24

MikeGoldsmith added 2 commits April 7, 2026 15:22

merge origin

105ed1b

Assisted-by: Claude Sonnet 4.6

merge upstream/main

7375dff

Assisted-by: Claude Sonnet 4.6

tammy-baylis-swi reviewed Apr 7, 2026

View reviewed changes

Comment thread opentelemetry-sdk/src/opentelemetry/sdk/_configuration/_logger_provider.py

MikeGoldsmith mentioned this pull request Apr 8, 2026

bug(config): _map_compression does not support deflate for HTTP exporters #5059

Open

MikeGoldsmith added the config Issues and PRs related to implementing Declarative Config label Apr 8, 2026

MikeGoldsmith added this to Python PR digest Apr 8, 2026

MikeGoldsmith moved this to Approved PRs in Python PR digest Apr 8, 2026

MikeGoldsmith added 2 commits April 9, 2026 15:41

merge upstream/main

8f47a77

Assisted-by: Claude Sonnet 4.6

merge upstream/main

1ee239a

Assisted-by: Claude Sonnet 4.6

xrmx reviewed Apr 13, 2026

View reviewed changes

Comment thread opentelemetry-sdk/src/opentelemetry/sdk/_configuration/_logger_provider.py Outdated

xrmx approved these changes Apr 13, 2026

View reviewed changes

MikeGoldsmith added 2 commits April 13, 2026 11:15

remove stale SDK bug note fixed in v1.41.0

f760153

Assisted-by: Claude Sonnet 4.6

merge upstream/main into mike/config-logger-provider

44f7a5f

Assisted-by: Claude Sonnet 4.6

MikeGoldsmith force-pushed the mike/config-logger-provider branch from 9af2cfe to 44f7a5f Compare April 13, 2026 10:17

xrmx requested a review from tammy-baylis-swi April 13, 2026 15:06

MikeGoldsmith and others added 3 commits April 13, 2026 16:13

Merge branch 'main' into mike/config-logger-provider

88d48fb

Merge branch 'main' into mike/config-logger-provider

7dced1e

Merge branch 'main' into mike/config-logger-provider

30458d2

xrmx enabled auto-merge (squash) April 13, 2026 15:39

xrmx merged commit 7477b10 into open-telemetry:main Apr 13, 2026
465 of 466 checks passed

github-project-automation Bot moved this from Approved PRs to Done in Python PR digest Apr 13, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(config): Add LoggerProvider support for declarative config#4990

feat(config): Add LoggerProvider support for declarative config#4990
xrmx merged 30 commits intoopen-telemetry:mainfrom
MikeGoldsmith:mike/config-logger-provider

MikeGoldsmith commented Mar 17, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Conversation

MikeGoldsmith commented Mar 17, 2026

Description

What's included

Env-var suppression

Type of change

How Has This Been Tested?

Does This PR Require a Contrib Repo Change?

Checklist

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants