(feat) Exposing the new minSampleCountToEvaluateDatapoint through CustomMonitoring
(fix) Fixing the minSampleCountToEvaluateDatapoint MathExpression's period as it defaults to 5 minutes. This didn't come up during testing as I tested it using 5 minute period. Apparently, if we don't set the period on MathExpression explicitly, it overrides all child metrics to 5 minute, reference. To avoid similar situations in the future, extended the unit test to cover custom periods.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license
Fixes #452
Follow up to https://github.com/cdklabs/cdk-monitoring-constructs/pull/453:
minSampleCountToEvaluateDatapoint
through CustomMonitoringminSampleCountToEvaluateDatapoint
MathExpression's period as it defaults to 5 minutes. This didn't come up during testing as I tested it using 5 minute period. Apparently, if we don't set the period on MathExpression explicitly, it overrides all child metrics to 5 minute, reference. To avoid similar situations in the future, extended the unit test to cover custom periods.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license