Skip to content

Commit

Permalink
Remove [test] package from opentelemetry-sdk (open-telemetry#3735)
Browse files Browse the repository at this point in the history
Fixes open-telemetry#3717

Co-authored-by: Leighton Chen <[email protected]>
  • Loading branch information
ocelotl and lzchen authored Mar 7, 2024
1 parent 8ad10f7 commit 41cf788
Show file tree
Hide file tree
Showing 3 changed files with 23 additions and 4 deletions.
3 changes: 0 additions & 3 deletions opentelemetry-sdk/pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -30,9 +30,6 @@ dependencies = [
"typing-extensions >= 3.7.4",
]

[project.optional-dependencies]
test = []

[project.entry-points.opentelemetry_environment_variables]
sdk = "opentelemetry.sdk.environment_variables"

Expand Down
19 changes: 19 additions & 0 deletions opentelemetry-sdk/test-requirements.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
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 opentelemetry-semantic-conventions
-e opentelemetry-sdk
5 changes: 4 additions & 1 deletion tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -99,6 +99,9 @@ setenv =
commands_pre =
; Install without -e to test the actual installation
py3{8,9,10,11}: python -m pip install -U pip setuptools wheel

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

; Install common packages for all the tests. These are not needed in all the
; cases but it saves a lot of boilerplate in this file.
opentelemetry: pip install {toxinidir}/opentelemetry-api {toxinidir}/opentelemetry-semantic-conventions {toxinidir}/opentelemetry-sdk {toxinidir}/tests/opentelemetry-test-utils
Expand Down Expand Up @@ -212,7 +215,7 @@ deps =
commands_pre =
python -m pip install -e {toxinidir}/opentelemetry-api[test]
python -m pip install -e {toxinidir}/opentelemetry-semantic-conventions[test]
python -m pip install -e {toxinidir}/opentelemetry-sdk[test]
pip install -r {toxinidir}/opentelemetry-sdk/test-requirements.txt
python -m pip install -e {toxinidir}/opentelemetry-proto[test]
python -m pip install -e {toxinidir}/tests/opentelemetry-test-utils[test]
python -m pip install -e {toxinidir}/shim/opentelemetry-opentracing-shim[test]
Expand Down

0 comments on commit 41cf788

Please sign in to comment.