Skip to content

Commit

Permalink
Remove [test] package from opentelemetry-exporter-otlp-proto-http (#3742
Browse files Browse the repository at this point in the history
)

Fixes #3723

Co-authored-by: Leighton Chen <[email protected]>
  • Loading branch information
ocelotl and lzchen authored Mar 14, 2024
1 parent 6c1940a commit 6af918b
Show file tree
Hide file tree
Showing 4 changed files with 65 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -33,11 +33,6 @@ dependencies = [
"requests ~= 2.7",
]

[project.optional-dependencies]
test = [
"responses >= 0.22.0, < 0.25",
]

[project.entry-points.opentelemetry_traces_exporter]
otlp_proto_http = "opentelemetry.exporter.otlp.proto.http.trace_exporter:OTLPSpanExporter"

Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
asgiref==3.7.2
attrs==23.2.0
certifi==2024.2.2
charset-normalizer==3.3.2
Deprecated==1.2.14
flaky==3.7.0
googleapis-common-protos==1.62.0
idna==3.6
importlib-metadata==6.11.0
iniconfig==2.0.0
packaging==23.2
pluggy==1.4.0
protobuf==3.19.6
py==1.11.0
py-cpuinfo==9.0.0
pytest==7.1.3
pytest-benchmark==4.0.0
PyYAML==6.0.1
requests==2.31.0
responses==0.24.1
tomli==2.0.1
typing_extensions==4.10.0
urllib3==2.2.1
wrapt==1.16.0
zipp==3.17.0
-e opentelemetry-api
-e exporter/opentelemetry-exporter-otlp-proto-common
-e opentelemetry-proto
-e opentelemetry-sdk
-e opentelemetry-semantic-conventions
-e exporter/opentelemetry-exporter-otlp-proto-http
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
asgiref==3.7.2
attrs==23.2.0
certifi==2024.2.2
charset-normalizer==3.3.2
Deprecated==1.2.14
flaky==3.7.0
googleapis-common-protos==1.62.0
idna==3.6
importlib-metadata==6.11.0
iniconfig==2.0.0
packaging==23.2
pluggy==1.4.0
protobuf==4.25.3
py==1.11.0
py-cpuinfo==9.0.0
pytest==7.1.3
pytest-benchmark==4.0.0
PyYAML==6.0.1
requests==2.31.0
responses==0.24.1
tomli==2.0.1
typing_extensions==4.10.0
urllib3==2.2.1
wrapt==1.16.0
zipp==3.17.0
-e opentelemetry-api
-e exporter/opentelemetry-exporter-otlp-proto-common
-e opentelemetry-proto
-e opentelemetry-sdk
-e opentelemetry-semantic-conventions
-e exporter/opentelemetry-exporter-otlp-proto-http
7 changes: 3 additions & 4 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -131,9 +131,8 @@ commands_pre =
exporter-otlp-proto-grpc: pip install {toxinidir}/exporter/opentelemetry-exporter-otlp-proto-common
exporter-otlp-proto-grpc: pip install {toxinidir}/exporter/opentelemetry-exporter-otlp-proto-grpc

exporter-otlp-proto-http: pip install {toxinidir}/opentelemetry-proto
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]
proto3-opentelemetry-exporter-otlp-proto-http: pip install -r {toxinidir}/exporter/opentelemetry-exporter-otlp-proto-http/test-requirements-0.txt
proto4-opentelemetry-exporter-otlp-proto-http: pip install -r {toxinidir}/exporter/opentelemetry-exporter-otlp-proto-http/test-requirements-1.txt

opentracing-shim: pip install {toxinidir}/opentelemetry-sdk
opentracing-shim: pip install {toxinidir}/shim/opentelemetry-opentracing-shim
Expand Down Expand Up @@ -224,7 +223,7 @@ commands_pre =
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]
python -m pip install -e {toxinidir}/exporter/opentelemetry-exporter-otlp-proto-http[test]
pip install -r {toxinidir}/exporter/opentelemetry-exporter-otlp-proto-http/test-requirements-1.txt
python -m pip install -e {toxinidir}/exporter/opentelemetry-exporter-otlp[test]
python -m pip install -e {toxinidir}/exporter/opentelemetry-exporter-prometheus[test]
python -m pip install -e {toxinidir}/exporter/opentelemetry-exporter-zipkin-json[test]
Expand Down

0 comments on commit 6af918b

Please sign in to comment.