-
Notifications
You must be signed in to change notification settings - Fork 349
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
support terraform 0.12 #49
Conversation
The examples also need to be updated to support Terraform v0.12. @ivankorn Please go ahead and start a new PR which incorporates this one but also updates examples. |
name = "${var.name}-http-proxy" | ||
url_map = "${element(compact(concat(list(var.url_map), google_compute_url_map.default.*.self_link)), 0)}" | ||
url_map = element( |
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.
maybe compact(concat(list(var.url_map), google_compute_url_map.default.*.self_link))[0]
project = var.project | ||
count = var.ssl ? 1 : 0 | ||
name = "${var.name}-https-proxy" | ||
url_map = element( |
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.
maybe compact(concat(list(var.url_map), google_compute_url_map.default.*.self_link))[0]
description = "Map backend indices to list of backend maps." | ||
type = "map" | ||
type = map |
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.
IMO, this should become a list as backend_params is a list, but this is not what this PR is about :)
Changes
terraform 0.12upgrade
command of terraform 0.12.3 to upgrade syntax of this modulebackends
(the above command set it tomap(string)
, which is not correct)Reference
Fixes #49