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

Migrating from OpenTelemetry Jaeger exporter to OTLP exporter #685

Closed
ppatierno opened this issue Nov 8, 2022 · 0 comments · Fixed by #687
Closed

Migrating from OpenTelemetry Jaeger exporter to OTLP exporter #685

ppatierno opened this issue Nov 8, 2022 · 0 comments · Fixed by #687
Assignees
Milestone

Comments

@ppatierno
Copy link
Member

ppatierno commented Nov 8, 2022

As for the operator with strimzi/strimzi-kafka-operator#7592, we should migrate from OpenTelemetry Jaeger exporter to the OTLP exporter.

Referring to strimzi/strimzi-kafka-operator#7570, the current Jaeger Exporter, used by default when OpenTelemetry is enabled, doesn't have support for TLS via SDK environment variables.
At same time, the Jaeger Exporter is going to be deprecated (details open-telemetry/opentelemetry-specification#2858) and the OpenTelemetry community is pushing for using the OTLP exporter which has TLS support and it's anyway usable with a Jaeger backend which has support for OTLP protocol since 1.35 version.
We should migrate to OTLP Exporter as soon as possible, which means:

  • removing the Jaeger Exporter dependency opentelemetry-exporter-jaeger and replacing it with the opentelemetry-exporter-otlp
  • making the new OTLP Exporter the default one by setting the OTEL_TRACES_EXPORTER=otlp in our images script or just leaving that env var empty because it will be the default behavior for OpenTelemetry

The migration should be done before the next release, so it also means that the #684 can be closed and documentation doesn't need any statement around TLS.

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 a pull request may close this issue.

1 participant