#pr-reviews - October 2026

14 messages

Pull Request Reviews for Cloud Posse Projects

October 2026
Ivan Pinatti
Ivan Pinattiabout 1 month ago
Happy Friday everyone!! One last PR this week por favor;
• github.com/cloudposse-terraform-components/aws-waf/pull/87 -- It makes byte_match_statement optional in rate-based scope-down statements and adds or_byte_match_statements, so one rate limit can cover several paths against a single counter, matching what module v1.18.1 already accepts
Ivan Pinatti
Ivan Pinatti29 days ago
New week and new PRs coming. Could you please review?
• github.com/cloudposse-terraform-components/aws-elasticache-redis/pull/104 -- it lets the component create and wire up the CloudWatch log groups for Redis slow and engine logs instead of expecting them to already exist, and exposes cluster_mode_enabled rather than always deriving it from num_shards.
Ivan Pinatti
Ivan Pinatti28 days ago
The release is stuck on this repo, could someone please unstuck it? ☝️
Ivan Pinatti
Ivan Pinatti20 days ago(edited)
Hello, could I please get a full review on these two @Igor Rodionov? These are the two new repos for CodeArtifact that I was talking with Erik recently.
• github.com/cloudposse/terraform-aws-codeartifact/pull/1
• github.com/cloudposse-terraform-components/aws-codeartifact/pull/3
Ivan Pinatti
Ivan Pinatti20 days ago
It is me again.. Not Mario!

I have two small PRs related to the SSO component, could I please get a review/approval?
• github.com/cloudposse-terraform-components/aws-ssosync/pull/52
• github.com/cloudposse-terraform-components/aws-ssosync/pull/51
Ivan Pinatti
Ivan Pinatti20 days ago
And a small bugfix in the other component
• github.com/cloudposse/terraform-aws-datadog-integration/pull/85
Ivan Pinatti
Ivan Pinatti18 days ago
PePe Amengual
PePe Amengual12 days ago
Marat Bakeev
Marat Bakeev6 days ago
Hi team, some PRs from me, please have a look when possible:

github.com/cloudposse/terraform-aws-route53-resolver-dns-firewall/pull/34 - Add an optional firewall_domain_redirection_action attribute

github.com/cloudposse-terraform-components/aws-route53-resolver-dns-firewall/pull/53 - upgrade the component for dns firewall to use that optional attribute

and an option for karpenter node pools to allow nested virtualization
github.com/cloudposse-terraform-components/aws-eks-karpenter-node-pool/pull/75
Ivan Pinatti
Ivan Pinatti5 days ago
Could I please get a review/approval for this PR?
• github.com/cloudposse-terraform-components/aws-aurora-postgres-resources/pull/77 -- adds an additional_extensions option to CloudPosse's aws-aurora-postgres-resources component, so Postgres extensions like pg_stat_statements (needed for Datadog Database Monitoring) are created in Terraform instead of by hand
Michael
Michael4 days ago
If anyone has a chance, this fixes some deprecated syntax, wires a missing parameter, and fixes the vpc tests:
github.com/cloudposse/terraform-aws-vpc/pull/171