#pr-reviews - May 2024

https://github.com/cloudposse/terraform-aws-elasticache-redis/pull/227

I added support for Serverless Redis to this module

Hey there, can someone have a look at this PR discussion?
https://github.com/cloudposse/terraform-aws-lb-s3-bucket/pull/82#issuecomment-2090913789
It’s about the correct use of the null label module to generate a default bucket name.

Hi Team, Gret help if you guys can help to review this PR https://github.com/cloudposse/terraform-aws-waf/pull/57

Very small PR (one-liner). Adding a try() around a length() function to stop it from throwing an error.

https://github.com/cloudposse/terraform-aws-rds/pull/179

Could you please review this pull request: https://github.com/terraform-aws-modules/terraform-aws-acm/pull/149
It would be nice to have this feature merged

In PR https://github.com/cloudposse/terraform-aws-ec2-bastion-server/pull/230/files

• The tflint step is failing here due to a github token issue
• The codeowners are set to admins
Is this repo out of date?

Hi, could I get the PR reviewed? https://github.com/cloudposse/terraform-aws-rds-cluster/pull/192

Hello all. could you check my PR? ref: https://github.com/cloudposse/terraform-aws-elasticache-memcached/pull/79 🙏

Hello, I need a PR review. Thanks 🙏 ref: https://github.com/cloudposse/terraform-aws-ec2-instance/pull/197

Could you take a look please? https://github.com/cloudposse/terraform-aws-eks-cluster/pull/227

Could i get some help with the failing checks in this pr ?

https://github.com/cloudposse/terraform-aws-components/pull/1032

hello all, new PR is here 🙂 I’m eating 🐛 https://github.com/cloudposse/terraform-aws-rds/pull/181 thank you for your time and effort! best! ❤️

Could I please get a review on https://github.com/cloudposse/terraform-aws-ecs-alb-service-task/pull/238 ?

Hi Team, help to review this https://github.com/cloudposse/terraform-aws-waf/pull/79 .
While using the WAF module we noticed this part is missing and just thought to raise the PR. Thank you

Could I get a PR review for overriding the dynamodb table name ?

https://github.com/cloudposse/terraform-aws-components/pull/1016

Hi. Could you please review this MR in the AWS DocumentDB module?
https://github.com/cloudposse/terraform-aws-documentdb-cluster/pull/94

Please have a look at this trivial PR, that would fix issue in the terraform-aws-components
https://github.com/cloudposse/terraform-aws-components/pull/1048

Hello,
Small PR to add missing AMI type in EKS Node group module
https://github.com/cloudposse/terraform-aws-eks-node-group/pull/182

Hi. Is it possible to continue the review? The terratest command and the rest of checks have passed.
https://sweetops.slack.com/archives/CUJPCP1K6/p1716842653008469