-
Notifications
You must be signed in to change notification settings - Fork 92
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
unittest in CircleCI in parallel #487
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will fasten up our testing
Much needed optimization |
@danielhoherd Can we spin up an issue ticket for this please. Will track this as a tech debt item |
@rishkarajgi turns out there already is one! https://github.com/astronomer/issues/issues/5199 |
I merged this to master and then cherry-picked into 1.8 and 1.9, and merged master into 1.10 |
Description
This adds parallel unit testing in CircleCI, which is something we do in astronomer/astronomer. The unittest step takes 18m20s on average as it is. https://app.circleci.com/insights/github/astronomer/airflow-chart/workflows/install-airflow-chart/jobs?branch=master&reporting-window=last-90-days This PR cuts that 18m20s down to about 4m20s https://app.circleci.com/pipelines/github/astronomer/airflow-chart/2126/workflows/ff7ab1a7-3067-4a78-9ecf-e4ab7d7701d3/jobs/23388
Related Issues
Testing
No QA testing is needed. These changes are only CircleCI related.
Merging
Merge everywhere.