Skip to content

Commit

Permalink
Remove [test] package from httpx instrumentation
Browse files Browse the repository at this point in the history
Fixes #2203
  • Loading branch information
ocelotl committed Mar 13, 2024
1 parent 1e0b11f commit b12137b
Show file tree
Hide file tree
Showing 4 changed files with 65 additions and 14 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -34,11 +34,6 @@ dependencies = [
instruments = [
"httpx >= 0.18.0",
]
test = [
"opentelemetry-instrumentation-httpx[instruments]",
"opentelemetry-sdk ~= 1.12",
"opentelemetry-test-utils == 0.45b0.dev",
]

[project.entry-points.opentelemetry_instrumentor]
httpx = "opentelemetry.instrumentation.httpx:HTTPXClientInstrumentor"
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
anyio==3.7.1
asgiref==3.7.2
attrs==23.2.0
certifi==2024.2.2
Deprecated==1.2.14
exceptiongroup==1.2.0
h11==0.12.0
httpcore==0.13.7
httpx==0.18.2
idna==3.6
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
respx==0.17.1
rfc3986==1.5.0
sniffio==1.3.1
tomli==2.0.1
typing_extensions==4.10.0
wrapt==1.16.0
zipp==3.17.0
-e opentelemetry-instrumentation
-e util/opentelemetry-util-http
-e instrumentation/opentelemetry-instrumentation-httpx
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
anyio==4.3.0
asgiref==3.7.2
attrs==23.2.0
certifi==2024.2.2
Deprecated==1.2.14
exceptiongroup==1.2.0
h11==0.14.0
httpcore==1.0.4
httpx==0.27.0
idna==3.6
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
respx==0.20.2
sniffio==1.3.1
tomli==2.0.1
typing_extensions==4.10.0
wrapt==1.16.0
zipp==3.17.0
-e opentelemetry-instrumentation
-e util/opentelemetry-util-http
-e instrumentation/opentelemetry-instrumentation-httpx
19 changes: 10 additions & 9 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -203,8 +203,8 @@ envlist =
pypy3-test-instrumentation-tortoiseorm

; opentelemetry-instrumentation-httpx
py3{8,9,10,11}-test-instrumentation-httpx-{18,21}
pypy3-test-instrumentation-httpx-{18,21}
py3{8,9,10,11}-test-instrumentation-httpx-{0,1}
pypy3-test-instrumentation-httpx-{0,1}

; opentelemetry-util-http
py3{8,9,10,11}-test-util-http
Expand Down Expand Up @@ -292,10 +292,6 @@ deps =
pymemcache-300: pymemcache >3.0.0,<3.4.2
pymemcache-342: pymemcache ==3.4.2
pymemcache-400: pymemcache ==4.0.0
httpx-18: httpx>=0.18.0,<0.19.0
httpx-18: respx~=0.17.0
httpx-21: httpx>=0.19.0
httpx-21: respx~=0.20.1
urllib3v-1: urllib3 >=1.0.0,<2.0.0
urllib3v-2: urllib3 >=2.0.0,<3.0.0

Expand Down Expand Up @@ -333,7 +329,7 @@ commands_pre =

grpc: pip install {toxinidir}/instrumentation/opentelemetry-instrumentation-grpc[test]

falcon-{1,2,3},flask-{213,220,300},django-{1,2,3,4},pyramid,tornado,starlette,fastapi,aiohttp,asgi,httpx-{18,21},requests,urllib,urllib3v-{1,2},wsgi: pip install {toxinidir}/util/opentelemetry-util-http
falcon-{1,2,3},flask-{213,220,300},django-{1,2,3,4},pyramid,tornado,starlette,fastapi,aiohttp,asgi,requests,urllib,urllib3v-{1,2},wsgi: pip install {toxinidir}/util/opentelemetry-util-http
wsgi,falcon-{1,2,3},flask-{213,220,300},django-{1,2,3,4},pyramid: pip install {toxinidir}/instrumentation/opentelemetry-instrumentation-wsgi[test]
asgi,django-{3,4},starlette,fastapi: pip install {toxinidir}/instrumentation/opentelemetry-instrumentation-asgi[test]

Expand Down Expand Up @@ -420,7 +416,12 @@ commands_pre =

asyncio: pip install {toxinidir}/instrumentation/opentelemetry-instrumentation-asyncio[test]

httpx-{18,21}: pip install {toxinidir}/instrumentation/opentelemetry-instrumentation-httpx[test]
; The numbers at the end of the environment names
; below mean these dependencies are being used:
; 0: httpx>=0.18.0,<0.19.0 respx~=0.17.0
; 1: httpx>=0.19.0 respx~=0.20.1
httpx-0: pip install -r {toxinidir}/instrumentation/opentelemetry-instrumentation-httpx/test-requirements-0.txt
httpx-1: pip install -r {toxinidir}/instrumentation/opentelemetry-instrumentation-httpx/test-requirements-1.txt

sdkextension-aws: pip install {toxinidir}/sdk-extension/opentelemetry-sdk-extension-aws[test]

Expand Down Expand Up @@ -579,7 +580,7 @@ commands_pre =
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-tornado[test]
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-tortoiseorm[test]
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-mysql[test]
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-httpx[test]
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-httpx
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-aws-lambda[test]
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-system-metrics[test]
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-asyncio[test]
Expand Down

0 comments on commit b12137b

Please sign in to comment.