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

Add support for deleting Auto Scaling Groups. #3

Closed
wants to merge 1 commit into from
Closed

Conversation

josh-padnick
Copy link
Contributor

Note that an ASG cannot be deleted until it contains 0 instances, so we first set the desired capacity down to 0, then wait, then delete the ASG.

Also fixed a bug where we were attempting to terminate EC2 Instances that were already terminated.

Note that an ASG cannot be deleted until it contains 0 instances, so we
first set the desired capacity down to 0, then wait, then delete the
ASG.

Also fixed a bug where we were attempting to terminate EC2 Instances
that were already terminated.

# Deleting ASGs after we nuke everything else is a hacky way to give set_asgs_num_desired_instances_to_zero enough
# time to complete its actions. We sleep an additional 10 seconds before proceeding just to be sure.
sleep 10
Copy link
Member

Choose a reason for hiding this comment

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

There's no way this will be enough. ASGs can easily take 2-5 min to empty out.

@brikis98
Copy link
Member

brikis98 commented Feb 6, 2018

We've moved to Go, and this functionality is now part of #5, so closing.

@brikis98 brikis98 closed this Feb 6, 2018
@josh-padnick josh-padnick deleted the asg branch February 18, 2018 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants