-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Migrate to @fastify/otel
instrumentation
#15130
Comments
is there something that speaks against just switching out the instrumentation right now? |
@fastify/otel
instrumentation
hmm, this only supports fastify 5+ 😬 |
See: fastify/otel#10 |
After talking about this some more, our idea is to:
We can use this blueprint for other similar situations too, in the future! This way, we continue to support fastify 3+, but can stay on the latest version of the official instrumentation and get improvements from there. |
Description
Fastify has taken ownership over the fastify Otel instrumentation and migrated the integration out of the opentelemetry-js-contrib repo into their own repo.
Once open-telemetry/opentelemetry-js-contrib#2652 lands, the current instrumentation, which we are using, will be disabled by default and removed completely by March 25, 2025.
Solution Brainstorm
Migrating to @fastify/otel is inevitable, but if integration proves difficult we have to specifically enable the integration,
The text was updated successfully, but these errors were encountered: