fix(deps): update opentelemetry-js monorepo to v2 (major) #11325
Conversation
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
There was a problem hiding this comment.
Pull request overview
This PR updates the OpenTelemetry JavaScript SDK from v1 to v2, which includes significant breaking changes and API improvements. The update modernizes resource creation, simplifies SDK initialization using NodeSDK, adds proper shutdown handling, prevents duplicate initialization, and wraps Sentry initialization in a configuration check.
Changes:
- Updated OpenTelemetry packages from v1.x to v2.x with breaking API changes
- Refactored initialization to use NodeSDK instead of manual provider setup
- Added duplicate initialization prevention and proper shutdown handlers
- Updated resource creation to use new
resourceFromAttributesAPI
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| server/open-telemetry.ts | Migrated to v2 SDK using NodeSDK, added shutdown handlers and duplicate initialization prevention |
| server/lib/sentry/init.ts | Wrapped Sentry initialization in configuration check to prevent initialization when not configured |
| package.json | Updated OpenTelemetry dependencies to v2.x versions and consolidated packages |
| package-lock.json | Dependency lockfile updates reflecting the v2 migration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| const sdk = new NodeSDK({ | ||
| instrumentations, | ||
| }); | ||
| try { | ||
| sdk.start(); | ||
| } catch (error) { |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
znarf
force-pushed
the
renovate/major-opentelemetry-js-monorepo
branch
from
521d932 to
44ee804
Compare
znarf
force-pushed
the
renovate/major-opentelemetry-js-monorepo
branch
from
44ee804 to
532ae0e
Compare
|
Cleaned up in #11369 |
Renovate Ignore NotificationBecause you closed this PR without merging, Renovate will ignore this update. You will not get PRs for any future If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR. |
2 participants
This PR contains the following updates:
^1.20.0→^2.0.01.30.1→2.3.0^1.20.0→^2.0.0Release Notes
open-telemetry/opentelemetry-js (@opentelemetry/resources)
v2.3.0Compare Source
🚀 Features
🐛 Bug Fixes
🏠 Internal
v2.2.0Compare Source
🐛 Bug Fixes
unrefTimer()util #5986 @pichlermarcotperformanceotperformance.now()andotperformance.timeOriginare not affected.performanceimplementation instead.🏠 Internal
v2.1.0Compare Source
🚀 Features
🐛 Bug Fixes
startTimeparam to ExponentialHistogramAccumulation. This only impacted the closurescript compiler. #5763 @trentm🏠 Internal
v2.0.1Compare Source
🐛 Bug Fixes
Accessing resource attributes before async attributes settledwarning when detecting resources #5546 @dyladangetNumberFromEnvshould return number | undefined #5874 @shubham-vunet📚 Documentation
sdk.start()as async #5617 @JacksonWeber🏠 Internal
parseKeyPairsIntoRecord()#5610 @cjihrigv2.0.0Compare Source
Summary
^18.19.0 || >=20.6.0. This means that support for Node.js 14 and 16 has been dropped.0.200.0are compatible with this release💥 Breaking Change
parentSpanContextand removeparentSpanIdfromSpanandReadableSpan#5450 @JacksonWeberSpansparentSpanIdwas replaced byparentSpanContext, to migrate to the new property, please replacespan.parentSpanId->span.parentSpanContext?.spanIdtypefield onMetricDescriptor#5291 @chancancodeInstrumentDescriptortype; useMetricDescriptorinstead #5277 @chancancode@opentelemetry/apipeer dependency to 1.9.0 #5254 @chancancodeSpanAttributeswithAttributes#5009 @david-lunaResourceAttributeswithAttributes#5016 @david-lunaViewandAggregationin favor ofViewOptionsandAggregationOption#4931 @pichlermarcnew Spanconstructor in favor ofTracer.startSpanAPI #5048 @david-lunaBasicTracerProvider.addSpanProcessorAPI in favor of constructor options. #5134 @david-lunaresourceproperty private inBasicTracerProviderand removegetActiveSpanProcessorAPI. #5192 @david-lunaIMetricReaderinterface and use it over abstract class #5311MeterProviderOptionsnow provides the more generalIMetricReadertype overMetricReaderMetricReaderin your public interface, consider accepting the more generalIMetricReaderinstead to avoid unintentional breaking changesBasicTracerProvider, the class offered multiple methods to facilitate the creation of exporters and auto-pairing withSpanProcessors.SpanProcessors to the base class constructor when extending_registeredExportershas been removed_getSpanExporterhas been removed_buildExporterFromEnvhas been removedIdGeneratorandRandomIdGenerator#5309 @pichlermarcInstrumentationLibrary#5308 @pichlermarcInstrumentationScopeinsteadInstrumentationLibrarywithInstrumentationScope#5308 @pichlermarcTracer.instrumentationLibrary->Tracer.instrumentationScopeReadableSpan.instrumentationLibrary->ReadableSpan.instrumentationScopeReadableSpanReadableSpan.instrumentationScopeoverReadableSpan.instrumentationLibrary#5308 @pichlermarcReadableSpan.instrumentationScopeoverReadableSpan.instrumentationLibrary#5308 @pichlermarc5.0.4#5145 @david-lunatypescript@<5.0.4typescriptin minor releases. We will only drop support for versions that are older than 2 years.AlwaysOnSamplerhas moved to@opentelemetry/sdk-trace-baseAlwaysOffSamplerhas moved to@opentelemetry/sdk-trace-baseParentBasedSamplerhas moved to@opentelemetry/sdk-trace-baseTraceIdRatioSamplerhas moved to@opentelemetry/sdk-trace-baseattributesobjectDetectedResourceplain objects instead ofnew Resource()BasicTracerProvider#register().OTEL_PROPAGATORSorwindow.OTEL_PROPAGATORSanymore, please pass the propagator toNodeTracerProvider#register()instead.BasicTracerProvider#register()will now fall back to defaults (tracecontextandbaggage)NodeTracerProvider#register().OTEL_PROPAGATORSanymore, please pass the propagator toNodeTracerProvider#register()instead.NodeTracerProvider#register()will now fall back to the defaults (tracecontextandbaggage)NodeSDKfrom@opentelemetry/sdk-node.WebTracerProvider#register().window.OTEL_PROPAGATORSanymore, please pass the propagator toWebTracerProvider#register()instead.WebTracerProvider#register()will now fall back to defaults (tracecontextandbaggage)EXPORTER_FACTORYis not used anymore and has been removedPROPAGATOR_FACTORYis not used anymore and has been removedForceFlushStatewas intended for internal use and has been removedTracerclass was unintentionally exported and has been removedTracer, please useBasicTracerProvider#getTracer(),NodeTracerProvider#getTracer()orWebTracerProvider#getTracer()Tracer, please use theTracertype from@opentelemetry/api^18.19.0 || >=20.6.0. Support for Node.js 14, 16, and early minor versions of 18 and 20 have been dropped. This applies to all packages except the 'api' and 'semantic-conventions' packages. #5395 @trentm@opentelemetry/resourcein favor of@opentelemetry/opentelemetry-browser-detector#5420VERSIONwas an internal constant that was unintentionally exported. It has been removed without replacement.isWrappedhas been removed in favor ofisWrappedfrom@opentelemetry/instrumentationShimWrappedhas been removed in favor ofShimWrappedfrom@opentelemetry/instrumentationhexToBase64was a utility function that is not used by the SDK anymore. It has been removed without replacement.hexToBinarywas a utility function that now internal to@opentelemetry/otlp-tranformer. It has been removed without replacement.baggageUtils.getKeyPariswas an internal utility function that was unintentionally exported. It has been removed without replacement.baggageUtils.serializeKeyPairswas an internal utility function that was unintentionally exported. It has been removed without replacement.baggageUtils.parseKeyPairsIntoRecord,has been removed in favor ofparseKeyPairsIntoRecordbaggageUtils.parsePairKeyValuewas an internal utility function that was unintentionally exported. It has been removed without replacement.TimeOriginLegacyhas been removed without replacement.isAttributeKeywas an internal utility function that was unintentionally exported. It has been removed without replacement.window.OTEL_*is now not supported anymore, please pass configuration options to constructors instead.window.OTEL_*is now not supported anymore, please pass configuration options to constructors instead.Resourceclass toResourceImpland makes it package-privateIResourceinterface toResourceresourceFromAttributesto create aResourcefrom aDetectedAttributesobjectdefaultResourceto create a default resource #5467 @pichlermarcemptyResourceto create an empty resource #5467 @pichlermarcwindow.OTEL_*is now not supported anymoreenvDetectorin browser environments, please migrate to manually creating a resource.ParentBasedAlwaysOnSampleroverAlwaysOnSamplerwhen bogus data is supplied toOTEL_TRACES_SAMPLERgetEnv(),getEnvWithoutDefaults()#5481 @pichlermarcgetEnv()has been replaced bygetStringFromEnv(),getNumberFromEnv(),getBooleanFromEnv(),getStringListFromEnv()getStringFromEnv("OTEL_FOO") ?? "my-default")getEnvWithoutDefaults()has been replaced bygetStringFromEnv(),getNumberFromEnv(),getBooleanFromEnv(),getStringListFromEnv()DEFAULT_ENVIRONMENThas been removed, please inline any defaults from now onENVIRONMENThas been removed without replacementRAW_ENVIRONMENThas been removed without replacementparseEnvironmenthas been removed without replacementBasicTracerProvider#register()to improve tree-shaking #5503 @pichlermarcBasicTracerProvider#register()has been removedpropagation.setGlobalPropagator()from@opentelemetry/apicontext.setGlobalContextManager()from@opentelemetry/api@opentelemetry/api,@opentelemetry/api-logs,@opentelemetry/api-events, and@opentelemetry/semantic-conventions#5456 @david-lunaDEFAULT_ATTRIBUTE_VALUE_LENTGHT_LIMIThas been removed, please useInfinityinsteadDEFAULT_ATTRIBUTE_VALUE_COUNT_LIMIThas been removed, please use128insteadDEFAULT_SPAN_ATTRIBUTE_PER_EVENT_COUNT_LIMIThas been removed, please use128insteadDEFAULT_SPAN_ATTRIBUTE_PER_LINK_COUNT_LIMIThas been removed, please use128instead🚀 (Enhancement)
WebTracerProviderconstructor now does not throw anymore whencontextManagerorpropagatorare passed as extra options to the constructordiagLogLevelFromStringutility #5475 @pichlermarc🐛 (Bug Fix)
url.parsefromnode:url#5390 @chancancodePeriodicExportingMetricReaderwhen there are no metrics to export. #5288 @jacksonweber🏠 (Internal)
InstrumentDescriptortype now extendsMetricDescriptor; moved publicInstrumentTypetype enum into./src/export/MetricData.ts#5277GaugeandMetricAdviceworkaround types in favor of the upstream@opentelemetry/apitypes #5254 @chancancodeBasicTracerProvider._registeredSpanProcessorsprivate property. #5134 @david-lunaBasicTracerProvider.activeSpanProcessorprivate property. #5211 @david-lunamodulecompiler option tonode16. #5347 @david-lunasemverpackage with internal semantic versioning check implementation to get rid ofsemverpackage initialization overhead especially in the AWS Lambda environment during coldstart #5305 @serkan-ozal@opentelemetry/semantic-conventionsdep to allow better de-duplication in installs #5439 @trentmConfiguration
📅 Schedule: Branch creation - "every weekday after 2:00 am before 6:00 am,every weekend" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.
This PR was generated by Mend Renovate. View the repository job log.