RB

/terratest

/test all

For hygiene, some additional items - terraform-docs output added to readme - pre-commit hooks to run terraform-docs and other hooks Thank you for considering

For spot, it could be as simple as updating the launch template to expose the [market_options](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/launch_template#market-options) argument and/or providing our own `aws_launch_template` arn as an input as an escape hatch....

Looks like the competing fck-nat uses the latest amazon Linux 2023. Mentioned for inspiration. https://github.com/AndrewGuenther/fck-nat

@mamccormick isn't that what the following does ? https://github.com/cloudposse/terraform-aws-ecs-cloudwatch-autoscaling/blob/7a9c37e90f5b5a16efc4a3d18434fe49c1190747/main.tf#L15 https://github.com/cloudposse/terraform-aws-ecs-cloudwatch-autoscaling/blob/7a9c37e90f5b5a16efc4a3d18434fe49c1190747/main.tf#L24-L29 https://github.com/cloudposse/terraform-aws-ecs-cloudwatch-autoscaling/blob/7a9c37e90f5b5a16efc4a3d18434fe49c1190747/main.tf#L43-L48

Ah then you probably need to use this module with the ecs sns module https://github.com/cloudposse/terraform-aws-ecs-cloudwatch-sns-alarms

Awesome. Thanks for figuring it out @daltonsena ! Perhaps one of you could put in a PR to update our `README.yaml` file to include the `cloudposse/ecs-cloudwatch-sns-alarms/aws` module reference in the...

@syphernl is this still an issue with the latest version?

@brilong That PR is untested and was in a draft mode. I closed the PR for now until we are ready to work on it again. I'm unsure if a...