Multiple updates, fixes (#1)

Co-authored-by: Max Williams <[email protected]>

Author: max-rocket-internet

README.md

@@ -12,17 +12,17 @@ See [example](example) for a full example implimentation of both modules, multip
 
 ```hcl
 module "dashboard" {
-  source  = "git::https://github.com/deliveryhero/terraform-aws-service-quota-alarms.git//modules/dashboard?ref=1.4"
+  source  = "git::https://github.com/deliveryhero/terraform-aws-service-quota-alarms.git//modules/dashboard?ref=1.5"
   regions = ["us-east-1"]
 }
 
 module "trusted_advisor_alarms" {
-  source  = "git::https://github.com/deliveryhero/terraform-aws-service-quota-alarms.git//modules/trusted_advisor_alarms?ref=1.4"
+  source  = "git::https://github.com/deliveryhero/terraform-aws-service-quota-alarms.git//modules/trusted_advisor_alarms?ref=1.5"
   regions = ["us-east-1"]
 }
 
 module "usage_alarms" {
-  source = "git::https://github.com/deliveryhero/terraform-aws-service-quota-alarms.git//modules/usage_alarms?ref=1.4"
+  source = "git::https://github.com/deliveryhero/terraform-aws-service-quota-alarms.git//modules/usage_alarms?ref=1.5"
 }
 ```
 

modules/dashboard/usage.tf

@@ -74,7 +74,7 @@ locals {
         {
           type = "metric"
           properties = {
-            stat   = "Sum"
+            stat   = "Maximum"
             region = region
             period = 300
             view   = "timeSeries"
@@ -91,7 +91,7 @@ locals {
               for metric in metrics : flatten([
                 [
                   "AWS/Usage", "ResourceCount", "Class", metric["class"], "Resource", metric["resource"], "Service", metric["service_name"], "Type", "Resource",
-                  { id = metric["id"], region = metric["region"], visible = false }
+                  { id = metric["id"], region = metric["region"], visible = false, "stat" = metric["resource"] == "NumberOfMessagesPublishedPerAccount" ? "Sum" : "Maximum" }
                 ]
               ])
               ],

modules/trusted_advisor_alarms/main.tf

@@ -107,8 +107,8 @@ resource "aws_cloudwatch_metric_alarm" "main" {
   metric_name         = "ServiceLimitUsage"
   namespace           = "AWS/TrustedAdvisor"
   ok_actions          = var.cloudwatch_alarm_actions
-  period              = 3600
-  statistic           = "Average"
+  period              = 300
+  statistic           = "Maximum"
   tags                = var.tags
   threshold           = var.cloudwatch_alarm_threshold / 100
   treat_missing_data  = "ignore"

modules/usage_alarms/main.tf

@@ -112,8 +112,8 @@ resource "aws_cloudwatch_metric_alarm" "main" {
       }
       metric_name = "ResourceCount"
       namespace   = "AWS/Usage"
-      period      = 3600
-      stat        = "Average"
+      period      = 300
+      stat        = each.value["resource"] == "NumberOfMessagesPublishedPerAccount" ? "Sum" : "Maximum"
     }
   }
 }