elastic / elastic/elastic-otel-node

consider adding @fastify/otel instrumentation

Open
#1,386 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
12
Forks
22
Avg merge
16h 54m
Merged PRs (30d)
5

Description

- Soonish instr-fastify will no longer be getting new versions (and will be removed from auto-instrumentations-node). We should drop it.
- Consider adding `@fastify/otel`, the blessed community replacement.
- We could have the new one get enabled via explicit `fastify` in `OTEL_NODE_ENABLED_INSTRUMENTATIONS`, or require explicit clear `@fastify/otel` name. I lean towards the latter for clarity. Having "fastify" in there "just work" migh be misleading because the instr is different.
- If we add `@fastify/otel` are there going to be dep duplicates to worry about? E.g. otel/instr version. We could perhaps use overrides... or just engage upstream.
- At time of writing it has the latest `@opentelemetry/instrumentation` on main, but hasn't had recent releases.

Contributor guide

Open the contributing guide

Research direction

Start by tracing how instr-fastify is currently registered and how OTEL_NODE_ENABLED_INSTRUMENTATIONS selects instrumentation. Check the package dependency versions and upstream release status for @fastify/otel; done means the old instrumentation is removed and the chosen explicit-name behavior works without unresolved duplicate dependencies.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, nodejs
Domain
backend, observability
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.