Add `monitor_cpu_utilization_too_high_period` variable to configure the number of seconds of the period for CPU

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 44 forks source link

Add `monitor_cpu_utilization_too_high_period` variable to configure the number of seconds of the period for CPU #15

Closed mmarcotte closed 3 years ago

mmarcotte commented 3 years ago

Would you consider adding another variable to adjust the interval between checks in the CPU Utilization?

metric-config

Our application relies on a highly available cluster, and so a period of 60 seconds with a higher CPU threshold (thanks for making that configurable!) would really suit us nicely. Happy to contribute with a PR for you to review.

🙌 Thank you for making this module! It's been super helpful for our application since we launched late last year.

dubiety commented 3 years ago

Hi @mmarcotte,

Thank you for raising the issue. I've added the variables you want. Please refer to release v2.1.0 If there's no further question, I'll close this bug next week. Thanks.

mmarcotte commented 3 years ago

This is perfect, thank you so much! 🙏