fix: use LESS_THAN_THRESHOLD for addMinUsageCountAlarm (#318)

`LESS_THAN_LOWER_THRESHOLD` is for: ``` * Specified statistic is lower than the anomaly model band. * Used only for alarms based on anomaly detection models ``` ...which is not what it was being used for. --- _By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license_
cdklabs · Jan 28, 2023 · 07d9896 · 07d9896
1 parent fdd5ac1
commit 07d9896
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 10 deletions.
diff --git a/lib/common/monitoring/alarms/UsageAlarmFactory.ts b/lib/common/monitoring/alarms/UsageAlarmFactory.ts
@@ -47,7 +47,7 @@ export class UsageAlarmFactory {
         props.treatMissingDataOverride ?? TreatMissingData.MISSING,
       comparisonOperator:
         props.comparisonOperatorOverride ??
-        ComparisonOperator.LESS_THAN_LOWER_THRESHOLD,
+        ComparisonOperator.LESS_THAN_THRESHOLD,
       ...props,
       disambiguator,
       threshold: props.minCount,

diff --git a/test/monitoring/aws-cloudwatch/__snapshots__/LogMonitoring.test.ts.snap b/test/monitoring/aws-cloudwatch/__snapshots__/LogMonitoring.test.ts.snap
diff --git a/test/monitoring/aws-lambda/__snapshots__/LambdaFunctionMonitoring.test.ts.snap b/test/monitoring/aws-lambda/__snapshots__/LambdaFunctionMonitoring.test.ts.snap