Skip to content
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

OpenTelemetry Support #2066

Closed
smeubank opened this issue Nov 25, 2022 · 4 comments · Fixed by #2453
Closed

OpenTelemetry Support #2066

smeubank opened this issue Nov 25, 2022 · 4 comments · Fixed by #2453
Assignees
Labels
Feature New feature or request

Comments

@smeubank
Copy link
Member

smeubank commented Nov 25, 2022

Problem Statement

OpenTelemetry has growing popularity in the observability space as one of the top CNCF open standard for APM.

It's data model differs from Sentry so it cannot be natively ingested. But many OpenTelemetry users would like to send their OTEL data to Sentry.

Solution Brainstorm

Details

dogfooding

The cartservice on our OTel Demo is .NET

https://github.com/getsentry/opentelemetry-demo/tree/main/src/cartservice

Reference

Demo project from OTEL, we created a fork to instrument with Sentry SDKs, to test and demo the implementation. Projects are in the Sentry SDKs org

For more context, these PRs are not exhaustive list of the work done on other SDKs

@smeubank smeubank added Feature New feature or request Platform: .NET labels Nov 25, 2022
@mattjohnsonpint
Copy link
Contributor

Related: #334

@bruno-garcia
Copy link
Member

cartservice in the demo can be instrumented for testing: https://github.com/getsentry/opentelemetry-demo/tree/main/src/cartservice

@source-studio
Copy link

@mattjohnsonpint - could I ask if there is an ETA for this feature? The .NET OpenTelemetry implementation is reasonably mature now, so it shouldn't present too many roadblocks. Given the popularity of .NET, I'm surprised Sentry deprioritised this implementation, relative to the other supported languages.

@jamescrosswell
Copy link
Collaborator

@source-studio sorry no one got back to you sooner. I'll see if I can pick this one up... I haven't yet gathered enough context to be able to estimate how long this might take but you should see some updates/activity shortly.

@mattjohnsonpint mattjohnsonpint removed their assignment Jun 20, 2023
@jamescrosswell jamescrosswell linked a pull request Jun 21, 2023 that will close this issue
1 task
@jamescrosswell jamescrosswell self-assigned this Jun 21, 2023
@jamescrosswell jamescrosswell removed a link to a pull request Jun 21, 2023
1 task
@smeubank smeubank linked a pull request Jun 26, 2023 that will close this issue
1 task
@jamescrosswell jamescrosswell removed a link to a pull request Jun 28, 2023
1 task
@jamescrosswell jamescrosswell linked a pull request Jun 28, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New feature or request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants