#pr-reviews - August 2022

Hey crew, a buddy and I are working on some EKS changes, would love a review on this whenever someone gets a chance: https://github.com/cloudposse/terraform-aws-eks-node-group/pull/126 (needs another reviewer besides me 🙂 )

👀 https://github.com/cloudposse/terraform-aws-eks-workers/pull/93 pretty please 🙂

https://github.com/cloudposse/terraform-aws-ssm-tls-self-signed-cert/pull/14
If possible 🙏
This breaks a number of other modules right now which are dependent on this

Hi all.
I think I found a bug in the terraform-aws-ecs-web-app module. EFS volumes are not being correctly set in the resulting task definition by the the terraform-aws-ecs-alb-service-task
Draft PR here.
Who can I discuss this with?
Thanks!

Hi, can i please get this merged https://github.com/cloudposse/terraform-aws-rds-cluster/pull/149, the required changes have been made and tests have passed. Thanks 🙂

pr review: https://github.com/cloudposse/terraform-aws-eks-cluster/pull/162

pr review: https://github.com/cloudposse/terraform-aws-acm-request-certificate/pull/59

another PR needs some progress, someone can help? https://github.com/cloudposse/terraform-aws-ec2-instance/pull/117

Could somebody take a look? Cloudflare removed two fields from healthcheck in 3.20. I already commented about bump supported version in required providers

https://github.com/cloudposse/terraform-cloudflare-zone/pull/7

copy of PR with fixed conflicts: https://github.com/cloudposse/terraform-aws-ec2-instance/pull/136

Hello guys, i have a PR for terraform aws rds cluster @ https://github.com/cloudposse/terraform-aws-rds-cluster/pull/149 .. its approved. Can i please get a ETA when this can be merged? Its a blocker for us. Thank you :)

Could someone take a look pr pls