-
Notifications
You must be signed in to change notification settings - Fork 148
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
upgrade module version #93
upgrade module version #93
Conversation
Thanks for the PR! 🚀 |
@ericyz thanks for the PR :) - it looks like the docker build is failing with these updated parameters. Can you verify that it is working as expected through a local run? Also for consistency, can you please also update these files? (they help ensure the cloudbuild YAML and Dockerfile can still be used in isolation from the rest of the code) |
/gcbrun |
Thanks @rjerrems ! I used the wrong terraform hash and it's been updated now. |
Looks like you need to regenerate docs again @ericyz |
The other error looks like too many concurrent runs happening elsewhere, so should be ok once linting is fixed. |
The project-factory min support TF version is v0.13.0 now, so update the Terraform version used in Cloud Build module