-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Add Open Telemetry Exporter #11386
base: master
Are you sure you want to change the base?
Add Open Telemetry Exporter #11386
Conversation
Changes in OpenAPI specification found. Expand to see details.What's Changed
|
Wouldn't auto-instrumentation be more useful and flexible for everyone? The hardcoded instrumentation seems rather overkill for most use cases. |
Never really thought of auto instrumentation. If someone is hosting in containers though it'll make it hard to do, so having this option at least gives the Hoster a way to get decent metrics etc. I plan to try and add some tracing and better metrics so this is just a start |
This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged. |
Add OTEL exporter options. For anyone that self hosts and has a collector, it would be useful for thme to get the metrics and tracing to help monitor performance.