fpco / terraform-aws-foundation

Establish a solid Foundation on AWS with these modules for Terraform
MIT License
204 stars 98 forks source link

module init-snippet-attach-ebs-volume: Retrying attaching EBS #282

Closed Magicloud closed 4 years ago

Magicloud commented 4 years ago

In a rare case, the instance is running, but its AWS status is not "running". Then attaching process fails. The failure won't stop the instance hence ASG won't retry the whole process. This leaves us in an unexpected status.

This change is a patch up.


name: Pull request template about: Make a PR to terraform-aws-foundation

Please include the following in your PR:

Please also note that these are not hard requirements, but merely serve to define what maintainers are looking for in PR's. Including these will more likely lead to your PR being reviewed and accepted.

ketzacoatl commented 4 years ago

@Magicloud in the case where the volume never attaches, this will block, loop and not complete, is that correct? I think that's ok, but we should make note of the limitation and scope out updates to address this in the future (I have some ideas).