@aws-cdk/aws-cloudwatch/lib/alarm.ts: Cloudwatch Alarm with p100 statistics is missing ExtendedStatistic property in CFN template after recent cdk change.

[pearlswan]

Describe the bug

Starting recently, I noticed when using p100 statistics for cloudwatch Alarms, the generated CFN template is missing ExtendedStatistic property. However the same code was working previously. Also p100 is a valid value if create alarm from AWS console.

Looks like this change changes behavior for validating percentiles.
853e3d6

Expected Behavior

The generated CFN template should have ExtendedStatistic property.

Current Behavior

The generated CFN template is missing the ExtendedStatistic property.

Reproduction Steps

        const alarm = new Alarm(this, `----`, {
            alarmName: `----`,
            metric: new Metric({
                dimensionsMap: {
                    Program: program,
                },
                metricName: '----',
                namespace: namespace ,
                period: Duration.minutes(1),
                statistic: Stats.p(100),
            }),
            comparisonOperator: ComparisonOperator.GREATER_THAN_OR_EQUAL_TO_THRESHOLD,
            evaluationPeriods: 1,
            threshold: ----,
            treatMissingData: TreatMissingData.NOT_BREACHING,
        });

Possible Solution

No response

Additional Information/Context

No response

CDK CLI Version

na

Framework Version

No response

Node.js Version

14

OS

macOs

Language

Typescript

Language Version

No response

Other information

No response

[peterwoodworth]

Thanks for reporting this, I was able to confirm that the "ExtendedStatistic": "p100", prop in the template disappears before and after this change. We will try to address this promptly

[github-actions]

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.