-
Notifications
You must be signed in to change notification settings - Fork 696
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
Update terraform to 0.13.7 #490
Update terraform to 0.13.7 #490
Conversation
@daniel-cit @bharathkkb @rjerrems Integration tests failed, could you please advise what went wrong as I don't have access to Cloud Build project. Thanks and much appreciated! |
@vovinacci - I have taken a look at it looks like one of the steps is failing (across a number of PRs). Worth noting that the test runner is currently on 0.13.5 - not sure if we have plans to update @bharathkkb ? |
@rjerrems I don't expect the runner to complain since we should only have >0.13 constraints in config. |
* master: docs: Remove redundant header from Jenkins bootstrap (terraform-google-modules#491)
@rjerrems Many thanks for the fix, all checks are back to green. @bharathkkb Having |
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.
LGTM
@vovinacci we are hoping to update all modules upstream to support 1.0 soon so this would also support 1.0. We would not require 1.0 in requirements but will switch our CI to do testing with 1.0. I expect current release to be mostly compatible with minor changes (deprecated list() fn, sensitive values being set) |
This PR sets the
terraform
version to0.13.7
in the code and and in README references.Main reasoning behind this PR is that
0.13.7
is the latest version (and most likely the last one in0.13
) and it's not easy to get 0.13.6 as a package. So, it helps with local development a lot as it won't require to resolvestate snapshot was created by Terraform v0.13.7, which is newer than current v0.13.6
error.