Skip to content
This repository has been archived by the owner on Sep 9, 2022. It is now read-only.

Adding compatibility with actual terraform version #484

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

fipciu1996
Copy link

Yesterday I tried to use terraforming with to get ec2 instances and there was some uncompatibility with new version of terraform. In fact tags aren't declared as resource, but as a map so it changes to "tags = {" instead of "tags {" and keys in this shouldn't be specified as a String. After rebuilding it works correctly

Yesterday I tried to use terraforming with to get ec2 instances and there was some uncompatibility with new version of terraform. In fact tags aren't declared as resource, but as a map so it changes to "tags = {" instead of "tags {" and keys in this shouldn't be specified as a String. After rebuilding it works correctly
@coveralls
Copy link

coveralls commented Aug 24, 2019

Coverage Status

Coverage remained the same at 100.0% when pulling 41e5930 on fipciu1996:feature/compliance-with-tags into 518879d on dtan4:master.

@notjames
Copy link
Contributor

What is the risk of this change on backward compatibility? Seems like the way this change is committed could a potential problem.

@fipciu1996
Copy link
Author

fipciu1996 commented Aug 24, 2019

What is the risk of this change on backward compatibility? Seems like the way this change is committed could a potential problem.

Actually as I find out it terraforming now supports terraform between 0.9.3 - 0.12.0-alpha2 (October 30, 2018) so probably there will be more and more people who will have potential problems using terraforming. In my opinion it should be compatible with newer versions or it should be mentioned in README.md that you need older version of terraform

@notjames
Copy link
Contributor

What is the risk of this change on backward compatibility? Seems like the way this change is committed could a potential problem.

Actually as I find out it terraforming now supports terraform between 0.9.3 - 0.12.0-alpha2 (October 30, 2018) so probably there will be more and more people who will have potential problems using terraforming. In my opinion it should be compatible with newer versions or it should be mentioned in README.md that you need older version of terraform

I agree. I'd skip this commit to be in favor of a change to the README and maybe looking into supporting newer versions of terraform.

@theaboutbox
Copy link

I think that the tags = { } syntax works for earlier version of Terraform, it's just that Terraform stopped supporting the tags { ... } syntax in v0.12. (hashicorp/terraform#19240) This would also make the output of this tool match the Terraform documentation.

It looks like PR #442 is trying to address the same issue.

I am definitely interested in seeing Terraformer output HCL that's valid in 0.12. If there are any issues with either of the existing PRs, I'd be happy to dig in and help fix them.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants