From b2785fada033024958b6a405b8382f520f332093 Mon Sep 17 00:00:00 2001 From: Jeremy Simkins Date: Fri, 5 Jul 2019 19:18:22 -0400 Subject: [PATCH 1/2] Fixed legacy block type --- main.tf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/main.tf b/main.tf index f8a978e..701950d 100644 --- a/main.tf +++ b/main.tf @@ -59,7 +59,7 @@ resource "aws_cloudwatch_metric_alarm" "alarm" { var.default_threshold )}" - dimensions { + dimensions = { DistributionId = "${var.distribution_id}" Region = "Global" } From 95a1d1043a611a94a2ac5c0e79c11ec1ae54c72f Mon Sep 17 00:00:00 2001 From: Jeremy Simkins Date: Fri, 5 Jul 2019 19:49:54 -0400 Subject: [PATCH 2/2] BUG FIX for proper content type on alarm_actions --- main.tf | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/main.tf b/main.tf index 701950d..10c4758 100644 --- a/main.tf +++ b/main.tf @@ -67,9 +67,9 @@ resource "aws_cloudwatch_metric_alarm" "alarm" { # For some reason Terraform aborted its execution when the result # of split() was an empty list (i.e., when no actions are given). # By using compact() the abort magically goes away. - alarm_actions = ["${compact(split(",", lookup( + alarm_actions = "${compact(split(",", lookup( var.alarms[element(keys(var.alarms), count.index)], "actions", "" - )))}"] + )))}" }