opentelemetry-specification icon indicating copy to clipboard operation
opentelemetry-specification copied to clipboard

Published 20 hours ago • verified publisher icon open-telemetry

Deprecate jaeger exporters.

Open breedx-splk opened this issue 3 years ago • 1 comments
trafficstars

Earlier this year, jaeger acquired native support for trace ingest via OTLP. Additionally, the language-specific client libraries that contained the jaeger thrift exporter implementations were EOLed via archival. At the top of the READMEs for those archived clients, Jaeger recommends switching to OpenTelemetry.

Since the client repos were archived, they will not receive security updates. Otel languages whose jaeger exporters depend on those clients will therefore need to reimplement the exporters or drop support for them. Because jaeger is actively urging users to switch to OpenTelemetry clients, I think that deprecation makes sense.

Continued support for jaeger exporters is a maintenance burden for otel language maintainers. Furthermore, the existence of so many exporter choices often serves to confuse users (eg. "I'm running jaeger and there's an otel jaeger exporter, so I should use that one?") and makes understanding otel <-> jaeger interoperation more difficult.

Related issue: https://github.com/open-telemetry/opentelemetry-specification/issues/2859

breedx-splk avatar Oct 08 '22 00:10 breedx-splk

cc @yurishkuro

carlosalberto avatar Oct 10 '22 14:10 carlosalberto

@yurishkuro Ping

carlosalberto avatar Oct 14 '22 14:10 carlosalberto

@carlosalberto discussion still ongoing in https://github.com/open-telemetry/opentelemetry-specification/issues/2859

yurishkuro avatar Oct 14 '22 15:10 yurishkuro

This PR was marked stale due to lack of activity. It will be closed in 7 days.

github-actions[bot] avatar Oct 28 '22 03:10 github-actions[bot]

Definitely not stale. Finishing up blog post here https://github.com/open-telemetry/opentelemetry.io/pull/1892 and looking into cross posting with Jaeger blog.

breedx-splk avatar Oct 31 '22 15:10 breedx-splk

This PR was marked stale due to lack of activity. It will be closed in 7 days.

github-actions[bot] avatar Nov 11 '22 03:11 github-actions[bot]

This PR was marked stale due to lack of activity. It will be closed in 7 days.

github-actions[bot] avatar Nov 25 '22 03:11 github-actions[bot]

Approving assuming that the PR will switch to Ready for Review after the communications are done.

Yeah, and I think we're close...just need some additional input on deprecation period length on #2859.

breedx-splk avatar Nov 30 '22 23:11 breedx-splk

We are almost ready to merge this one. I'd like a final review cycle by @yurishkuro and we are good to go.

carlosalberto avatar Dec 19 '22 23:12 carlosalberto