#pr-reviews - December 2025

Can I get a PR review on this Elasticsearch component improvement? This introduces new configuration options and refactors how key variables are set and passed to the Elasticsearch module. The main focus is on improving flexibility for domain and subdomain naming, advanced security options, and storage configuration, while making these options configurable via input variables

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

Would I be able to get a review on changing the Helm provider version constraint from >= 2.7 to ~> 2.7
for the EKS Datadog Agent component?

https://github.com/cloudposse-terraform-components/aws-eks-datadog-agent/pull/69

Hi folks,
could I ask you to review https://github.com/cloudposse/terraform-aws-ec2-client-vpn/pull/96? This change isn't mine, but I've been testing it recently and it works great. The current version of this module has one simple but serious drawback that makes it difficult to use in a real environment.

hello!
anyone that can have a look at https://github.com/cloudposse/terraform-aws-ec2-autoscale-group/pull/162?

May I get a review on this SQS -> Lambda trigger please https://github.com/cloudposse-terraform-components/aws-lambda/pull/44

Irregular contributor here 👋

Small change to set some limits on labels due to AWS resource character limits. Not sure if there are additional changes to /test that need to be made. Thanks!
https://github.com/cloudposse/terraform-aws-eks-fargate-profile/pull/73

May I get a review on this Geodesic PR? https://github.com/cloudposse/geodesic/pull/980

It includes:
- Codespell typo fixes
- Improve shell script quality and security by fixing unquoted variables and unsafe array splitting patterns identified by ShellCheck
- Replace unsafe array assignment patterns like array=($(command)) with the more robust mapfile -t array < <(command) syntax
- Quote all variable expansions to prevent word splitting and globbing
- Fix command substitutions that could break with special characters

Hi everyone, can you please review this PR - https://github.com/cloudposse-terraform-components/aws-eks-alb-controller-ingress-class/pull/50

client vpn component broke after latest update:
https://github.com/cloudposse-terraform-components/aws-ec2-client-vpn/pull/47

Hello everyone, can you please review this PR

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