#pr-reviews - January 2021

Hi, I closed this PR for it may need more fixes, https://github.com/cloudposse/terraform-aws-ecs-alb-service-task/pull/89/commits/4c47d80f4d9aa51b23f51b3861b93a2008f9871c

do we need list type for all 3 roles, service/execute/task?

got a tf 0.14 upgrade PR here https://github.com/cloudposse/terraform-aws-route53-cluster-zone/pull/39

got another 0.14 PR https://github.com/cloudposse/terraform-aws-acm-request-certificate/pull/35

got a PR to fix a regression from my last PR https://github.com/cloudposse/terraform-aws-acm-request-certificate/pull/38

Hello. Is anyone able to help me push through this PR? (I’m currently having to reference my fork in our code which is less than ideal) - It needs approval and unsure as to why the tests are failing as I haven’t changed the default behaviour. Thanks

https://github.com/cloudposse/terraform-aws-msk-apache-kafka-cluster/pull/6

Hello! Could somebody take a look? https://github.com/cloudposse/terraform-aws-ecs-web-app/pull/79

fyi...

Hi there. Can someone please take a look at https://github.com/cloudposse/terraform-aws-dynamic-subnets/pull/115? The current version (with the default settings) is broken