Added functionality to handle EBS read latency

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

Added functionality to handle EBS read latency #22

Closed bsakweson closed 2 years ago

dubiety commented 2 years ago

This module is meant to support cloudwatch alarms recommended by AWS. I'm prone not to approve this PR unless there is a very good justification.

bsakweson commented 2 years ago

Completely agree, what I added is a cloudwatch alarm for EBS read latency, specifically for Opensearch. Please see this URL.

dubiety commented 2 years ago

Thank you for providing the link. Since there are thousands of metrics we can monitor, if read latency is added others can send PRs for different metrics. It could explode this module. Therefore, I will not approve this PR. Welcome to fork and modify for your needs. Thanks for your understanding.