search

clusterinthecloud / terraform

Terraform config for Cluster in the Cloud
https://cluster-in-the-cloud.readthedocs.io
MIT License
20 stars 23 forks source link

Dictionary in network.tf #39

Closed tethig closed 4 years ago

tethig commented 4 years ago

Hi, I have terraform v0.12.5 and terraform init threw an error. The fix (suggested by terraform validate) is super simple, add an equals in line 53 of network.tf (to give tcp_options = {). However, I haven't tested if that's compatible with previous versions of terraform. thanks very much for your work, Ben

tethig commented 4 years ago

I now see this is the repo for the guide. I apologise for posting here and will refrain from cross-posting unless you advise. Ben

milliams commented 4 years ago

We currently only support Terraform 0.11 but are planning up updating to 0.12 very soon. We've got a big change with support or GCP incoming but after that we can look at updating to 0.12.

tethig commented 4 years ago

Hi, Thanks very much for your response. That's great. I will actually persist a little with mods on my cloned repo (the 0.12upgrade script seems to take care of what I thought was the thorniest issue) and let you know if I get it working. Failing that I will roll back Terraform to 0.11. many thanks again! Ben

tethig commented 4 years ago

I'm happy to report that when I got all my (user-specific) ducks in a row the current version is effectively fixed by terraform by terraform 0.12upgrade. The update script fixes variable declarations and formatting of blocks. This is better than using the validate function.

milliams commented 4 years ago

We have now updated to Terraform 0.12 so this is now fixed.