-
Notifications
You must be signed in to change notification settings - Fork 1.9k
chore: upgrade OpenTelemetry deps and refresh tracing setup in client #28268
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR upgrades OpenTelemetry dependencies to their latest versions and updates the tracing setup to use the new SDK APIs introduced in @opentelemetry/sdk-trace-base@2
.
- Bumps OpenTelemetry packages to newer versions and unifies them through workspace catalog
- Migrates from deprecated APIs to new patterns (Resource constructor → resourceFromAttributes, provider.register() → trace.setGlobalTracerProvider())
- Updates semantic convention imports to use the new ATTR_* naming convention
Reviewed Changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
pnpm-workspace.yaml | Adds new OpenTelemetry package versions to workspace catalog |
packages/query-plan-executor/package.json | Migrates to use catalog reference for sdk-trace-base |
packages/instrumentation/README.md | Updates documentation examples to use new OpenTelemetry v2 APIs |
packages/client/tests/functional/tracing/tests.ts | Migrates test setup to new API patterns and fixes span property access |
packages/client/tests/functional/tracing-no-sampling/tests.ts | Updates tracing setup with new APIs |
packages/client/tests/functional/tracing-filtered-spans/tests.ts | Updates tracing setup with new APIs |
packages/client/tests/functional/tracing-disabled/tests.ts | Updates tracing setup with new APIs |
packages/client/package.json | Bumps OpenTelemetry dependencies to latest versions |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
size-limit report 📦
|
63005f3
to
7f1c927
Compare
- Bump `@opentelemetry/*` packages and unify them using workspace catalog. - Align tracing functional tests with the latest SDK APIs. - Update `@prisma/instrumentation` README examples to match the new configuration patterns in `@opentelemetry/sdk-trace-base@2`.
7f1c927
to
30ce209
Compare
@opentelemetry/*
packages.@prisma/instrumentation
README examples to match the new configuration patterns in@opentelemetry/sdk-trace-base@2
.Part of https://linear.app/prisma-company/issue/TML-1317/add-qc-support-in-local-ppg-with-accelerate-api