asgard icon indicating copy to clipboard operation
asgard copied to clipboard

Force delete support for ASGs

Open acasas opened this issue 11 years ago • 8 comments

This patch should allow users to force-delete ASGs. It follows a similar approach to what was already implemented for clusters, using the GroupDeleteOperation class.

acasas avatar Jun 11 '13 23:06 acasas

It's helpful to include the Jira ID at the start of the pull request name, and at the start of one of the commits within the pull request.

joesondow avatar Jun 11 '13 23:06 joesondow

asgard-pull-requests #70 SUCCESS This pull request looks good

Almost all of the code of the method deserves to be deleted and replaced with something more similar to what Cluster.delete does. The booleans and instance count protection and the try-catch blocks were only present because we were doing a synchronous Amazon call with a lot of edge cases to handle.

joesondow avatar Jun 11 '13 23:06 joesondow

This actually wouldn't be an improvement for the user, since the purpose of using Force Delete is that it's possible to delete an ASG that still contains some instances, whether the instances are real or phantoms. There was an internal email this afternoon about this very problem.

joesondow avatar Jun 11 '13 23:06 joesondow

asgard-pull-requests #71 SUCCESS This pull request looks good

asgard-pull-requests #72 SUCCESS This pull request looks good

@joesondow Is there value in the extra work that GroupDeleteOperation is doing here? I'm asking not really for this issue, but to see if it should be in the new auto deploy workflow.

claymccoy avatar Jun 12 '13 03:06 claymccoy

asgard-pull-requests #73 SUCCESS This pull request looks good