Skip to content

[Exporter.Geneva] Add thread-safety regression tests for MsgPackTraceExporter and MsgPackLogExporter#3886

Merged
rajkumar-rangaraj merged 6 commits into
open-telemetry:mainfrom
mattsains:investigateGenevaThreadingIssue
Feb 19, 2026
Merged

[Exporter.Geneva] Add thread-safety regression tests for MsgPackTraceExporter and MsgPackLogExporter#3886
rajkumar-rangaraj merged 6 commits into
open-telemetry:mainfrom
mattsains:investigateGenevaThreadingIssue

Conversation

@mattsains

Copy link
Copy Markdown
Contributor

This PR is based on #3882

Changes

Added two tests that run the log and trace exporter in parallel looking for race conditions

Merge requirement checklist

  • CONTRIBUTING guidelines followed (license requirements, nullable enabled, static analysis, etc.)
  • Unit tests added/updated
  • [n/a] Appropriate CHANGELOG.md files updated for non-trivial changes
  • [n/a] Changes in public API reviewed (if applicable)

@mattsains
mattsains requested a review from a team as a code owner February 19, 2026 17:34
@github-actions github-actions Bot added the comp:exporter.geneva Things related to OpenTelemetry.Exporter.Geneva label Feb 19, 2026
@codecov

codecov Bot commented Feb 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.03%. Comparing base (4a4fab7) to head (ce7ac07).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3886      +/-   ##
==========================================
+ Coverage   71.59%   72.03%   +0.43%     
==========================================
  Files         447      447              
  Lines       17827    17815      -12     
==========================================
+ Hits        12764    12833      +69     
+ Misses       5063     4982      -81     
Flag Coverage Δ
unittests-Exporter.Geneva 54.60% <ø> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 13 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rajkumar-rangaraj rajkumar-rangaraj left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rajkumar-rangaraj
rajkumar-rangaraj added this pull request to the merge queue Feb 19, 2026
Merged via the queue into open-telemetry:main with commit 9f7b05c Feb 19, 2026
78 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp:exporter.geneva Things related to OpenTelemetry.Exporter.Geneva

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants