diff --git a/CHANGELOG.md b/CHANGELOG.md index dac90bf15f3..48962fc0709 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -12,6 +12,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 ## Unreleased +- Fix `BatchLogRecordProcessor` default `schedule_delay_millis` from 5000ms to 1000ms to comply with the OTel specification. Note: logs will be exported 5x more frequently by default (e.g. for users who don't explicitly set the `OTEL_BLRP_SCHEDULE_DELAY` env var). + ([#4991](https://github.com/open-telemetry/opentelemetry-python/issues/4991)) - `opentelemetry-sdk`: Add file configuration support with YAML/JSON loading, environment variable substitution, and schema validation against the vendored OTel config JSON schema ([#4898](https://github.com/open-telemetry/opentelemetry-python/pull/4898)) - Fix intermittent CI failures in `getting-started` and `tracecontext` jobs caused by GitHub git CDN SHA propagation lag by installing contrib packages from the already-checked-out local copy instead of a second git clone diff --git a/opentelemetry-sdk/src/opentelemetry/sdk/_logs/_internal/export/__init__.py b/opentelemetry-sdk/src/opentelemetry/sdk/_logs/_internal/export/__init__.py index 52ce2d6e6ff..1f8015e52f3 100644 --- a/opentelemetry-sdk/src/opentelemetry/sdk/_logs/_internal/export/__init__.py +++ b/opentelemetry-sdk/src/opentelemetry/sdk/_logs/_internal/export/__init__.py @@ -44,7 +44,7 @@ ) from opentelemetry.sdk.resources import Resource -_DEFAULT_SCHEDULE_DELAY_MILLIS = 5000 +_DEFAULT_SCHEDULE_DELAY_MILLIS = 1000 _DEFAULT_MAX_EXPORT_BATCH_SIZE = 512 _DEFAULT_EXPORT_TIMEOUT_MILLIS = 30000 _DEFAULT_MAX_QUEUE_SIZE = 2048 diff --git a/opentelemetry-sdk/src/opentelemetry/sdk/environment_variables/__init__.py b/opentelemetry-sdk/src/opentelemetry/sdk/environment_variables/__init__.py index f049415a15b..7e143dd3bb6 100644 --- a/opentelemetry-sdk/src/opentelemetry/sdk/environment_variables/__init__.py +++ b/opentelemetry-sdk/src/opentelemetry/sdk/environment_variables/__init__.py @@ -79,7 +79,7 @@ .. envvar:: OTEL_BLRP_SCHEDULE_DELAY The :envvar:`OTEL_BLRP_SCHEDULE_DELAY` represents the delay interval between two consecutive exports of the BatchLogRecordProcessor. -Default: 5000 +Default: 1000 """ OTEL_BLRP_EXPORT_TIMEOUT = "OTEL_BLRP_EXPORT_TIMEOUT" diff --git a/opentelemetry-sdk/tests/logs/test_export.py b/opentelemetry-sdk/tests/logs/test_export.py index 12909b2e225..0156b01fea0 100644 --- a/opentelemetry-sdk/tests/logs/test_export.py +++ b/opentelemetry-sdk/tests/logs/test_export.py @@ -513,7 +513,7 @@ def test_args_defaults(self): log_record_processor._batch_processor._max_queue_size, 2048 ) self.assertEqual( - log_record_processor._batch_processor._schedule_delay, 5 + log_record_processor._batch_processor._schedule_delay, 1 ) self.assertEqual( log_record_processor._batch_processor._max_export_batch_size, 512 @@ -544,7 +544,7 @@ def test_args_env_var_value_error(self): log_record_processor._batch_processor._max_queue_size, 2048 ) self.assertEqual( - log_record_processor._batch_processor._schedule_delay, 5 + log_record_processor._batch_processor._schedule_delay, 1 ) self.assertEqual( log_record_processor._batch_processor._max_export_batch_size, 512 @@ -570,7 +570,7 @@ def test_args_none_defaults(self): log_record_processor._batch_processor._max_queue_size, 2048 ) self.assertEqual( - log_record_processor._batch_processor._schedule_delay, 5 + log_record_processor._batch_processor._schedule_delay, 1 ) self.assertEqual( log_record_processor._batch_processor._max_export_batch_size, 512