[all] migrate from deprecated addSpanProcessor to *TracerProvider constructor option
#2645
Labels
type:feature-tracking
A feature with sub-issues that need to be addressed
Comments
pichlermarc
added
the
type:feature-tracking
2 tasks
chancancode
added a commit
to tildeio/opentelemetry-js-contrib
that referenced
this issue
Jan 16, 2025
This migrates from the deprecated `addSpanProcessor` method to the constructor option in preparation for the upcoming SDK 2.0 release. This commit migrates everything in the `examples` folder and there are no changes to the published packages. Ref open-telemetry#2645
chancancode
added a commit
to tildeio/opentelemetry-js-contrib
that referenced
this issue
Jan 17, 2025
This migrates from the deprecated `addSpanProcessor` method to the constructor option in preparation for the upcoming SDK 2.0 release. This commit migrates everything in the README.md of each package, other than that there are no runtime code changes. Technically this is a user-facing change for users that previously followed the examples. Ref open-telemetry#2645
chancancode
added a commit
to tildeio/opentelemetry-js-contrib
that referenced
this issue
Jan 17, 2025
This migrates from the deprecated `addSpanProcessor` method to the constructor option in preparation for the upcoming SDK 2.0 release. This commit migrates all usages **in tests** for each package, and there are no changes to the runtime code. Ref open-telemetry#2645
chancancode
added a commit
to tildeio/opentelemetry-js-contrib
that referenced
this issue
Jan 17, 2025
This migrates from the deprecated `addSpanProcessor` method to the constructor option in preparation for the upcoming SDK 2.0 release. This commit migrates the runtime code in the test-utils package and should be included in the package's CHANGELOG. Ref open-telemetry#2645
This was referenced Jan 17, 2025
Collectively, these PRs addresses all usages in this repository. After all of them are merged, this can be closed. |
Awesome, thanks for working on this 🙂 |
chancancode
added a commit
to tildeio/opentelemetry-js-contrib
that referenced
this issue
Jan 17, 2025
This migrates from the deprecated `addSpanProcessor` method to the constructor option in preparation for the upcoming SDK 2.0 release. This commit migrates all usages **in tests** for each package, and there are no changes to the runtime code. Ref open-telemetry#2645
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
*TracerProvider#addSpanProcessor()is deprecated in favor of a constructor option and will be removed in SDK 2.0. To ensure a smoother transition for the contrib repo when we publish SDK 2.0, we should take care of this before we release 2.0 on February 17.This issue is considered done when we have applied these changes in all packages of this repository
*TracerProvider#addSpanProcessor()to the constructor optionPackages checklist:
Note
For traigers/approvers/maintainers please link the appropriate PRs and update check boxes as we work through them. Some of these packages may not use this method anywhere. In that case, please simple mark them as done and add a comment "not used"
The text was updated successfully, but these errors were encountered: