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

Added OTEL for tracing, metrics, logging (Console and OTLP exporters) #384

Merged
merged 16 commits into from
Mar 19, 2024

Conversation

napalm684
Copy link
Contributor

@napalm684 napalm684 commented Feb 22, 2024

Foundation for #136

Added Grafana setup utilizing Tempo for tracing (backed by Prometheus), Prometheus for metrics and Loki for logging. Enabled features to correlate traces, metrics and logs within Grafana. All services added to:

https://github.com/FritzAndFriends/TagzApp/blob/a1fdd75f314bbcee7d6d10ccafa278ec6545a548/docker-compose.local.yml

until @csharpfritz figures out what he wants to do with them going forward. Simply run docker compose -f ./docker-compose.local.yml up and the observability service stack will spin up. Run TagzApp.Blazor pointed at otlp exporter for traces, logs and metrics and let OTEL collector forward to the appropriate places/formats.

Note: The newly announced .NET 8 dashboards are pre-installed and wired up to prometheus (see here):

https://devblogs.microsoft.com/dotnet/introducing-aspnetcore-metrics-and-grafana-dashboards-in-dotnet-8/

@napalm684
Copy link
Contributor Author

Started a blog post series related to this work. Part 1 up - https://shawn.vause.us/posts/opentelemetry-dotnet/

@napalm684
Copy link
Contributor Author

Logs are here @csharpfritz
image

Something doesn't seem to be working though with the trace to logs feature I will have to look into that. You should be able to explore the tempo traces and have a link on the trace/span to jump to logs.

@napalm684
Copy link
Contributor Author

Scratch that it is here:
image

@napalm684
Copy link
Contributor Author

@csharpfritz what would you like to see so we can get this merged up?

@csharpfritz
Copy link
Contributor

csharpfritz commented Mar 4, 2024 via email

@napalm684
Copy link
Contributor Author

napalm684 commented Mar 19, 2024

Initial Dashboard:
image
Known Issues:

  • Query not handling top 5 posters over a chunk of time (top 5 posters for the life of the data currently)

@csharpfritz csharpfritz merged commit 011bf8f into FritzAndFriends:main Mar 19, 2024
2 checks passed
@napalm684 napalm684 deleted the dev branch March 22, 2024 01:53
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

Successfully merging this pull request may close these issues.

None yet

2 participants