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

Parallelize the builds #292

Merged
merged 6 commits into from
Oct 23, 2019
Merged

Parallelize the builds #292

merged 6 commits into from
Oct 23, 2019

Conversation

aaron-lane
Copy link
Contributor

Running all Kitchen instances in serial is taking over 3 hours to complete which is an extreme amount of time to wait for validation of changes.

@aaron-lane aaron-lane added the enhancement New feature or request label Oct 22, 2019
@aaron-lane aaron-lane self-assigned this Oct 22, 2019
@aaron-lane
Copy link
Contributor Author

Looks like the build was reduced to 33 minutes!

@morgante
Copy link
Contributor

@aaron-lane Looks like tests are failing though.

Running all instances in serial is taking over 3 hours to complete.
This version fixes `check_documentation` to ignore the autogen
directory.
This content needs to be generated specifically for each module. It
does not make sense to include it in the autogen template.
@bharathkkb
Copy link
Member

@aaron-lane is it possible to template int.cloudbuild.yaml and generate it automatically based on whats in test/ci? I thought that was the use of test/ci maybe I am missing something.

@morgante
Copy link
Contributor

@bharathkkb We could probably write a script to do that, but it's not a high priority right now.

test/ci stores the actual test definitions and config, the Cloud Build config is just calling them.

@morgante morgante merged commit adf341e into master Oct 23, 2019
@aaron-lane aaron-lane deleted the aaron-lane-parallelize-builds branch October 23, 2019 17:17
This was referenced Oct 23, 2019
CPL-markus pushed a commit to WALTER-GROUP/terraform-google-kubernetes-engine that referenced this pull request Jul 15, 2024
…-modules/aaron-lane-parallelize-builds

Parallelize the builds
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants