[mateenali66]

Changes

Adds me to the Approvers list in README.md, per discussion with @marcalff on Slack. He suggested approver directly instead of triager, since the repo has no triager team.

PRs I authored that merged:

• [SDK] Add ComposableSampler interface and CompositeSampler #4133 ComposableSampler interface and CompositeSampler
• [CONFIGURATION] Add builders for the container and process resource detectors #4417, [RESOURCE DETECTOR] Add the host resource detector #4418, [CONFIGURATION] Add a resource detector extension example #4423, [CONFIGURATION] Complete resource detection support in SdkBuilder #4424 resource detector builders, the host detector, the extension example and SdkBuilder support
• [BUG] Rewind the curl request body with a seek callback #4557 rewind the curl request body with a seek callback, which came out of the [BUG] OTLP/HTTP drops export batches on transport errors: no CURLOPT_SEEKFUNCTION, and IsRetryable() requires CURLE_OK #4549 triage, and [BUG] Compare the curl seek offset before narrowing it to size_t #4630 as its follow-up
• [CODE HEALTH] Fix clang-tidy misc-use-internal-linkage warnings #3985, [CODE HEALTH] Fix clang-tidy narrowing-conversions warnings in tests #3987, [CODE HEALTH] Move trace and baggage propagation test classes into anonymous namespace #4199, [CODE HEALTH] Move context propagation test classes into anonymous namespace #4200, [CODE HEALTH] Move ext and exporters test classes into anonymous namespace #4225, [CODE HEALTH] Move otlp, zipkin and prometheus test classes into anonymous namespace #4231 clang-tidy and test cleanups

PRs from other contributors I reviewed:

• [BUG] Prevent lost condition-variable wakeups during OTLP file, periodic metric, and batch span processor shutdown #4365, [SDK] Fix lost wakeup in BatchSpanProcessor shutdown/force-flush notify #4382, [CONFIGURATION] Apply metric view attribute filters #4385, [BUG] Keep the curl retry deadline stable within an attempt #4452, [SDK] Fix MetricReader post shutdown handling #4548, [METRICS SDK] last collection of metrics before shutdown #4550, [LOGS] Fix Logger::EmitLogRecord unsafe static_cast to Recordable #4624 (merged)
• [METRICS] Fix stale async attribute sets in cumulative exports (#4108) #4484, [EXPORTER] Fix Elasticsearch log exporter Export blocking forever on a non-responding client #4530, [EXPORTER] Avoid std::terminate when the OTLP HTTP response body cannot be processed #4587, [CI] Check header-only public files with IWYU #4643 (still open)

Issues I triaged, mostly with a reproduction or a trace through the code:

• [SECURITY] Transitive Vulnerabilities In CivetWeb via Prometheus-Cpp #4491, [SDK] BatchLogRecordProcessor still drains the queue in a tight loop, inconsistent with BatchSpanProcessor fix #4498, [BUG] BatchSpanProcessor::Shutdown ignores its timeout when the worker thread is blocked in Export() #4532, [BUG] MetricReader::Shutdown() logs "cannot invoke shutdown twice" but calls OnShutDown() again anyway #4536, [BUG] Logger::EmitLogRecord() blindly static_casts a caller-supplied LogRecord to Recordable #4537, [BUG] OTLP/HTTP drops export batches on transport errors: no CURLOPT_SEEKFUNCTION, and IsRetryable() requires CURLE_OK #4549, [BUG] Retry-After beyond max_backoff stalls the curl retry queue and HttpClient shutdown #4631, Latest code fails to build using CMake FetchContent on macOS and openSUSE Leap 16.0 with ZLIB-related errors #4661, Build fails on Ubuntu 26.04 due to abseil issue: GCC 15 difference where <cstdint> must be #included explicitly #4662
• filed [BUG] SessionState::TimedOut is never dispatched, so every timeout surfaces as SendFailed #4604

The suggested triage label was applied on #4491, #4532, #4536, #4537 and #4549.

For significant contributions please make sure you have completed the following items:

• CHANGELOG.md updated for non-trivial changes
• Unit tests have been added
• Changes in public API reviewed

[marcalff]

Thanks for your contributions.

[dbarker]

Thanks Mateen!

[lalitb]

Thanks for the contributions. @mateenali66 .

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.83%. Comparing base (f19af27) to head (3724bd9).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #4699   +/-   ##
=======================================
  Coverage   86.83%   86.83%           
=======================================
  Files         525      525           
  Lines       20726    20726           
=======================================
  Hits        17995    17995           
  Misses       2731     2731           

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[esigo]

Thanks for your contributions:)

[marcalff]

Approved by all maintainers.