Skip to content

Use all cores in CI-run pytest jobs #6215

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 4 commits into from
Closed

Use all cores in CI-run pytest jobs #6215

wants to merge 4 commits into from

Conversation

pavoljuhas
Copy link
Collaborator

REVERT THIS BEFORE MERGING.

@pavoljuhas
Fix pytest randomly-seed to a constant
f7941b0 
REVERT THIS BEFORE MERGING.
@CirqBot CirqBot added the Size: XS <10 lines changed label Jul 22, 2023
@CirqBot CirqBot added the size: S 10< lines changed <50 label Jul 22, 2023
@pavoljuhas
Use the -n logical option for parallel pytest
3356737 
Requires more recent pytest-xdist.
@pavoljuhas
Copy link
Collaborator Author

There is no difference for pytest --numprocesses values auto, logical or $(nproc). Withdrawn.

@pavoljuhas pavoljuhas closed this Jul 22, 2023
@pavoljuhas pavoljuhas deleted the use-all-ci-cores branch July 22, 2023 06:24
@pavoljuhas pavoljuhas mentioned this pull request Jul 24, 2023
Closed
@nafaynajam
Copy link
Contributor

@pavoljuhas Understood.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
size: S 10< lines changed <50 Size: XS <10 lines changed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pavoljuhas @nafaynajam @CirqBot