Skip to content

chore(*): migrate use of sdk-trace-base and sdk-trace-node to sdk-trace - #6851

Merged
trentm merged 9 commits into
open-telemetry:mainfrom
trentm:trentm-migrate-to-sdk-trace-take2
Jun 26, 2026
Merged

chore(*): migrate use of sdk-trace-base and sdk-trace-node to sdk-trace#6851
trentm merged 9 commits into
open-telemetry:mainfrom
trentm:trentm-migrate-to-sdk-trace-take2

Conversation

@trentm

@trentm trentm commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

The sdk-trace package is the intended replacement for sdk-trace-base and sdk-trace-node.
The intention is to deprecate sdk-trace-base and sdk-trace-node soonish.

Notes:

  • Usages of @opentelemetry/sdk-trace-web have generally not been updated
    in this PR. This is deferred to a future PR. (I may need help with this.)
  • Some examples/ are not updated in this PR. For example, I'd like to
    update "examples/http/..." in a separate PR.

Refs: #6828 (migrated sdk-node package to use sdk-trace)

The sdk-trace package is the intended replacement for sdk-trace-base
and sdk-trace-node.
The intention is to deprecate sdk-trace-base and sdk-trace-node soonish.

Notes:
- Usages of `@opentelemetry/sdk-trace-web` have generally *not* been updated
  in this PR. This is deferred to a future PR.
- Some examples/ are not updated in this PR. For example, I'd like to
  update "examples/http/..." in a separate PR.
@trentm trentm self-assigned this Jun 25, 2026
@codecov

codecov Bot commented Jun 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.57%. Comparing base (c28e693) to head (0497f13).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6851   +/-   ##
=======================================
  Coverage   95.57%   95.57%           
=======================================
  Files         387      387           
  Lines       13146    13146           
  Branches     3001     3001           
=======================================
  Hits        12564    12564           
  Misses        582      582           
Files with missing lines Coverage Δ
api/src/trace/SamplingResult.ts 100.00% <ø> (ø)
.../exporter-trace-otlp-grpc/src/OTLPTraceExporter.ts 100.00% <ø> (ø)
...tlp-http/src/platform/browser/OTLPTraceExporter.ts 100.00% <ø> (ø)
...e-otlp-http/src/platform/node/OTLPTraceExporter.ts 100.00% <ø> (ø)
...lp-proto/src/platform/browser/OTLPTraceExporter.ts 100.00% <ø> (ø)
...-otlp-proto/src/platform/node/OTLPTraceExporter.ts 100.00% <ø> (ø)
...al/packages/otlp-transformer/src/trace/internal.ts 98.55% <ø> (ø)
.../packages/otlp-transformer/src/trace/json/trace.ts 100.00% <ø> (ø)
...transformer/src/trace/protobuf/trace-serializer.ts 96.62% <ø> (ø)
...kages/otlp-transformer/src/trace/protobuf/trace.ts 100.00% <ø> (ø)
... and 9 more
🚀 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.

@trentm
trentm marked this pull request as ready for review June 25, 2026 16:35
@trentm
trentm requested review from a team as code owners June 25, 2026 16:35
Comment thread .github/ISSUE_TEMPLATE/bug_report.yaml Outdated
Comment thread examples/basic-tracer-node/package.json Outdated
trentm and others added 3 commits June 26, 2026 09:48
Co-authored-by: Marc Pichler <marc.pichler@dynatrace.com>
Co-authored-by: Marc Pichler <marc.pichler@dynatrace.com>
@trentm
trentm added this pull request to the merge queue Jun 26, 2026
Merged via the queue into open-telemetry:main with commit 300c92d Jun 26, 2026
30 checks passed
@trentm
trentm deleted the trentm-migrate-to-sdk-trace-take2 branch June 26, 2026 18:29
@maryliag maryliag mentioned this pull request Jun 26, 2026
@otelbot-js otelbot-js Bot mentioned this pull request Jul 2, 2026
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.

2 participants