Remove references to Jaeger exporters

open-telemetry · Nov 23, 2023 · 91f91eb · 91f91eb
1 parent ed98861
commit 91f91eb
Show file tree

Hide file tree

Showing 10 changed files with 3 additions and 98 deletions.
diff --git a/.flake8 b/.flake8
@@ -23,10 +23,6 @@ exclude =
   venv*/
   target
   __pycache__
-  exporter/opentelemetry-exporter-jaeger-proto-grpc/src/opentelemetry/exporter/jaeger/proto/grpc/gen/
-  exporter/opentelemetry-exporter-jaeger-proto-grpc/build/*
-  exporter/opentelemetry-exporter-jaeger-thrift/src/opentelemetry/exporter/jaeger/thrift/gen/
-  exporter/opentelemetry-exporter-jaeger-thrift/build/*
   exporter/opentelemetry-exporter-zipkin-proto-http/src/opentelemetry/exporter/zipkin/proto/http/v2/gen/
   docs/examples/opentelemetry-example-app/src/opentelemetry_example_app/grpc/gen/
   docs/examples/opentelemetry-example-app/build/*

diff --git a/docs-requirements.txt b/docs-requirements.txt
@@ -24,7 +24,3 @@ wrapt>=1.0.0,<2.0.0
 # See https://github.com/pallets/markupsafe/issues/282
 # breaking change introduced in markupsafe causes jinja, flask to break
 markupsafe==2.0.1
-
-# Jaeger generated protobufs do not currently support protobuf 4.x. This can be removed once
-# they're regenerated.
-protobuf~=3.19
diff --git a/docs/exporter/jaeger/jaeger.rst b/docs/exporter/jaeger/jaeger.rst
diff --git a/docs/getting_started/jaeger_example.py b/docs/getting_started/jaeger_example.py
diff --git a/eachdist.ini b/eachdist.ini
@@ -24,9 +24,6 @@ packages=
     opentelemetry-exporter-otlp-proto-grpc
     opentelemetry-exporter-otlp-proto-http
     opentelemetry-exporter-otlp
-    opentelemetry-exporter-jaeger-thrift
-    opentelemetry-exporter-jaeger-proto-grpc
-    opentelemetry-exporter-jaeger
     opentelemetry-api
 
 [prerelease]

diff --git a/opentelemetry-sdk/tests/test_configurator.py b/opentelemetry-sdk/tests/test_configurator.py
@@ -826,10 +826,10 @@ def test_otlp_exporter_conflict(self):
             )
         assert len(logs_context.output) == 1
 
-    @patch.dict(environ, {"OTEL_TRACES_EXPORTER": "jaeger,zipkin"})
+    @patch.dict(environ, {"OTEL_TRACES_EXPORTER": "zipkin"})
     def test_multiple_exporters(self):
         self.assertEqual(
-            sorted(_get_exporter_names("traces")), ["jaeger", "zipkin"]
+            sorted(_get_exporter_names("traces")), ["zipkin"]
         )
 
     @patch.dict(environ, {"OTEL_TRACES_EXPORTER": "none"})

diff --git a/pyproject.toml b/pyproject.toml
@@ -9,8 +9,6 @@ exclude = '''
     .venv.*|
     target.*|
     .*/build/lib/.*|
-    exporter/opentelemetry-exporter-jaeger-proto-grpc/src/opentelemetry/exporter/jaeger/proto/grpc/gen|
-    exporter/opentelemetry-exporter-jaeger-thrift/src/opentelemetry/exporter/jaeger/thrift/gen|
     exporter/opentelemetry-exporter-zipkin-proto-http/src/opentelemetry/exporter/zipkin/proto/http/v2/gen|
     opentelemetry-proto/src/opentelemetry/proto/.*/.*|
     scripts

diff --git a/rationale.md b/rationale.md
@@ -28,7 +28,7 @@ Public portions of the SDK (constructors, configuration, end-user interfaces) mu
 
 ## Core components
 
-Core components refer to the set of components which are required as per the spec. This includes API, SDK, propagators (B3 and Jaeger) and exporters which are required by the specification. These exporters are OTLP, Jaeger and Zipkin.
+Core components refer to the set of components which are required as per the spec. This includes API, SDK, propagators (B3 and Jaeger) and exporters which are required by the specification. These exporters are OTLP, and Zipkin.
 
 ## Mature or stable Signals
 

diff --git a/scripts/coverage.sh b/scripts/coverage.sh
@@ -32,8 +32,6 @@ cov opentelemetry-sdk
 cov exporter/opentelemetry-exporter-datadog
 cov instrumentation/opentelemetry-instrumentation-flask
 cov instrumentation/opentelemetry-instrumentation-requests
-cov exporter/opentelemetry-exporter-jaeger-proto-grpc
-cov exporter/opentelemetry-exporter-jaeger-thrift
 cov instrumentation/opentelemetry-instrumentation-opentracing-shim
 cov util/opentelemetry-util-http
 cov exporter/opentelemetry-exporter-zipkin

diff --git a/tox.ini b/tox.ini
@@ -213,9 +213,6 @@ commands_pre =
   python -m pip install -e {toxinidir}/tests/opentelemetry-test-utils[test]
   python -m pip install -e {toxinidir}/shim/opentelemetry-opentracing-shim[test]
   python -m pip install -e {toxinidir}/shim/opentelemetry-opencensus-shim[test]
-  python -m pip install -e {toxinidir}/exporter/opentelemetry-exporter-jaeger-proto-grpc[test]
-  python -m pip install -e {toxinidir}/exporter/opentelemetry-exporter-jaeger-thrift[test]
-  python -m pip install -e {toxinidir}/exporter/opentelemetry-exporter-jaeger[test]
   python -m pip install -e {toxinidir}/exporter/opentelemetry-exporter-opencensus[test]
   python -m pip install -e {toxinidir}/exporter/opentelemetry-exporter-otlp-proto-common[test]
   python -m pip install -e {toxinidir}/exporter/opentelemetry-exporter-otlp-proto-grpc[test]