Changed Canary alarm threshold from 100 to 50 (#44)

Signed-off-by: Brandon Shien <[email protected]>
opensearch-project · Jun 25, 2024 · 21ca56a · 21ca56a
1 parent 204c575
commit 21ca56a
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 12 deletions.
diff --git a/infrastructure/lib/constructs/canarySns.ts b/infrastructure/lib/constructs/canarySns.ts
@@ -25,7 +25,7 @@ export class canarySns extends SnsMonitors {
     private canaryFailed(alertName: string, canary: Canary): [Alarm, string] {
         const alarmObject = new cloudwatch.Alarm(this, `error_alarm_${alertName}`, {
             metric: canary.metricSuccessPercent(),
-            threshold: 100,
+            threshold: 50,
             evaluationPeriods: 1,
             comparisonOperator: cloudwatch.ComparisonOperator.LESS_THAN_THRESHOLD,
             datapointsToAlarm: 1,

diff --git a/infrastructure/package-lock.json b/infrastructure/package-lock.json
diff --git a/infrastructure/test/monitoring-stack.test.ts b/infrastructure/test/monitoring-stack.test.ts
@@ -226,7 +226,7 @@ test('Monitoring Stack Test', () => {
         "Namespace": "CloudWatchSynthetics",
         "Period": 300,
         "Statistic": "Average",
-        "Threshold": 100,
+        "Threshold": 50,
         "TreatMissingData": "notBreaching"
     });
 });