Skip to content

Commit

Permalink
Remove [test] package from opentelemetry-exporter-zipkin-json (open-t…
Browse files Browse the repository at this point in the history
  • Loading branch information
ocelotl committed Mar 19, 2024
1 parent d5f4946 commit 799d259
Show file tree
Hide file tree
Showing 3 changed files with 24 additions and 8 deletions.
3 changes: 0 additions & 3 deletions exporter/opentelemetry-exporter-zipkin/pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,6 @@ dependencies = [
"opentelemetry-exporter-zipkin-proto-http == 1.24.0.dev",
]

[project.optional-dependencies]
test = []

[project.entry-points.opentelemetry_traces_exporter]
zipkin = "opentelemetry.exporter.zipkin.proto.http:ZipkinExporter"

Expand Down
22 changes: 22 additions & 0 deletions exporter/opentelemetry-exporter-zipkin/test-requirements.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
asgiref==3.7.2
attrs==23.2.0
Deprecated==1.2.14
flaky==3.7.0
importlib-metadata==6.11.0
iniconfig==2.0.0
packaging==23.2
pluggy==1.4.0
py==1.11.0
py-cpuinfo==9.0.0
pytest==7.1.3
pytest-benchmark==4.0.0
tomli==2.0.1
typing_extensions==4.10.0
wrapt==1.16.0
zipp==3.17.0
-e opentelemetry-api
-e exporter/opentelemetry-exporter-zipkin-json
-e exporter/opentelemetry-exporter-zipkin-proto-http
-e opentelemetry-sdk
-e opentelemetry-semantic-conventions
-e exporter/opentelemetry-exporter-zipkin
7 changes: 2 additions & 5 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -141,9 +141,7 @@ commands_pre =

exporter-prometheus: pip install -r {toxinidir}/exporter/opentelemetry-exporter-prometheus/test-requirements.txt

exporter-zipkin-combined: pip install {toxinidir}/exporter/opentelemetry-exporter-zipkin-json
exporter-zipkin-combined: pip install {toxinidir}/exporter/opentelemetry-exporter-zipkin-proto-http
exporter-zipkin-combined: pip install {toxinidir}/exporter/opentelemetry-exporter-zipkin
exporter-zipkin-combined: pip install -r {toxinidir}/exporter/opentelemetry-exporter-zipkin/test-requirements.txt

exporter-zipkin-proto-http: pip install -r {toxinidir}/exporter/opentelemetry-exporter-zipkin-proto-http/test-requirements.txt

Expand Down Expand Up @@ -214,7 +212,6 @@ commands_pre =
pip install -r {toxinidir}/opentelemetry-api/test-requirements.txt
pip install -r {toxinidir}/opentelemetry-sdk/test-requirements.txt
pip install -r {toxinidir}/opentelemetry-semantic-conventions/test-requirements.txt
python -m pip install -e {toxinidir}/opentelemetry-proto[test]
pip install -r {toxinidir}/opentelemetry-proto/test-requirements.txt
pip install -r {toxinidir}/shim/opentelemetry-opentracing-shim/test-requirements.txt
pip install -r {toxinidir}/shim/opentelemetry-opencensus-shim/test-requirements.txt
Expand All @@ -227,9 +224,9 @@ commands_pre =
pip install -r {toxinidir}/exporter/opentelemetry-exporter-prometheus/test-requirements.txt
pip install -r {toxinidir}/exporter/opentelemetry-exporter-zipkin-proto-http/test-requirements.txt
pip install -r {toxinidir}/exporter/opentelemetry-exporter-zipkin-json/test-requirements.txt
python -m pip install -e {toxinidir}/exporter/opentelemetry-exporter-zipkin[test]
pip install -r {toxinidir}/propagator/opentelemetry-propagator-b3/test-requirements.txt
pip install -r {toxinidir}/propagator/opentelemetry-propagator-jaeger/test-requirements.txt
pip install -r {toxinidir}/exporter/opentelemetry-exporter-zipkin/test-requirements.txt
# Pin protobuf version due to lint failing on v3.20.0
# https://github.com/protocolbuffers/protobuf/issues/9730
python -m pip install protobuf==3.19.4
Expand Down

0 comments on commit 799d259

Please sign in to comment.