Skip to content
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

feat: Adds max_healthy_percentage and skip_matching to autoscaling group #256

Conversation

easterbonbon
Copy link
Contributor

Description

Adds max_healthy_percentage and skip_matching to aws_autoscaling_group instance_refresh.preferences.

Small aside: also updated the examples to look for an Amazon Linux 2 AMI rather than an Amazon Linux AMI which is EOL.

Motivation and Context

These two settings were added in the AWS Terraform provider and currently there is no way to specify them using this module.

max_healthy_percentage was added in v5.32.0 (see changelog)
and
skip_matching was added in v4.13.0 (see changelog)

Breaking Changes

No

How Has This Been Tested?

  • I have updated at least one of the examples/* to demonstrate and validate my change(s)
  • I have tested and validated these changes using one or more of the provided examples/* projects
  • I have executed pre-commit run -a on my pull request

…rcentage attribute support was added and allow configuring max_healthy_percentage
…es examples to look for Amazon Linux 2 AMIs since Amazon Linux AMIs are already end of life.
@@ -4,7 +4,7 @@ terraform {
required_providers {
aws = {
source = "hashicorp/aws"
version = ">= 5.26"
version = ">= 5.32"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need to change this throughout - aligning all of the examples as well

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I've updated the required version in the examples and updated the docs. Let me know if there anything else I've missed :)

Copy link
Member

@bryantbiggs bryantbiggs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome, thank you!

@bryantbiggs bryantbiggs merged commit faf98ef into terraform-aws-modules:master Feb 21, 2024
6 checks passed
antonbabenko pushed a commit that referenced this pull request Feb 21, 2024
## [7.4.0](v7.3.1...v7.4.0) (2024-02-21)

### Features

* Adds `max_healthy_percentage` and `skip_matching` to autoscaling group ([#256](#256)) ([faf98ef](faf98ef))
@antonbabenko
Copy link
Member

This PR is included in version 7.4.0 🎉

@easterbonbon easterbonbon deleted the feat/instance_refresh_updates branch February 21, 2024 22:30
Copy link

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants