-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added scaling cooldown variables. #53
Conversation
Added scaling cooldown variables.
/terratest |
@Jon-Berg Hi, can you update the pr so that it passes the tests? otherwise, it is likely to be closed due to staleness. |
/terratest |
|
/terratest |
there seems to be some issue with the readme generation :-( as some bits seems to be flipping around. Letrs try if we can get lucky - as currently the merging is blocked because of that. WIll raise the readme issue internally. |
The build harness that's being used is now updated to fix the flipping readme issues. |
@Jon-Berg can you update the readme once more? |
I did but it doesn't seem to change anything |
@Jon-Berg would you mind running |
/terratest |
@Jon-Berg 🎉 Thanks for the ride ! |
Looking into getting this released (seems also something 🐌 ) |
These changes were released in v0.1.0. |
what
Added scaling cooldown variables.
why
Need to control autoscaling cooldown times to handle usage spikes.
references
https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/appautoscaling_policy#target_tracking_scaling_policy_configuration