Skip to content

Commit

Permalink
chore: revert "docs(cloudwatch): document cross-account/cross-region …
Browse files Browse the repository at this point in the history
…alarm limitations" (aws#26246)

Reverts aws#26204

The documentation that cross-account alarms with `evaluationPeriods > 1` is incorrect, or at least too strict. I was able to create an alarm in Account B that references a metric in Account A with evaluationPeriod = 2.
  • Loading branch information
kaizencc authored and bmoffatt committed Jul 28, 2023
1 parent 599517d commit 4eacc1b
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions packages/aws-cdk-lib/aws-cloudwatch/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -274,13 +274,6 @@ The most important properties to set while creating an Alarms are:

To create a cross-account alarm, make sure you have enabled [cross-account functionality](https://docs.aws.amazon.com/AmazonCloudWatch/latest/monitoring/Cross-Account-Cross-Region.html) in CloudWatch. Then, set the `account` property in the `Metric` object either manually or via the `metric.attachTo()` method.

Please note that it is **not possible** to:

- Create a cross-Account alarm that as `evaluationPeriods > 1`. The reason is that the only
way to pass an AccountID is to use the [`Metrics` field of the Alarm resource](https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-cloudwatch-alarm.html#cfn-cloudwatch-alarm-metrics). If we use the `Metrics` field, the CloudWatch event that is
used to evaluate the Alarm doesn't have a `SampleCount` field anymore ("[When CloudWatch evaluates alarms, periods are aggregated into single data points](https://docs.aws.amazon.com/AmazonCloudWatch/latest/monitoring/Create-alarm-on-metric-math-expression.html)"). The result is that the Alarm cannot evaluate more than 1 period.
- Create a cross-Region alarm ([source](https://docs.aws.amazon.com/AmazonCloudWatch/latest/monitoring/Cross-Account-Cross-Region.html)).

### Alarm Actions

To add actions to an alarm, use the integration classes from the
Expand Down

0 comments on commit 4eacc1b

Please sign in to comment.