Skip to content

test: Reduce tests run in short tests jobs #4319

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 1 commit into from
Aug 8, 2024

Conversation

terriko
Copy link
Contributor

@terriko terriko commented Aug 8, 2024

Our short tests are currently not much shorter than our long tests. Since we've seldom seen only one version of python have a problem with a checker, I'm experimenting to see if running the many MANY mapping tests only in the long_tests jobs will help us reduce the time it takes for short tests to run.

I've also added some pytest insrumentation so we can see which of our tests take the longest to run so that we can hopefully find other places to reduce our usage footprint.

Our short tests are currently not much shorter than our long tests.
Since we've seldom seen only one version of python have a problem with a
checker, I'm experimenting to see if running the many MANY mapping tests
only in the long_tests jobs will help us reduce the time it takes for
short tests to run.

I've also added some pytest insrumentation so we can see which of our
tests take the longest to run so that we can hopefully find other places
to reduce our usage footprint.

* fixes intel#4315

Signed-off-by: Terri Oda <[email protected]>
@terriko
Copy link
Contributor Author

terriko commented Aug 8, 2024

Looks like this reduced each short test run by around 10 minutes (or 1/3 of the total run time) so I'm pretty happy with that.

Copy link

@el-tipton el-tipton left a comment

Choose a reason for hiding this comment

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

Looks good.

@terriko terriko merged commit 0881252 into intel:main Aug 8, 2024
22 checks passed
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.

ci: reduce tests run in short test jobs
2 participants