sul-dlss-deprecated / terraform-module-aws-ecs

DEPRECATED - Use sul-dlss/terraform-aws/modules/local/ecs - AWS ECS module for Terraform
2 stars 1 forks source link

Do we need a new alarm for scaling down? #13

Open aaron-collier opened 6 years ago

aaron-collier commented 6 years ago

Currently the scaling down is using the same CloudWatch alarm as the scale up. I’m pretty sure we need a different alarm for scaling down. See https://docs.aws.amazon.com/AmazonECS/latest/developerguide/cloudwatch_alarm_autoscaling.html

(Optional) You can also create another alarm that triggers when the memory reservation is below 25%, which you can use to remove a container instance from your Auto Scaling group.

jonrober commented 5 years ago

Can you look at the above commit and see if that answers for you?