-
-
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
Unable to create clean custom OpenTelemetry setup combined with Sentry #12678
Comments
Thank you for opening this, and for the repro app! We will look into this to figure out what's going on. |
Hi, sorry for the wait. The reproduction repo is private so I cannot look at it, but I am still gonna try to be useful here. As a first step I would recommend upgrading to the latest SDK version. We changed a few things around how we do tracing in the Next.js SDK recently. I don't think it is gonna solve the sampling decision stuff entirely, but it may help. Additionally, I'd like to better understand the specific issues you are facing. You mentioned that Sentry takes over sampling decisions, would you mind elaborating with specifics or share some examples? Basically, I want to get an understanding of the status quo and where we want to get to. |
This issue has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you remove the label "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
I'm having this issue with my Next.js v.15 application, any updates on this? You can check the code here: https://github.com/anyulled/my-portfolio-website/blob/main/src/instrumentation.ts |
@anyulled Please check our guide on how to combine Sentry with your own opentelemetry setup: https://docs.sentry.io/platforms/javascript/guides/node/opentelemetry/custom-setup/ Seems like you didn't do that from looking at your setup. |
Is there an existing issue for this?
How do you use Sentry?
Sentry Saas (sentry.io)
Which SDK are you using?
@sentry/nextjs
SDK Version
8.12.0
Framework Version
Next 14.0.4
Link to Sentry event
No response
SDK Setup/Reproduction Example
No response
Steps to Reproduce
I want to run a custom OpenTelemetry installation next to using Sentry, but Sentry's tracing / performance features interfere with my own setup. I still want to use Sentry tracing, in addition to my own setup.
I followed the instructions as per https://docs.sentry.io/platforms/javascript/guides/nextjs/tracing/instrumentation/opentelemetry/. Additionally, I set up a repo demonstrating my setup. See
sentry-and-custom-otel
branch here https://github.com/tnolet/tims-otel-app/tree/sentry-and-custom-otelsentry-and-custom-otel
branch.npm install
npm run dev
export SENTRY_DSN="some DSN"
localhost:3000
You will now see some debug logging from Sentry showing a trace was recorded and flushed, e.g.
Note that you will NOT see the
console.log
statements that are added to the custom OTel setup as perinstrumentation.ts
.instrumentation.ts
and remove or comment out the Sentry relevant part. Just leave theregisterOtel()
function.npm run dev
curl
command to trigger the custom OTel setup with the customer sampler.The logs should now show something like:
Expected Result
I can run a separate, custom OTel setup with distinct sampling rules and exporters next to a Sentry installation.
Actual Result
See the reproduction steps. The result is I can run either Sentry or my own custom OTel setup, not both.
ref: #12191 (reply in thread)
The text was updated successfully, but these errors were encountered: