Deprecate jaeger exporters. (open-telemetry#2858)

carlosalberto · Dec 20, 2022 · 837415d · 837415d
1 parent a1fbb0b
commit 837415d
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 7 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -11,6 +11,9 @@ release.
 
 ### Traces
 
+- Deprecate jaeger exporter, scheduled for spec removal in July 2023.
+  [#2858](https://github.com/open-telemetry/opentelemetry-specification/pull/2858)
+
 ### Metrics
 
 ### Logs

diff --git a/spec-compliance-matrix.md b/spec-compliance-matrix.md
@@ -334,9 +334,9 @@ Note: Support for environment variables is optional.
 | Event attributes mapping to Annotations                                        |          | +  | +    | +  | +           | +    | +      | +   | +    | +   | +    | +     |
 | Integer microseconds in timestamps                                             |          | N/A| +    |    | +           | +    | -      | +   | +    | +   | +    | +     |
 | **[Jaeger](specification/trace/sdk_exporters/jaeger.md)**                      | Optional | Go  | Java | JS  | Python | Ruby | Erlang | PHP | Rust | C++ | .NET | Swift |
-| [Jaeger Thrift over UDP][jaegerThriftUDP]                                      | *        | +  |      |    | +           | +    | -      | -   | +    | +   | +    | +     |
-| [Jaeger Protobuf via gRPC][jaegerProtobuf]                                     | *        | -  | +    |    | +           | -    | -      | -   |      | -   | -    | -     |
-| [Jaeger Thrift over HTTP][jaegerThriftHTTP]                                    | *        | +  | -    |    | +           | +    | -      | -   | +    | +   | +    | -     |
+| [Jaeger Thrift over UDP][jaegerThriftUDP]                                      | X        | +  |      |    | +           | +    | -      | -   | +    | +   | +    | +     |
+| [Jaeger Protobuf via gRPC][jaegerProtobuf]                                     | X        | -  | +    |    | +           | -    | -      | -   |      | -   | -    | -     |
+| [Jaeger Thrift over HTTP][jaegerThriftHTTP]                                    | X        | +  | -    |    | +           | +    | -      | -   | +    | +   | +    | -     |
 | Service name mapping                                                           |          | +  | +    |    | +           | +    | -      | -   |      | +   | +    | +     |
 | Resource to Process mapping                                                    |          | +  | +    |    | +           | +    | -      | -   | +    | -   | +    | -     |
 | InstrumentationLibrary mapping                                                 |          | +  | +    |    | +           | +    | -      | -   | +    | -   | +    | -     |

diff --git a/specification/sdk-environment-variables.md b/specification/sdk-environment-variables.md
@@ -163,7 +163,14 @@ See [OpenTelemetry Protocol Exporter Configuration Options](./protocol/exporter.
 
 ## Jaeger Exporter
 
-**Status**: [Stable](document-status.md)
+**Status**: [Deprecated](document-status.md)
+
+Jaeger exporter support will be removed from OpenTelemetry in July 2023.
+
+_Note: Jaeger supports the [OpenTelemetry protocol natively][jaeger_otlp] and most users
+should export to Jaeger using OTLP. These environment variables remain here
+only for backwards compatibility and will be removed in a future version. SDKs MAY include
+Jaeger exporters, but Jaeger export is not required._
 
 The `OTEL_EXPORTER_JAEGER_PROTOCOL` environment variable
 MAY by used to specify the transport protocol.
@@ -177,6 +184,7 @@ The value MUST be one of:
 [jaeger_http]: https://www.jaegertracing.io/docs/latest/apis/#thrift-over-http-stable
 [jaeger_grpc]: https://www.jaegertracing.io/docs/latest/apis/#protobuf-via-grpc-stable
 [jaeger_udp]: https://www.jaegertracing.io/docs/latest/apis/#thrift-over-udp-stable
+[jaeger_otlp]: https://www.jaegertracing.io/docs/latest/apis/#opentelemetry-protocol-stable
 
 The default transport protocol SHOULD be `http/thrift.binary` unless
 SDKs have good reasons to choose other as the default

diff --git a/specification/trace/sdk_exporters/jaeger.md b/specification/trace/sdk_exporters/jaeger.md
@@ -7,10 +7,11 @@ The generic transformation [rules specified here](../../common/mapping-to-non-ot
 particular generic transformation rule and the rule in this document contradict
 then the rule in this document MUST be used.
 
-Jaeger accepts spans in two formats:
+Jaeger accepts spans in the following formats:
 
-* Thrift `Batch`, defined in [jaeger-idl/.../jaeger.thrift](https://github.com/jaegertracing/jaeger-idl/blob/master/thrift/jaeger.thrift), accepted via UDP or HTTP
-* Protobuf `Batch`, defined in [jaeger-idl/.../model.proto](https://github.com/jaegertracing/jaeger-idl/blob/master/proto/api_v2/model.proto), accepted via gRPC
+* OpenTelemetry Protocol (OTLP), defined in [opentelemetry-proto](https://github.com/open-telemetry/opentelemetry-proto)
+* Thrift `Batch`, defined in [jaeger-idl/.../jaeger.thrift](https://github.com/jaegertracing/jaeger-idl/blob/main/thrift/jaeger.thrift), accepted via UDP or HTTP
+* Protobuf `Batch`, defined in [jaeger-idl/.../model.proto](https://github.com/jaegertracing/jaeger-idl/blob/main/proto/api_v2/model.proto), accepted via gRPC
 
 See also: