Skip to content

Conversation

@pichlermarc
Copy link
Member

@pichlermarc pichlermarc commented Oct 6, 2025

Which problem is this PR solving?

A whole bunch of bundlers (at least vite, and rollup, likely many more) are failing to build when @opentelemetry/sdk-node is used. The reason for it is that we try to lazy-load @opentelemetry/exporter-jaeger if present, which we've marked as deprecated for a long time now. That exporter has been superseded by @opentelemetry/exporter-trace-otlp-http et al. since Jaeger now has native support for OTLP

This PR removes the lazy-loading in favor of the OTLP exporter, which remains fully supported.

Though deprecated, the Jaeger exporter will continue to be published from this repo until we mark @opentelemetry/exporter-trace-otlp-http as stable.

Fixes #4297

@codecov
Copy link

codecov bot commented Oct 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.23%. Comparing base (986719e) to head (5e0ceaa).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@ ## main #5989 +/- ## ======================================= Coverage 95.22% 95.23% ======================================= Files 311 311 Lines 8603 8597 -6 Branches 1801 1801 ======================================= - Hits 8192 8187 -5  + Misses 411 410 -1 
Files with missing lines Coverage Δ
...ental/packages/opentelemetry-sdk-node/src/utils.ts 92.30% <ø> (+0.39%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
@pichlermarc pichlermarc changed the title feat(sdk-node)!: drop dynamic loading of jaeger exporter feat(sdk-node)!: drop lazy-loading of jaeger exporter Oct 6, 2025
@pichlermarc pichlermarc marked this pull request as ready for review October 6, 2025 18:22
@pichlermarc pichlermarc requested a review from a team as a code owner October 6, 2025 18:22
@pichlermarc
Copy link
Member Author

I'll leave this open for a bit longer to give everyone some time to voice concerns, if any.
If there's no opposing comments I'll merge this in on Monday. 🙂

@pichlermarc pichlermarc force-pushed the feat/drop-jaeger-exporter branch from 91f3693 to 53072ce Compare October 13, 2025 08:34
@pichlermarc pichlermarc force-pushed the feat/drop-jaeger-exporter branch from 53072ce to 5e0ceaa Compare October 13, 2025 08:40
@pichlermarc pichlermarc added this pull request to the merge queue Oct 13, 2025
Merged via the queue into open-telemetry:main with commit b758808 Oct 13, 2025
25 checks passed
@pichlermarc pichlermarc deleted the feat/drop-jaeger-exporter branch October 13, 2025 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

3 participants