-
Notifications
You must be signed in to change notification settings - Fork 331
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
Migrate from OpenCensus to OpenTelemetry #855
Comments
Looks like OpenTelemetry Go SDK is in beta now: https://github.com/open-telemetry/opentelemetry-go |
This issue is stale because it has been open for 90 days with no |
/lifecycle frozen |
/unassign |
/unassign @jjzeng-seattle |
FYI - there's a bridge for OpenCensus => OpenTelemetry So we can use the OTel exporters and slowly transition the API usage over https://github.com/open-telemetry/opentelemetry-go/tree/main/bridge/opencensus#user-journey I think tracing would be the best POC to test this out with |
FYI - Bug in metrics/otelhttp handler causes high cardinality |
When performing the migration we should consider how we name the service - The suggestion here is that hostname is not a good one because it changes as pods come up and down - knative/serving#14075 |
Otel go sdk i snow stable wrt metrics. See knative/serving#14486 for pointers. We should define a plan on how to work on this one. |
OpenCensus & OpenTracing have merged to form OpenTelemetry (see https://github.com/census-instrumentation/opencensus-go & https://medium.com/opentracing/a-roadmap-to-convergence-b074e5815289).
Knative should to move to instrumenting metrics and traces with OpenTelemetry instead of OpenCensus ahead of OpenCensus getting deprecated.
The text was updated successfully, but these errors were encountered: