Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Quote strings issue in Terraform 0.12 #11

Closed
rafaljanicki opened this issue Aug 11, 2019 · 3 comments · Fixed by #21
Closed

Quote strings issue in Terraform 0.12 #11

rafaljanicki opened this issue Aug 11, 2019 · 3 comments · Fixed by #21

Comments

@rafaljanicki
Copy link

When running the plan with the module included, Terraform 0.12.6 produces following error:

 on .terraform/modules/cloudtrail-cloudwatch-alarms/cloudposse-terraform-aws-cloudtrail-cloudwatch-alarms-0577581/alarms.tf line 149, in resource "aws_cloudwatch_dashboard" "main_individual":
 149: 
 150: 

Quoted strings may not be split over multiple lines. To produce a multi-line
string, either use the \n escape to represent a newline character or use the
"heredoc" multi-line template syntax.

The line numbers change as I think TF doesn't like the whole resource block.
@carlaabanes
Copy link

using terraform .12 too. which terraform lower version should be used?

mjuarez pushed a commit to mjuarez/terraform-aws-cloudtrail-cloudwatch-alarms that referenced this issue Oct 16, 2019
Move widget format to local. Fixes cloudposse#11
b4d22ac
@mjuarez mjuarez mentioned this issue Oct 16, 2019
Closed
@EvgeniiProkofevItomych
Copy link

Any progress on this issue? Just wondering why it still open, sorry.

@tgourley01
Copy link

Any workarounds?

@dotCipher dotCipher mentioned this issue Oct 21, 2020
Merged
Gowiem added a commit that referenced this issue Oct 26, 2020
@dotCipher @Gowiem
Rewrite for TF 0.12 and testing (#21)
32ca6a2 
## what

* Brings this module into the new ecosystem of tools used by CloudPosse, and updates `terraform` compatibility to `v0.12.x`.
* Adds `.github` dir with all the CloudPosse defaults
* Created a complete test case with updated CloudPosse module references
* Updated syntax for dashboard creation to use terraform native code, then `jsonencode`
* Extract all metric definitions into single `metric_definitions.tf` file for readability

## why
* Hasn't been updated in a while, and I would like to use it 😄 

## references
* Closes #11 

Co-authored-by: Matt Gowie <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@rafaljanicki @tgourley01 @carlaabanes @EvgeniiProkofevItomych