terraform-aws-ecs-web-app icon indicating copy to clipboard operation
terraform-aws-ecs-web-app copied to clipboard

Incorrect defaults for ecs_alarms_cpu_utilization_high

Open stormmore opened this issue 5 years ago • 1 comments
trafficstars

Any of the action variables should default to [""] otherwise even if you set alb_groupd_alarms_enabled = false you will get errors about empty list of strings:

https://github.com/cloudposse/terraform-aws-ecs-web-app/blob/9d868399eaf3f7b1326ef9cda4fee505f9a292d3/variables.tf#L307-L407

Sorry don't have time right now to fix, test and pr.

stormmore avatar Jan 06 '20 19:01 stormmore

Out tests are passing so I do not think we are experiencing the same issue, have you looked at the new updates? @stormmore

jamengual avatar Jul 09 '20 18:07 jamengual