Skip to content
This repository has been archived by the owner on Feb 5, 2025. It is now read-only.

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 requested a review from tnek September 2, 2021 19:47
@google-cla google-cla bot added the cla: yes label Sep 2, 2021
@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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants