Skip to content

Commit

Permalink
Remove testing for Jaeger exporter (#3509)
Browse files Browse the repository at this point in the history
  • Loading branch information
ocelotl authored Nov 6, 2023
1 parent fcaa6c0 commit 2a8d4ed
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 14 deletions.
2 changes: 2 additions & 0 deletions exporter/opentelemetry-exporter-jaeger-proto-grpc/README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ OpenTelemetry Jaeger Protobuf Exporter
Since v1.35, the Jaeger supports OTLP natively. Please use the OTLP exporter instead.
Support for this exporter will end July 2023.

This package is no longer being tested.

|pypi|

.. |pypi| image:: https://badge.fury.io/py/opentelemetry-exporter-jaeger-proto-grpc.svg
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ classifiers = [
"Typing :: Typed",
]
dependencies = [
"protobuf ~= 3.20.0",
"googleapis-common-protos ~= 1.52, < 1.60.0",
"grpcio >= 1.0.0, < 2.0.0",
"opentelemetry-api ~= 1.3",
Expand Down
2 changes: 2 additions & 0 deletions exporter/opentelemetry-exporter-jaeger-thrift/README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ OpenTelemetry Jaeger Thrift Exporter
Since v1.35, the Jaeger supports OTLP natively. Please use the OTLP exporter instead.
Support for this exporter will end July 2023.

This package is no longer being tested.

|pypi|

.. |pypi| image:: https://badge.fury.io/py/opentelemetry-exporter-jaeger-thrift.svg
Expand Down
2 changes: 2 additions & 0 deletions exporter/opentelemetry-exporter-jaeger/README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ OpenTelemetry Jaeger Exporter
Since v1.35, the Jaeger supports OTLP natively. Please use the OTLP exporter instead.
Support for this exporter will end July 2023.

This package is no longer being tested.

|pypi|

.. |pypi| image:: https://badge.fury.io/py/opentelemetry-exporter-jaeger.svg
Expand Down
13 changes: 0 additions & 13 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -27,12 +27,6 @@ envlist =
py3{7,8,9,10,11}-opentelemetry-opencensus-shim
; opencensus-shim intentionally excluded from pypy3 (grpcio install fails)

py3{7,8,9,10,11}-opentelemetry-exporter-jaeger-combined

py3{7,8,9,10,11}-opentelemetry-exporter-jaeger-proto-grpc

py3{7,8,9,10,11}-opentelemetry-exporter-jaeger-thrift

py3{7,8,9,10,11}-opentelemetry-exporter-opencensus
; exporter-opencensus intentionally excluded from pypy3

Expand Down Expand Up @@ -106,9 +100,6 @@ changedir =
opentracing-shim: shim/opentelemetry-opentracing-shim/tests
opencensus-shim: shim/opentelemetry-opencensus-shim/tests

exporter-jaeger-combined: exporter/opentelemetry-exporter-jaeger/tests
exporter-jaeger-proto-grpc: exporter/opentelemetry-exporter-jaeger-proto-grpc/tests
exporter-jaeger-thrift: exporter/opentelemetry-exporter-jaeger-thrift/tests
exporter-opencensus: exporter/opentelemetry-exporter-opencensus/tests
exporter-otlp-proto-common: exporter/opentelemetry-exporter-otlp-proto-common/tests
exporter-otlp-combined: exporter/opentelemetry-exporter-otlp/tests
Expand Down Expand Up @@ -157,10 +148,6 @@ commands_pre =
exporter-otlp-proto-http: pip install {toxinidir}/exporter/opentelemetry-exporter-otlp-proto-common
exporter-otlp-proto-http: pip install {toxinidir}/exporter/opentelemetry-exporter-otlp-proto-http[test]

exporter-jaeger-combined: pip install {toxinidir}/exporter/opentelemetry-exporter-jaeger-proto-grpc {toxinidir}/exporter/opentelemetry-exporter-jaeger-thrift {toxinidir}/exporter/opentelemetry-exporter-jaeger
exporter-jaeger-proto-grpc: pip install {toxinidir}/exporter/opentelemetry-exporter-jaeger-proto-grpc
exporter-jaeger-thrift: pip install {toxinidir}/exporter/opentelemetry-exporter-jaeger-thrift

opentracing-shim: pip install {toxinidir}/opentelemetry-sdk
opentracing-shim: pip install {toxinidir}/shim/opentelemetry-opentracing-shim

Expand Down

0 comments on commit 2a8d4ed

Please sign in to comment.