OTEL tracing in serverless driver - does it exist yet?
Title basically says it all! I want to try and find a way to do OTEL tracing for a request lifecycle including queries. Curious if this has been implemented yet?
10 Replies
quickest-silverOP•15mo ago
Also it might be worth potentially making a role for JS not node? Tends to be quite different
sunny-green•15mo ago
Unfortunately this hasn't been implemented yet.
Do you mind creating an issue on the serverless driver repo and including all the details? This way we can keep track of your request
https://github.com/neondatabase/serverless/issues/new?template=Blank+issue
GitHub
Build software better, together
GitHub is where people build software. More than 100 million people use GitHub to discover, fork, and contribute to over 420 million projects.
sunny-green•15mo ago
Hmm, why a JS role?
quickest-silverOP•15mo ago
So imo node and a serverless engine like you get on vercel are so different that you might have issues that affect one but not the other (for example issues re Buffer or something weird like that)
Shall do, thanks
quickest-silverOP•15mo ago
GitHub
OTEL tracing in driver · Issue #100 · neondatabase/serverless
Support for OpenTelemetry in the serverless driver for transactions/individual queries would be useful to allow for full tracing from request to response of all queries my application is performing
quickest-silverOP•15mo ago
I actually kinda really want this feature so I'll clone down the repo, download a few deps, and take a shot in the plane (I'm travelling soon) 🙂
sunny-green•15mo ago
That would be very cool. (safe travels! 😄)
quickest-silverOP•15mo ago
I have a patch on my hard drive
need to figure out testing this then should be good
btw, is there any reason that the API needs to use json? I notice a bigint todo that could totally be fixed with the usage of msgpack, and give your backend more info about intent
(to be clear, not advocating for yeeting json content type support, just whatever serialization package you use likely trivially supports msgpack too)
^ was getting test errors, forgot a return, seems to be passing now
quickest-silverOP•15mo ago
GitHub
Add OTEL support by IAmJSD · Pull Request #102 · neondatabase/serve...
Adds opentelemetry support to the serverless driver. I need to do further testing, but would appreciate testing/feedback and this seems to pass the latency tests so dropping here
yelping-magenta•7mo ago
Would love to see this merged, but the PR has gone stale. There are conflicts. @JustSomeDev do you think you can update the PR?