Merge pull request #15 from geekcell/alarm-config

fix: alarm configuration showing up in every change set

Author: Ic3w0lf

README.md

@@ -149,7 +149,7 @@ preconfigured solution for seamless scalability and high availability."
 
 ## Resources
 
-- resource.aws_cloudwatch_log_group.main (main.tf#310)
+- resource.aws_cloudwatch_log_group.main (main.tf#313)
 - resource.aws_codedeploy_app.main (main.tf#231)
 - resource.aws_codedeploy_deployment_group.main (main.tf#238)
 - resource.aws_ecs_service.main (main.tf#56)

main.tf

@@ -268,10 +268,13 @@ resource "aws_codedeploy_deployment_group" "main" {
     events  = var.codedeploy_auto_rollback_events
   }
 
-  alarm_configuration {
-    enabled                   = length(var.codedeploy_cloudwatch_alarm_names) > 0 ? true : false
-    alarms                    = var.codedeploy_cloudwatch_alarm_names
-    ignore_poll_alarm_failure = var.codedeploy_ignore_poll_alarm_failure
+  dynamic "alarm_configuration" {
+    for_each = length(var.codedeploy_cloudwatch_alarm_names) > 0 ? [true] : []
+
+    content {
+      alarms                    = var.codedeploy_cloudwatch_alarm_names
+      ignore_poll_alarm_failure = var.codedeploy_ignore_poll_alarm_failure
+    }
   }
 
   load_balancer_info {