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

[PR #2396/3c7e9eca backport][stable-9] Deprecate managing instances through autoscaling_group #2399

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Nov 22, 2024

This is a backport of PR #2396 as merged into main (3c7e9ec).

SUMMARY

Managing instances directly through autoscaling_group has lead to a significant sprawl in the code which is difficult to test. With Amazon now also supporting "Instance Refresh" through its own API we can handle rolling replacements using less home-brew code (available through autoscaling_instance_refresh). The autoscaling_instance also has support for a more complete set of attach/detach/standby/terminate functionality (including instance protection management).

Deprecates the following parameters:

  • replace_all_instances -> amazon.aws.autoscaling_instance_refresh
  • replace_batch_size (tied to replace_all_instances)
  • lc_check (tied to replace_all_instances)
  • lt_check (tied to replace_all_instances)
  • detach_instances -> amazon.aws.autoscaling_instance
  • replace_instances -> amazon.aws.autoscaling_instance
  • decrement_desired_capacity (tied to detach_instances and replace_instances)

This functionality provided through these parameters is now available through autoscaling_instance and autoscaling_instance_refresh

Also adds group_name as an alias for name on both autoscaling_instance_refresh and autoscaling_group for consistency with autoscaling_instance where name would be ambiguous.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME
  • autoscaling_group
  • autoscaling_instance
  • autoscaling_instance_refresh
ADDITIONAL INFORMATION

SUMMARY
Managing instances directly through autoscaling_group has lead to a significant sprawl in the code which is difficult to test.  With Amazon now also supporting "Instance Refresh" through its own API we can handle rolling replacements using less home-brew code (available through autoscaling_instance_refresh).  The autoscaling_instance also has support for a more complete set of attach/detach/standby/terminate functionality (including instance protection management).
Deprecates the following parameters:

replace_all_instances -> amazon.aws.autoscaling_instance_refresh
replace_batch_size (tied to replace_all_instances)
lc_check (tied to replace_all_instances)
lt_check (tied to replace_all_instances)
detach_instances -> amazon.aws.autoscaling_instance
replace_instances -> amazon.aws.autoscaling_instance
decrement_desired_capacity (tied to detach_instances and replace_instances)

This functionality provided through these parameters is now available through autoscaling_instance and autoscaling_instance_refresh
Also adds group_name as an alias for name on both autoscaling_instance_refresh and autoscaling_group for consistency with autoscaling_instance where name would be ambiguous.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME

autoscaling_group
autoscaling_instance
autoscaling_instance_refresh

ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis
Reviewed-by: Mark Chappell
(cherry picked from commit 3c7e9ec)
Copy link

github-actions bot commented Nov 22, 2024

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/726f9e08bcc44a8c9e9aa1f0699383f5

✔️ ansible-galaxy-importer SUCCESS in 3m 30s
✔️ build-ansible-collection SUCCESS in 10m 12s
✔️ ansible-test-splitter SUCCESS in 4m 03s
✔️ integration-amazon.aws-1 SUCCESS in 26m 49s
✔️ integration-amazon.aws-2 SUCCESS in 10m 44s
✔️ integration-amazon.aws-3 SUCCESS in 13m 07s
Skipped 41 jobs

@alinabuzachis alinabuzachis added the mergeit Merge the PR (SoftwareFactory) label Nov 22, 2024
Copy link
Contributor

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/f253721df89c4e259bb08e0754938966

✔️ ansible-galaxy-importer SUCCESS in 3m 35s
✔️ build-ansible-collection SUCCESS in 10m 00s
✔️ ansible-test-splitter SUCCESS in 3m 55s
✔️ integration-amazon.aws-1 SUCCESS in 32m 24s
✔️ integration-amazon.aws-2 SUCCESS in 9m 59s
✔️ integration-amazon.aws-3 SUCCESS in 13m 39s
Skipped 41 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 7f3f095 into stable-9 Nov 22, 2024
45 checks passed
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-9/3c7e9eca848857127f11b9aa16f5c2796ef4b8a8/pr-2396 branch November 22, 2024 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants