Skip to content

fix(opentelemetry-sdk-node): add missing @opentelemetry/otlp-exporter-base dependency#6520

Merged
pichlermarc merged 3 commits intoopen-telemetry:mainfrom
gotgenes:fix/sdk-node-missing-otlp-exporter-base-dep
Mar 24, 2026
Merged

fix(opentelemetry-sdk-node): add missing @opentelemetry/otlp-exporter-base dependency#6520
pichlermarc merged 3 commits intoopen-telemetry:mainfrom
gotgenes:fix/sdk-node-missing-otlp-exporter-base-dep

Conversation

@gotgenes
Copy link
Copy Markdown
Contributor

Which problem is this PR solving?

@opentelemetry/sdk-node imports CompressionAlgorithm from @opentelemetry/otlp-exporter-base in src/utils.ts (added in #6407), but the dependency is not declared in the package's package.json. This causes a runtime Cannot find module error under strict package managers like pnpm that do not hoist transitive dependencies.

Fixes #6519

Short description of the changes

  • Add "@opentelemetry/otlp-exporter-base": "0.213.0" to dependencies in experimental/packages/opentelemetry-sdk-node/package.json
  • Add changelog entry in experimental/CHANGELOG.md

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Checklist:

  • Followed the style guidelines of this project
  • Unit tests have been added
  • Documentation has been updated

@gotgenes gotgenes requested a review from a team as a code owner March 23, 2026 17:54
@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented Mar 23, 2026

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: gotgenes / name: Chris Lasher (05c28f7)
  • ✅ login: pichlermarc / name: Marc Pichler (a663801, ccffea7)

…-base dependency

The import of CompressionAlgorithm from @opentelemetry/otlp-exporter-base was
added in open-telemetry#6407 but the dependency was not added to package.json. This causes
a runtime error under strict package managers like pnpm that do not hoist
transitive dependencies.

Signed-off-by: Chris Lasher <chris.lasher@gmail.com>
@gotgenes gotgenes force-pushed the fix/sdk-node-missing-otlp-exporter-base-dep branch from 0393b3b to 05c28f7 Compare March 23, 2026 17:55
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.73%. Comparing base (e9831e7) to head (ccffea7).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6520      +/-   ##
==========================================
- Coverage   95.75%   95.73%   -0.02%     
==========================================
  Files         364      364              
  Lines       12096    12104       +8     
  Branches     2885     2887       +2     
==========================================
+ Hits        11582    11588       +6     
- Misses        514      516       +2     

see 1 file with indirect coverage changes

🚀 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 added this pull request to the merge queue Mar 24, 2026
Merged via the queue into open-telemetry:main with commit b5a6b12 Mar 24, 2026
27 checks passed
@otelbot
Copy link
Copy Markdown
Contributor

otelbot Bot commented Mar 24, 2026

Thank you for your contribution @gotgenes! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey.

@gotgenes gotgenes deleted the fix/sdk-node-missing-otlp-exporter-base-dep branch March 24, 2026 13:39
@gotgenes
Copy link
Copy Markdown
Contributor Author

Thank you, @pichlermarc!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(opentelemetry-sdk-node): missing @opentelemetry/otlp-exporter-base dependency

2 participants