airbnb / airbnb/binaryalert

Cloudwatch Alarm Statistic for downloader_sqs_age

Open
#158 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
1.5k
Forks
179
PR merge metrics
No merged PRs in 30d

Description

https://github.com/airbnb/binaryalert/blob/a9c0f06affc35e1f8e45bb77f835b92350c68a0b/terraform/cloudwatch_metric_alarm.tf#L82-L83

I am a bit baffled. This alarm is to catch a message age before it hits 75% of the retention time. In such a case why is the statistic set to minimum rather than maximum?

This would mean that the minimum of the oldest message would be checked instead of the maximum of the oldest message. Also the evaluation_periods of 10 seems rather generous.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.