RB

Results 868 comments of RB

The more I think about it, the more outputting that information to construct your own `aws_ecs_service` outside of this module, seems like the more scalable solution since lifecycle arguments cannot...

@Nuru as a follow up to my last comment. Now I'm in need of this 😄 There are times when you want to create task/exec roles, task definition but without...

@ByJacob the next step is to create a separate pr to * use the new version of the security group module similar to the way it was used before the...

@ByJacob it was opted to revert the sg module in this repo so now it's back to using the raw resources which means the repo is now unfrozen. Apologies for...

Thanks for the contribution. I'm just trying to get a second set of eyes on this to make sure it's what we want in this module.

Yep that makes sense. I'd say this issue is resolved https://github.com/cloudposse/terraform-aws-cloudfront-s3-cdn/blob/c7e6f3e6d1513157ca4c4e05379e39bd4dc20409/main.tf#L374-L377

Hmm... could argue that if you pass in the `var.parent_zone_name` that the `var.dns_alias_enabled` is extraneous.