Change alarm periods

guardian · Nov 10, 2023 · bb9f716 · bb9f716
1 parent da5e756
commit bb9f716
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/notificationworkerlambda/cdk/lib/__snapshots__/senderworker.test.ts.snap b/notificationworkerlambda/cdk/lib/__snapshots__/senderworker.test.ts.snap
@@ -536,7 +536,7 @@ Object {
             "Ref": "AlarmTopicArn",
           },
         ],
-        "Period": 60,
+        "Period": 3600,
         "Statistic": "Sum",
         "Threshold": 0,
         "TreatMissingData": "breaching",
@@ -1006,7 +1006,7 @@ Object {
             "Ref": "AlarmTopicArn",
           },
         ],
-        "Period": 60,
+        "Period": 3600,
         "Statistic": "Sum",
         "Threshold": 0,
         "TreatMissingData": "breaching",
@@ -1476,7 +1476,7 @@ Object {
             "Ref": "AlarmTopicArn",
           },
         ],
-        "Period": 60,
+        "Period": 86400,
         "Statistic": "Sum",
         "Threshold": 0,
         "TreatMissingData": "breaching",
@@ -1946,7 +1946,7 @@ Object {
             "Ref": "AlarmTopicArn",
           },
         ],
-        "Period": 60,
+        "Period": 3600,
         "Statistic": "Sum",
         "Threshold": 0,
         "TreatMissingData": "breaching",
@@ -2416,7 +2416,7 @@ Object {
             "Ref": "AlarmTopicArn",
           },
         ],
-        "Period": 60,
+        "Period": 86400,
         "Statistic": "Sum",
         "Threshold": 0,
         "TreatMissingData": "breaching",

diff --git a/notificationworkerlambda/cdk/lib/senderworker.ts b/notificationworkerlambda/cdk/lib/senderworker.ts
@@ -154,8 +154,7 @@ class SenderWorker extends cdkcore.Construct  {
       evaluationPeriods: 1,
       threshold: 0,
       // whole day for editions, 60 minutes for others
-      // metric: senderLambdaCtr.metricInvocations({period: cdk.Duration.seconds(props.dailyAlarmPeriod ? 60 * 60 * 24 : 60 * 60), statistic: "Sum"}),
-      metric: senderLambdaCtr.metricInvocations({period: cdk.Duration.seconds(60), statistic: "Sum"}),
+      metric: senderLambdaCtr.metricInvocations({period: cdk.Duration.seconds(props.dailyAlarmPeriod ? 60 * 60 * 24 : 60 * 60), statistic: "Sum"}),
       treatMissingData: cloudwatch.TreatMissingData.BREACHING,
       actionsEnabled: true // isEnabled
     })