ci: Pin transitive dependencies for tests on Python>=3.8 by alexander-alderman-webb · Pull Request #6437 · getsentry/sentry-python

alexander-alderman-webb · 2026-05-28T13:24:53Z

Description

Add transitive dependencies to all test groups that are generated by populate_tox.py.
Dependency versions are resolved via a dry run of pip through uv on the relevant Python version.
Pins one set of dependencies per Python version and per library version.

Update DEPENDENCIES_CACHE to also be keyed on the Python version, since dependency versions of a package depend on the Python version. The cache is now indexed on the package name, package version, and Python version.

Issues

Reminders

Please add tests to validate your changes, and lint your code using tox -e linters.
Add GH Issue ID & Linear ID (if applicable)
PR title should use conventional commit style (feat:, fix:, ref:, meta:)
For external contributors: CONTRIBUTING.md, Sentry SDK development docs, Discord community

github-actions · 2026-05-28T13:26:51Z

Codecov Results 📊

📊 Comparison with Base Branch

Metric	Change
Total Tests	📉 -23
Passed Tests	📉 -22
Failed Tests	📈 +1
Skipped Tests	📉 -2

➕ New Tests (1)

View new tests

test_agent_data_from_scope[True-False]
- File: tests.integrations.pydantic_ai.test_pydantic_ai
- Status: ❌ Failing

❌ Failed Tests

`test_agent_data_from_scope[True-False]`

File: tests.integrations.pydantic_ai.test_pydantic_ai
Suite: py3.14t-pydantic_ai-v2.0.0b4
Error: pytest.PytestUnraisableExceptionWarning: Exception ignored while calling deallocator <function BaseEventLoop.del at 0x475c5afd740>: None

Stack Trace

../../_temp/uv-python-dir/cpython-3.14.5+freethreaded-linux-x86_64-gnu/lib/python3.14t/asyncio/base_events.py:760: in __del__
    self.close()
../../_temp/uv-python-dir/cpython-3.14.5+freethreaded-linux-x86_64-gnu/lib/python3.14t/asyncio/unix_events.py:70: in close
    super().close()
../../_temp/uv-python-dir/cpython-3.14.5+freethreaded-linux-x86_64-gnu/lib/python3.14t/asyncio/selector_events.py:104: in close
    self._close_self_pipe()
../../_temp/uv-python-dir/cpython-3.14.5+freethreaded-linux-x86_64-gnu/lib/python3.14t/asyncio/selector_events.py:111: in _close_self_pipe
    self._remove_reader(self._ssock.fileno())
../../_temp/uv-python-dir/cpython-3.14.5+freethreaded-linux-x86_64-gnu/lib/python3.14t/asyncio/selector_events.py:296: in _remove_reader
    key = self._selector.get_map().get(fd)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
../../_temp/uv-python-dir/cpython-3.14.5+freethreaded-linux-x86_64-gnu/lib/python3.14t/selectors.py:70: in get
    fd = self._selector._fileobj_lookup(fileobj)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
../../_temp/uv-python-dir/cpython-3.14.5+freethreaded-linux-x86_64-gnu/lib/python3.14t/selectors.py:229: in _fileobj_lookup
    return _fileobj_to_fd(fileobj)
           ^^^^^^^^^^^^^^^^^^^^^^^
../../_temp/uv-python-dir/cpython-3.14.5+freethreaded-linux-x86_64-gnu/lib/python3.14t/selectors.py:42: in _fileobj_to_fd
    raise ValueError("Invalid file descriptor: {}".format(fd))
E   ValueError: Invalid file descriptor: -1

The above exception was the direct cause of the following exception:
.tox/py3.14t-pydantic_ai-v2.0.0b4/lib/python3.14t/site-packages/_pytest/runner.py:353: in from_call
    result: TResult | None = func()
                             ^^^^^^
.tox/py3.14t-pydantic_ai-v2.0.0b4/lib/python3.14t/site-packages/_pytest/runner.py:245: in <lambda>
    lambda: runtest_hook(item=item, **kwds),
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
.tox/py3.14t-pydantic_ai-v2.0.0b4/lib/python3.14t/site-packages/pluggy/_hooks.py:512: in __call__
    return self._hookexec(self.name, self._hookimpls.copy(), kwargs, firstresult)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
.tox/py3.14t-pydantic_ai-v2.0.0b4/lib/python3.14t/site-packages/pluggy/_manager.py:120: in _hookexec
    return self._inner_hookexec(hook_name, methods, kwargs, firstresult)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
.tox/py3.14t-pydantic_ai-v2.0.0b4/lib/python3.14t/site-packages/_pytest/logging.py:850: in pytest_runtest_call
    yield
.tox/py3.14t-pydantic_ai-v2.0.0b4/lib/python3.14t/site-packages/pluggy/_callers.py:53: in run_old_style_hookwrapper
    return result.get_result()
           ^^^^^^^^^^^^^^^^^^^
.tox/py3.14t-pydantic_ai-v2.0.0b4/lib/python3.14t/site-packages/pluggy/_callers.py:38: in run_old_style_hookwrapper
    res = yield
          ^^^^^
.tox/py3.14t-pydantic_ai-v2.0.0b4/lib/python3.14t/site-packages/_pytest/capture.py:900: in pytest_runtest_call
    return (yield)
            ^^^^^
.tox/py3.14t-pydantic_ai-v2.0.0b4/lib/python3.14t/site-packages/pluggy/_callers.py:53: in run_old_style_hookwrapper
    return result.get_result()
           ^^^^^^^^^^^^^^^^^^^
.tox/py3.14t-pydantic_ai-v2.0.0b4/lib/python3.14t/site-packages/pluggy/_callers.py:38: in run_old_style_hookwrapper
    res = yield
          ^^^^^
.tox/py3.14t-pydantic_ai-v2.0.0b4/lib/python3.14t/site-packages/pluggy/_callers.py:53: in run_old_style_hookwrapper
    return result.get_result()
           ^^^^^^^^^^^^^^^^^^^
.tox/py3.14t-pydantic_ai-v2.0.0b4/lib/python3.14t/site-packages/pluggy/_callers.py:38: in run_old_style_hookwrapper
    res = yield
          ^^^^^
.tox/py3.14t-pydantic_ai-v2.0.0b4/lib/python3.14t/site-packages/_pytest/skipping.py:268: in pytest_runtest_call
    return (yield)
            ^^^^^
.tox/py3.14t-pydantic_ai-v2.0.0b4/lib/python3.14t/site-packages/_pytest/unraisableexception.py:158: in pytest_runtest_call
    collect_unraisable(item.config)
.tox/py3.14t-pydantic_ai-v2.0.0b4/lib/python3.14t/site-packages/_pytest/unraisableexception.py:79: in collect_unraisable
    raise errors[0]
.tox/py3.14t-pydantic_ai-v2.0.0b4/lib/python3.14t/site-packages/_pytest/unraisableexception.py:67: in collect_unraisable
    warnings.warn(pytest.PytestUnraisableExceptionWarning(msg))
E   pytest.PytestUnraisableExceptionWarning: Exception ignored while calling deallocator <function BaseEventLoop.__del__ at 0x475c5afd740>: None

✅ Patch coverage is 100.00%. Project has 2313 uncovered lines.
❌ Project coverage is 89.96%. Comparing base (base) to head (head).

Coverage diff

@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
- Coverage    89.99%    89.96%    -0.03%
==========================================
  Files          191       191         —
  Lines        23028     23028         —
  Branches      7904      7904         —
==========================================
+ Hits         20722     20715        -7
- Misses        2306      2313        +7
- Partials      1307      1307         —

Generated by Codecov Action

…ndencies

cursor

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit d26dd8a. Configure here.}

sl0thentr0py

maybe time to add tox.ini to .gitattributes as well

ci: Pin transitive dependencies for tests suites

cac908d

forgot tox.jinja

c332e3a

alexander-alderman-webb marked this pull request as ready for review May 28, 2026 13:30

alexander-alderman-webb requested a review from a team as a code owner May 28, 2026 13:30

alexander-alderman-webb changed the title ~~ci: Pin transitive dependencies for tests suites~~ ci: Pin transitive dependencies for tests suites ran on Python>=3.8 May 28, 2026

alexander-alderman-webb changed the title ~~ci: Pin transitive dependencies for tests suites ran on Python>=3.8~~ ci: Pin transitive dependencies for tests suites on Python>=3.8 May 28, 2026

alexander-alderman-webb marked this pull request as draft May 28, 2026 13:37

sentry-warden Bot reviewed May 28, 2026

View reviewed changes

Comment thread scripts/populate_tox/populate_tox.py

Comment thread scripts/populate_tox/populate_tox.py Outdated

sentry-warden Bot reviewed May 28, 2026

View reviewed changes

Comment thread scripts/populate_tox/populate_tox.py Outdated

alexander-alderman-webb added 7 commits May 28, 2026 15:44

cleanup

0db62e0

find dependencies for each Python version

b7b5385

fix cache

d34e749

merge master

bb4bcc2

.

fc7f034

merge master

510d3bc

unpin fakeredis

3457293

sentry-warden Bot reviewed May 29, 2026

View reviewed changes

Comment thread scripts/populate_tox/populate_tox.py Outdated

Comment thread scripts/populate_tox/populate_tox.py

sentry-warden Bot reviewed May 29, 2026

View reviewed changes

Comment thread scripts/populate_tox/populate_tox.py

sentry-warden Bot reviewed May 29, 2026

View reviewed changes

Comment thread scripts/populate_tox/populate_tox.py

alexander-alderman-webb added 5 commits May 29, 2026 13:35

fix boolean logic

601f8be

simplify

e00794f

merge master

9a2d30b

rerun script

3e7ad9f

unpin setuptools

47200a7

sentry-warden Bot reviewed May 29, 2026

View reviewed changes

Comment thread scripts/populate_tox/populate_tox.py

alexander-alderman-webb added 4 commits May 29, 2026 14:51

pin setuptools for pyramid

99ecfc9

add hash of constraints

1f729dc

merge master

18a1877

rerun script

cdd43f2

alexander-alderman-webb added 3 commits June 1, 2026 09:58

Merge branch 'master' into webb/populate-tox/transitive-dependencies

db82421

add setuptools to ray requirements

3dd3873

add setuptools to gevent

6ace6fd

sentry-warden Bot reviewed Jun 1, 2026

View reviewed changes

Comment thread scripts/populate_tox/populate_tox.py

alexander-alderman-webb added 3 commits June 1, 2026 13:16

narrower pin

cd47347

ci: Only pin setuptools in relevant tests

7d09c9e

Merge branch 'webb/setuptools' into webb/populate-tox/transitive-depe…

2da376e

…ndencies

alexander-alderman-webb changed the base branch from master to webb/setuptools June 1, 2026 11:23

trigger ci

d93dae6

alexander-alderman-webb changed the title ~~ci: Pin transitive dependencies for tests suites on Python>=3.8~~ ci: Pin transitive dependencies for tests on Python>=3.8 Jun 1, 2026

alexander-alderman-webb marked this pull request as ready for review June 1, 2026 11:30

sentry Bot reviewed Jun 1, 2026

View reviewed changes

Comment thread scripts/populate_tox/populate_tox.py

Base automatically changed from webb/setuptools to master June 1, 2026 11:37

merge master

d26dd8a

cursor Bot reviewed Jun 1, 2026

View reviewed changes

Comment thread scripts/populate_tox/tox.jinja

sl0thentr0py approved these changes Jun 1, 2026

View reviewed changes

add tox.ini to .gitattributes

b01571c

sentry Bot reviewed Jun 1, 2026

View reviewed changes

Comment thread scripts/populate_tox/populate_tox.py

sentry-warden Bot reviewed Jun 1, 2026

View reviewed changes

Comment thread scripts/populate_tox/populate_tox.py

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

ci: Pin transitive dependencies for tests on Python>=3.8#6437

ci: Pin transitive dependencies for tests on Python>=3.8#6437
alexander-alderman-webb wants to merge 27 commits into
masterfrom
webb/populate-tox/transitive-dependencies

alexander-alderman-webb commented May 28, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented May 28, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cursor Bot left a comment

Uh oh!

Uh oh!

sl0thentr0py left a comment •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

alexander-alderman-webb commented May 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Issues

Reminders

Uh oh!

github-actions Bot commented May 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Results 📊

📊 Comparison with Base Branch

➕ New Tests (1)

❌ Failed Tests

test_agent_data_from_scope[True-False]

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

sl0thentr0py left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

alexander-alderman-webb commented May 28, 2026 •

edited

Loading

github-actions Bot commented May 28, 2026 •

edited

Loading

`test_agent_data_from_scope[True-False]`

sl0thentr0py left a comment •

edited

Loading