Skip to content
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

Rewrote ci.yml workflow to parallelize steps #593

Merged
merged 1 commit into from
Sep 2, 2021

Conversation

pmarkowsky
Copy link
Contributor

@pmarkowsky pmarkowsky commented Sep 2, 2021

Build and test steps were taking more than 5 minutes to run so this cleans up
the workflow and parallelizes it.

It splits the workflow into the following 5 jobs.

Job Name Depends on What does it do?
preqs N/A Checks which files were modified in the PR and sets outputs for conditional execution of later jobs.
build_userspace preqs Ensures that any changes still result in a working build
build_driver preqs Ensures that any changes to the driver still build
unit_tests preqs Runs all unit tests
test_coverage preqs Runs all tests for coverage and uploads to Coveralls
preqs, build_userspace, build_driver, unit_tests, and test_coverage. The last four jobs depend on preqs.

Build and test steps were taking more than 5 minutes to run so this cleans up
the workflow and parallelizes it.
@pmarkowsky pmarkowsky merged commit 87dc191 into google:main Sep 2, 2021
@pmarkowsky pmarkowsky deleted the markowsky/ci-jobs branch September 2, 2021 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants