Skip to content

ci: limit pytest duration reports to the slowest 20 tests - #2523

Merged
Andy-Jost merged 3 commits into
NVIDIA:mainfrom
Andy-Jost:ajost/pytest-durations-top20
Aug 10, 2026
Merged

ci: limit pytest duration reports to the slowest 20 tests#2523
Andy-Jost merged 3 commits into
NVIDIA:mainfrom
Andy-Jost:ajost/pytest-durations-top20

Conversation

@Andy-Jost

Copy link
Copy Markdown
Contributor

Summary

CI was passing --durations=0, which prints a duration line for every test phase and makes job logs hard to scan. Switch wheel-test workflows and ci/tools/run-tests to --durations=20 so only the slowest 20 entries are reported.

Changes

  • .github/workflows/test-wheel-linux.yml
  • .github/workflows/test-wheel-windows.yml
  • ci/tools/run-tests

--durations=0 prints every test phase and floods CI logs. Report only
the slowest 20 instead.
@Andy-Jost Andy-Jost added this to the cuda.core 1.2.0 milestone Aug 6, 2026
@Andy-Jost Andy-Jost added P2 Low priority - Nice to have CI/CD CI/CD infrastructure labels Aug 6, 2026
@Andy-Jost Andy-Jost self-assigned this Aug 6, 2026
@Andy-Jost
Andy-Jost requested review from juenglin and mdboom August 6, 2026 19:16
@Andy-Jost

Copy link
Copy Markdown
Contributor Author

/ok to test

@mdboom mdboom left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we put this in the [tool.pytest] section in the pyproject.toml files to reduce the duplication and make it apply by default for developers?

Move the duration limit into pytest addopts so CI and local runs
share one default, instead of repeating --durations on every command.
@github-actions github-actions Bot added cuda.bindings Everything related to the cuda.bindings module cuda.core Everything related to the cuda.core module cuda.pathfinder Everything related to the cuda.pathfinder module labels Aug 7, 2026
@Andy-Jost
Andy-Jost requested a review from mdboom August 7, 2026 15:32
@github-actions

This comment has been minimized.

@Andy-Jost
Andy-Jost enabled auto-merge (squash) August 7, 2026 18:35
@Andy-Jost Andy-Jost added the PR review get-together Mark PRs you'd like the team to review at the weekly PR review get-together. label Aug 10, 2026
@Andy-Jost
Andy-Jost merged commit 757731a into NVIDIA:main Aug 10, 2026
110 checks passed
@github-actions

This comment has been minimized.

1 similar comment
@github-actions

Copy link
Copy Markdown
Doc Preview CI
Preview removed because the pull request was closed or merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD CI/CD infrastructure cuda.bindings Everything related to the cuda.bindings module cuda.core Everything related to the cuda.core module cuda.pathfinder Everything related to the cuda.pathfinder module P2 Low priority - Nice to have PR review get-together Mark PRs you'd like the team to review at the weekly PR review get-together.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants