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

use OpenTelemetry with Google Trace #9656

Closed
christopheblin opened this issue Jan 18, 2023 · 6 comments
Closed

use OpenTelemetry with Google Trace #9656

christopheblin opened this issue Jan 18, 2023 · 6 comments
Assignees

Comments

@christopheblin
Copy link

In my code, I am currently using builder.Services.AddGoogleDiagnosticsForAspNetCore() as documetned in https://cloud.google.com/dotnet/docs/reference/Google.Cloud.Diagnostics.AspNetCore3/latest

Now, I'd like to switch to a more generic Opentelemetry approach (like very well described with zipkin in https://craftbakery.dev/try-zipkin/)

I see that other languages should directly use Open Telemetry (https://cloud.google.com/trace/docs/setup#recommended_client_libraries)

=> is there a chance that we can already switch to Open Telemetry with Asp.NET Core ?
If yes, is there a chance to have some pointers ?

@jskeet
Copy link
Collaborator

jskeet commented Jan 18, 2023

Currently, no - but we're looking at this, as captured in #8366.

Assigning to @Rishabh-V to see if there's more to this issue than I can spot (as he's been looking at this) but I suspect it'll be best if you (@christopheblin) subscribe to #8366 and we keep that as the single source of updates for OpenTelemetry.

@Rishabh-V
Copy link
Contributor

Hi @christopheblin, There is an option to use OpenTelemetry with ASP.NET Core and Google Cloud Trace.

  • Leverage the OpenTelemetry packages with ASP.NET Core as described in this getting started guide in OpenTelemetry website. This should help set-up OpenTelemetry with your ASP.NET Core app.
  • The exporter shown in the guide is a Console Exporter. To export the data to Google Cloud Trace, use the community written StackDriver exporter described here

These steps should get you started. However, we can't actually recommend these packages at the moment, partly as they're third-party packages we're not responsible for, and partly because they're pre-releases.

@christopheblin
Copy link
Author

@jskeet @Rishabh-V thanks for the detailed answers, I already follow #8366 so I'm glad it will be included there ! this is such a pleasure to work with you ❤️

I'll investigate the solution proposed by @Rishabh-V next week and report here

@jskeet
Copy link
Collaborator

jskeet commented Jan 19, 2023

Great, thanks. I'll close this issue now, so we can keep everything on the other issue.

@jskeet jskeet closed this as completed Jan 19, 2023
@rajiv36
Copy link

rajiv36 commented Aug 14, 2023

@jskeet @Rishabh-V thanks for the detailed answers, I already follow #8366 so I'm glad it will be included there ! this is such a pleasure to work with you ❤️

I'll investigate the solution proposed by @Rishabh-V next week and report here

Hi Christopheblin, Did you give it a try. Wondering if you evaluated it(telemetry dot net with StackDriver exporter ) as I wanted to give it a try.

@christopheblin
Copy link
Author

@rajiv36 no I did not dig into OpenTelemetry, in the end, I used #9948

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants