Skip to content

Commit 1497972

Browse files
committed
Remove [test] packages from instrumentations
Fixes open-telemetry#2126
1 parent c9d709a commit 1497972

File tree

5 files changed

+43
-20
lines changed

5 files changed

+43
-20
lines changed

instrumentation/opentelemetry-instrumentation-flask/pyproject.toml

-6
Original file line numberDiff line numberDiff line change
@@ -35,14 +35,8 @@ dependencies = [
3535

3636
[project.optional-dependencies]
3737
instruments = [
38-
"werkzeug < 3.0.0",
3938
"flask >= 1.0",
4039
]
41-
test = [
42-
"opentelemetry-instrumentation-flask[instruments]",
43-
"markupsafe==2.1.2",
44-
"opentelemetry-test-utils == 0.44b0.dev",
45-
]
4640

4741
[project.entry-points.opentelemetry_instrumentor]
4842
flask = "opentelemetry.instrumentation.flask:FlaskInstrumentor"
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
#!/bin/bash
2+
3+
pip install -e ../../../opentelemetry-instrumentation
4+
pip install -e ../../../util/opentelemetry-util-http
5+
pip install -e ../../../instrumentation/opentelemetry-instrumentation-wsgi
6+
pip install "flask==2.1.3"
7+
pip install "werkzeug<3.0.0"
8+
9+
pip install -e ../../../instrumentation/opentelemetry-instrumentation-flask
10+
11+
pip freeze
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
#!/bin/bash
2+
3+
pip install -e ../../../opentelemetry-instrumentation
4+
pip install -e ../../../util/opentelemetry-util-http
5+
pip install -e ../../../instrumentation/opentelemetry-instrumentation-wsgi
6+
pip install "flask==2.2.0"
7+
pip install "werkzeug<3.0.0"
8+
9+
pip install -e ../../../instrumentation/opentelemetry-instrumentation-flask
10+
11+
pip freeze
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
#!/bin/bash
2+
3+
pip install -e ../../../opentelemetry-instrumentation
4+
pip install -e ../../../util/opentelemetry-util-http
5+
pip install -e ../../../instrumentation/opentelemetry-instrumentation-wsgi
6+
pip install "flask>=3.0.0"
7+
pip install "werkzeug>=3.0.0"
8+
9+
pip install -e ../../../instrumentation/opentelemetry-instrumentation-flask
10+
11+
pip freeze

tox.ini

+10-14
Original file line numberDiff line numberDiff line change
@@ -88,9 +88,9 @@ envlist =
8888
pypy3-test-instrumentation-fastapi
8989

9090
; opentelemetry-instrumentation-flask
91-
py3{7,8,9,10,11}-test-instrumentation-flask{213,220}
92-
py3{8,9,10,11}-test-instrumentation-flask{300}
93-
pypy3-test-instrumentation-flask{213,220}
91+
py3{7,8,9,10,11}-test-instrumentation-flask{0,1}
92+
py3{8,9,10,11}-test-instrumentation-flask{2}
93+
pypy3-test-instrumentation-flask{0,1}
9494

9595
; opentelemetry-instrumentation-urllib
9696
py3{7,8,9,10,11}-test-instrumentation-urllib
@@ -273,12 +273,6 @@ deps =
273273
falcon1: falcon ==1.4.1
274274
falcon2: falcon >=2.0.0,<3.0.0
275275
falcon3: falcon >=3.0.0,<4.0.0
276-
flask213: Flask ==2.1.3
277-
flask213: Werkzeug <3.0.0
278-
flask220: Flask ==2.2.0
279-
flask220: Werkzeug <3.0.0
280-
flask300: Flask >=3.0.0
281-
flask300: Werkzeug >=3.0.0
282276
grpc: pytest-asyncio
283277
sqlalchemy11: sqlalchemy>=1.1,<1.2
284278
sqlalchemy14: aiosqlite
@@ -329,7 +323,7 @@ changedir =
329323
test-instrumentation-elasticsearch{2,5,6}: instrumentation/opentelemetry-instrumentation-elasticsearch/tests
330324
test-instrumentation-falcon{1,2,3}: instrumentation/opentelemetry-instrumentation-falcon/tests
331325
test-instrumentation-fastapi: instrumentation/opentelemetry-instrumentation-fastapi/tests
332-
test-instrumentation-flask{213,220,300}: instrumentation/opentelemetry-instrumentation-flask/tests
326+
test-instrumentation-flask{0,1,2}: instrumentation/opentelemetry-instrumentation-flask/tests
333327
test-instrumentation-urllib: instrumentation/opentelemetry-instrumentation-urllib/tests
334328
test-instrumentation-urllib3v{1,2}: instrumentation/opentelemetry-instrumentation-urllib3/tests
335329
test-instrumentation-grpc: instrumentation/opentelemetry-instrumentation-grpc/tests
@@ -391,8 +385,8 @@ commands_pre =
391385

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

394-
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[test]
395-
wsgi,falcon{1,2,3},flask{213,220,300},django{1,2,3,4},pyramid: pip install {toxinidir}/instrumentation/opentelemetry-instrumentation-wsgi[test]
388+
falcon{1,2,3},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[test]
389+
wsgi,falcon{1,2,3},django{1,2,3,4},pyramid: pip install {toxinidir}/instrumentation/opentelemetry-instrumentation-wsgi[test]
396390
asgi,django{3,4},starlette,fastapi: pip install {toxinidir}/instrumentation/opentelemetry-instrumentation-asgi[test]
397391

398392
asyncpg: pip install {toxinidir}/instrumentation/opentelemetry-instrumentation-asyncpg[test]
@@ -406,7 +400,9 @@ commands_pre =
406400

407401
falcon{1,2,3}: pip install {toxinidir}/instrumentation/opentelemetry-instrumentation-falcon[test]
408402

409-
flask{213,220,300}: pip install {toxinidir}/instrumentation/opentelemetry-instrumentation-flask[test]
403+
flask0: {toxinidir}/instrumentation/opentelemetry-instrumentation-flask/test-0
404+
flask1: {toxinidir}/instrumentation/opentelemetry-instrumentation-flask/test-1
405+
flask2: {toxinidir}/instrumentation/opentelemetry-instrumentation-flask/test-2
410406

411407
urllib: pip install {toxinidir}/instrumentation/opentelemetry-instrumentation-urllib[test]
412408

@@ -543,7 +539,7 @@ commands_pre =
543539
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-grpc[test]
544540
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-falcon[test]
545541
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-boto[test]
546-
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-flask[test]
542+
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-flask
547543
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-sqlalchemy[test]
548544
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-cassandra[test]
549545
python -m pip install -e {toxinidir}/instrumentation/opentelemetry-instrumentation-celery[test]

0 commit comments

Comments
 (0)