Skip to content

Increase pytest workers from 2 to 10#669

Closed
sid-rl wants to merge 2 commits into
mainfrom
siddarth/patch
Closed

Increase pytest workers from 2 to 10#669
sid-rl wants to merge 2 commits into
mainfrom
siddarth/patch

Conversation

@sid-rl

@sid-rl sid-rl commented Nov 17, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@sid-rl sid-rl requested a review from alb-rl November 17, 2025 23:24
Comment thread .github/workflows/smoketests.yml Outdated
# Use 2 workers to run files in parallel.
# Tests within a file are run sequentially.
PYTEST_ADDOPTS: "-n 2 -m smoketest"
PYTEST_ADDOPTS: "-n 10 -m smoketest"

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.

10 is a big increase, let's try 5 first.

Try rebasing #667 on this and run the smoketests on the branch

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

sure thing!

@sid-rl sid-rl closed this Nov 19, 2025
@sid-rl sid-rl deleted the siddarth/patch branch November 19, 2025 23:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants