terraform-aws-modules / terraform-aws-autoscaling

Terraform module to create AWS Auto Scaling resources 🇺🇦
https://registry.terraform.io/modules/terraform-aws-modules/autoscaling/aws
Apache License 2.0
292 stars 556 forks source link

feat: Remove launch configuration support, add support for ignoring `desired_capacity` #173

Closed bryantbiggs closed 2 years ago

bryantbiggs commented 2 years ago

Description

Motivation and Context

With these two changes, we are then able to offload even more complexity and logic from the core EKS module and centralize the changes in the appropriate modules (i.e. - if a new feature is added to launch templates then we can update this module and propagate to the EKS module)

Breaking Changes

How Has This Been Tested?

github-actions[bot] commented 2 years ago

This PR has been automatically marked as stale because it has been open 30 days with no activity. Remove stale label or comment or this PR will be closed in 10 days

bryantbiggs commented 2 years ago

Shhhh

antonbabenko commented 2 years ago

@bryantbiggs Ops, I see that this one should have been merged around the same time as #179 . Could you finish this one and release version 5.1 right away?

bryantbiggs commented 2 years ago

ok should be all set @antonbabenko - I updated the upgrade document to include both this change and the tags change (even though this change will go out as v5.1.0)

antonbabenko commented 2 years ago

This PR is included in version 5.1.0 :tada:

github-actions[bot] commented 1 year ago

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.