Skip to content

Fast tests #1504

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.

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

Merged
merged 12 commits into from
Aug 4, 2022
Merged

Fast tests #1504

merged 12 commits into from
Aug 4, 2022

Conversation

antonpirker
Copy link
Member

@antonpirker antonpirker commented Jul 15, 2022

The first iteration of making our test suite faster. This PR runs our test suite using tox --parallel auto to use all CPU cores for running the tests. This brings the duration down from almost 40 minutes to around 13 minutes.

One of our tests (test_redis_backend_trace_propagation) needed to be disabled because it hung when run in parallel. We will fix this test at a later stage in the process of optimizing our test suite.

@antonpirker antonpirker marked this pull request as ready for review August 2, 2022 13:21
@vladanpaunovic vladanpaunovic linked an issue Aug 3, 2022 that may be closed by this pull request
@antonpirker antonpirker enabled auto-merge (squash) August 4, 2022 12:26
@antonpirker antonpirker merged commit 8b1e8ce into master Aug 4, 2022
@antonpirker antonpirker deleted the antonpirker/fast-tests branch August 4, 2022 12:41
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.

Parallelize tests as much as possible
2 participants