[BUG FIX] ES FreeStorageSpaceTotalTooLow Alarm

dubiety / terraform-aws-elasticsearch-cloudwatch-sns-alarms

terraform module that configures important alarms for AWS elasticsearch and send them to SNS topic

Apache License 2.0

30 stars 45 forks source link

[BUG FIX] ES FreeStorageSpaceTotalTooLow Alarm #23

Closed tirumerla closed 2 years ago

tirumerla commented 2 years ago

Context

Looks like there is a typo, there is no official ES cw metric FreeStorageSpaceTotal. It's just sum on FreeStorageSpace metric.
Tested on my local stack.

Thanks

dubiety commented 2 years ago

@tirumerla , this metric is meant to be sum of total storage space of all nodes, it's not a typo. If you want to check single node storage space, please use this metric.

tirumerla commented 2 years ago

@tirumerla , this metric is meant to be sum of total storage space of all nodes, it's not a typo. If you want to check single node storage space, please use this metric.

Thanks for your response. The sum should be on "FreeStorageSpace" across all nodes , currently this module is looking for metric "FreeStorageSpaceTotal" ( which is not an official metric ) with stat Sum. When you deploy this alarm it always show "insufficient data" since this metric doesn't exist and never gets populated.

dubiety commented 2 years ago

@tirumerla yes, you're right. I'll merge your PR. Thanks for contributing!