Description

Fixes #4671

This PR fixes a bug where the LoggingHandler in the OpenTelemetry Python SDK does not respect the OTEL_ATTRIBUTE_COUNT_LIMIT environment variable. When this environment variable is set, it should limit the number of attributes in log records, but the LoggingHandler was ignoring this setting entirely and only use the default value 128.
It means:

• If we set to small count limit like 3, logging 10 attributes still works without any attributes dropped.
• If we set to high count limit like 256, logging 200 attributes has (200-128= 72) attributes dropped.

Changes Made

Fixed the Root Cause Bug

The core issue was that LogRecord was created with _UnsetLogLimits which used UNSET values, and the LogLimits._from_env_if_absent method had a bug where it returned None immediately when value == cls.UNSET, never checking environment variables.

# Before (buggy):
_UnsetLogLimits = LogLimits(max_attributes=LogLimits.UNSET, ...)
# This caused _from_env_if_absent to return None immediately without reading env vars

# After (fixed):
# Removed _UnsetLogLimits entirely, now use None as default
limits = limits or LogLimits()  # LogLimits() correctly reads from env vars

Implementation Benefits

This implementation allows users to:

• Rely on environment variables for configuration (OTEL_ATTRIBUTE_COUNT_LIMIT)
• Use a cleaner, more intuitive API without special UNSET values

Type of change

• Bug fix (non-breaking change which fixes an issue)
• Code refactoring (improves code quality without changing functionality)

How Has This Been Tested?

You can run the tests using pytest:

# Install the package in development mode
pip install -e .

# Run all the LoggingHandler tests
pytest opentelemetry-sdk/tests/logs/test_handler.py -v

# Run all logging tests
pytest opentelemetry-sdk/tests/logs/ -v

# Run only the attribute count limit tests
pytest opentelemetry-sdk/tests/logs/test_handler.py::TestLoggingHandler -k "otel_attribute_count_limit" -v

Test Coverage

Added comprehensive tests to verify the fix works correctly:

1. test_otel_attribute_count_limit_respected_in_logging_handler - Tests that when OTEL_ATTRIBUTE_COUNT_LIMIT=3, only 3 attributes are kept and the rest are dropped
2. test_otel_attribute_count_limit_includes_code_attributes - Tests that when OTEL_ATTRIBUTE_COUNT_LIMIT=5, only 5 attributes are kept and the limit applies to all attribute types including code attributes
3. test_logging_handler_without_env_var_uses_default_limit - Tests that without the environment variable, the default limit of 128 is applied

All tests pass, confirming the fix works correctly across all these scenarios.

Does This PR Require a Contrib Repo Change?

• No.

Checklist:

• Followed the style guidelines of this project
• Unit tests have been added
• Documentation has been updated
• Fixed the original bug with OTEL_ATTRIBUTE_COUNT_LIMIT
• Refactored code for better maintainability
• Removed deprecated/unnecessary code patterns
• All existing tests continue to pass