Skip to content

[SDK] LogRecord attribute limits enforcement#4157

Open
thc1006 wants to merge 18 commits into
open-telemetry:mainfrom
thc1006:feat/log-record-limits-4126
Open

[SDK] LogRecord attribute limits enforcement#4157
thc1006 wants to merge 18 commits into
open-telemetry:mainfrom
thc1006:feat/log-record-limits-4126

[SDK] LogRecord limits: drop vtable implementation-detail comment

378442f
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jun 30, 2026 in 1s

82.84% (-0.05%) compared to a56f0e2

View this Pull Request on Codecov

82.84% (-0.05%) compared to a56f0e2

Details

Codecov Report

❌ Patch coverage is 93.28358% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.84%. Comparing base (a56f0e2) to head (378442f).

Files with missing lines Patch % Lines
...xporters/otlp/src/otlp_populate_attribute_utils.cc 80.77% 5 Missing ⚠️
...include/opentelemetry/sdk/common/attribute_utils.h 97.50% 1 Missing ⚠️
sdk/include/opentelemetry/sdk/logs/exporter.h 0.00% 1 Missing ⚠️
...clude/opentelemetry/sdk/logs/readable_log_record.h 0.00% 1 Missing ⚠️
sdk/include/opentelemetry/sdk/logs/recordable.h 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #4157      +/-   ##
==========================================
- Coverage   82.89%   82.84%   -0.05%     
==========================================
  Files         405      415      +10     
  Lines       17292    17428     +136     
==========================================
+ Hits        14332    14436     +104     
- Misses       2960     2992      +32     
Files with missing lines Coverage Δ
...ntelemetry/exporters/ostream/log_record_exporter.h 100.00% <100.00%> (ø)
...try/exporters/otlp/otlp_file_log_record_exporter.h 100.00% <100.00%> (ø)
...try/exporters/otlp/otlp_grpc_log_record_exporter.h 100.00% <100.00%> (ø)
...try/exporters/otlp/otlp_http_log_record_exporter.h 100.00% <100.00%> (ø)
...opentelemetry/exporters/otlp/otlp_log_recordable.h 100.00% <ø> (ø)
...try/exporters/otlp/otlp_populate_attribute_utils.h 100.00% <100.00%> (ø)
exporters/otlp/src/otlp_log_recordable.cc 38.16% <100.00%> (+2.11%) ⬆️
...pentelemetry/sdk/logs/batch_log_record_processor.h 100.00% <100.00%> (ø)
...include/opentelemetry/sdk/logs/log_record_limits.h 100.00% <100.00%> (ø)
sdk/include/opentelemetry/sdk/logs/processor.h 100.00% <100.00%> (ø)
... and 12 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.