https://cloudposse.com logo
Join Slack
Powered by
# pr-reviews
  • i

    Ivan Pinatti

    09/14/2026, 4:11 PM
    Could I please get a review on these 4 PRs? - github.com/cloudposse-terraform-components/aws-redshift/pull/52 - Exposes 17 variables the wrapper does not currently pass through, so encryption, snapshot restore, IAM roles and logging can't be set from a stack - github.com/cloudposse-terraform-components/aws-redshift/pull/53 - Adds vpc_component_name, which ~8 sibling components already have, plus subnet_ids for stacks with no CloudPosse-managed VPC - github.com/cloudposse-terraform-components/aws-redshift/pull/54 - Adds scheduled pause/resume support - github.com/cloudposse/terraform-aws-redshift-cluster/pull/52 - Exposes manual_snapshot_retention_period Two small things that would help: • The workflow runs on the three component PRs are sitting on action_required, so terraform CI hasn't reported on any of them yet, could someone approve those runs? • github.com/cloudposse-terraform-components/aws-redshift/pull/48 passes CI now that the remote-state bump landed. It's been open since June and it's a one-line module bump.
    a
    • 2
    • 3
  • b

    Ben Smith (Cloud Posse)

    09/14/2026, 8:17 PM
    Small pr for EKS Cluster github.com/cloudposse-terraform-components/aws-eks-cluster/pull/95
    merged 1
  • b

    Ben Smith (Cloud Posse)

    09/14/2026, 9:28 PM
    One tiny PR for EKS Cluster module github.com/cloudposse/terraform-aws-eks-cluster/pull/266
    👀 1
    merged 1
  • i

    Ivan Pinatti

    09/15/2026, 9:56 PM
    Please could I get a review on this PR? • github.com/cloudposse-terraform-components/aws-acm/pull/64 - It fixes a bug where the CloudPosse ACM component cannot handle a wildcard
    domain_name
    a
    • 2
    • 4
  • p

    PePe Amengual

    09/16/2026, 9:52 PM
    If I could gets some eyes on https://github.com/cloudposse-terraform-components/aws-config/pull/69
    merged 1
    • 1
    • 1
  • i

    Ivan Pinatti

    09/17/2026, 2:27 PM
    Could I please get an approval review on these two Renovate PRs? Both are green and just blocked on
    REVIEW_REQUIRED
    . • github.com/cloudposse-terraform-components/aws-redshift/pull/55 • github.com/cloudposse-terraform-components/aws-redshift/pull/46
    👀 1
    merged 1
  • i

    Ivan Pinatti

    09/17/2026, 2:33 PM
    Could I also get a review on this PR please? • github.com/cloudposse-terraform-components/aws-acm/pull/65 - Follow-up to the ACM wildcard fix. It adds an optional variable so one account can hold two certificates for the same domain without them overwriting each other's SSM parameter.
    merged 1
  • i

    Ivan Pinatti

    09/18/2026, 2:35 PM
    Happy Friday! The last PR for this week, could I please get a review? - github.com/cloudposse-terraform-components/aws-redshift/pull/56 - Exposes the last five pass-through variables now that the module pin moved to 1.7.0 in #55, including
    cluster_parameter_group_family
    which RA3/RG clusters need
    merged 1
  • i

    Ivan Pinatti

    09/21/2026, 3:33 PM
    Please could I get a review on this PR? • github.com/cloudposse-terraform-components/aws-dns-delegated/pull/73 - It adds an option to skip the parent zone lookup when the parent zone lives in a different account
    merged 1
  • p

    PePe Amengual

    09/22/2026, 5:05 PM
    any reason why the IPAM component used https://github.com/aws-ia/terraform-aws-ipam instead of a cloudposse managed module? last release of the AWS maintained module is a year old, are you guys open on creating a cloudposse owned module? ( I have cycles and interest)
    e
    • 2
    • 5
  • p

    PePe Amengual

    09/22/2026, 10:19 PM
    this is ready for review https://github.com/cloudposse/terraform-aws-ipam/pull/3
    e
    i
    a
    • 4
    • 21
  • m

    Michael

    09/23/2026, 3:53 PM
    If anyone has a chance to review, this adds some SSM functions to the test-helpers package. When we build an AMI with Packer, we use Terraform, Terratest, and SSM to run host checks to ensure it's configured as we intend. Contributing these back to the community for other's use could be beneficial for reusability github.com/cloudposse/test-helpers/pull/99
    i
    • 2
    • 4
  • i

    Ivan Pinatti

    09/23/2026, 7:11 PM
    Could I get a review please? • github.com/cloudposse-terraform-components/aws-datadog-monitor/pull/77 - It unblocks
    aws-datadog-monitor
    , where terraform init and terraform validate both fail on main today
    merged 1
  • i

    Ivan Pinatti

    09/24/2026, 3:26 PM
    Hey, could I please get a review on this one? • github.com/cloudposse/terraform-aws-waf/pull/134 - It adds captcha and challenge actions to byte-match and regex-match rules, two composite host-based rule types, and an OR scope-down for rate limiting.
    merged 1
    e
    i
    • 3
    • 12
  • i

    Ivan Pinatti

    09/28/2026, 3:19 PM
    Hello, could I get a review on this PR please? • github.com/cloudposse-terraform-components/aws-datadog-logs-archive/pull/122 - Exposes the four Datadog log archive tuning fields the component never wired (compression, partitioning, lookup attributes, scan cap) and defaults compression to ZSTD.
    merged 1
    i
    • 2
    • 2
  • i

    Ivan Pinatti

    09/30/2026, 3:53 PM
    Hello, could I get a review please? • github.com/cloudposse-terraform-components/aws-datadog-logs-archive/pull/123 - Adds
    query_exclusions
    so an archive can drop a class of logs, like health checks, without restating the whole query, and applies it to the
    catchall
    archive too.
    👀 1
    merged 1
    a
    • 2
    • 9
  • i

    Ivan Pinatti

    10/02/2026, 4:47 PM
    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
    👀 1
    merged 1
    m
    • 2
    • 1
  • i

    Ivan Pinatti

    10/05/2026, 5:25 PM
    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
    .
    👀 1
    merged 1
  • i

    Ivan Pinatti

    10/06/2026, 2:18 PM
    The release is stuck on this repo, could someone please unstuck it? ☝️
    p
    i
    • 3
    • 9
  • a

    Abdulwahab Abdusomad Olayiwola

    10/11/2026, 5:20 PM
    Hello, Can I get some reviews please? 1. github.com/cloudposse/terraform-aws-elastic-beanstalk-environment/pull/283 2. github.com/cloudposse/terraform-aws-elastic-beanstalk-environment/pull/284 3. github.com/cloudposse/terraform-aws-elastic-beanstalk-environment/pull/285
    e
    • 2
    • 4
  • i

    Ivan Pinatti

    10/14/2026, 3:03 PM
    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
    i
    a
    +2
    • 5
    • 50
  • i

    Ivan Pinatti

    10/14/2026, 9:15 PM
    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
    merged 1
    gratitude thank you 1
  • i

    Ivan Pinatti

    10/14/2026, 9:15 PM
    And a small bugfix in the other component • github.com/cloudposse/terraform-aws-datadog-integration/pull/85
    merged 1
    gratitude thank you 1
  • i

    Ivan Pinatti

    10/16/2026, 5:02 PM
    Hello, could I please get a review/approval on these? • github.com/cloudposse-terraform-components/aws-ssosync/pull/53 • github.com/cloudposse/terraform-aws-datadog-integration/pull/86
    merged 1
    gratitude thank you 1
  • p

    PePe Amengual

    10/22/2026, 11:53 PM
    I also have one https://github.com/cloudposse-terraform-components/aws-nlb/pull/32
    merged 1
    a
    • 2
    • 2
  • p

    PePe Amengual

    10/23/2026, 2:27 AM
    also I have this https://github.com/cloudposse-terraform-components/aws-ipam/pull/49
    merged 1
  • p

    PePe Amengual

    10/26/2026, 7:50 PM
    also this https://github.com/cloudposse-terraform-components/aws-alb/pull/80 and https://github.com/cloudposse-terraform-components/aws-alb/pull/81 and https://github.com/cloudposse-terraform-components/aws-alb/pull/82
    👀 1
    a
    • 2
    • 4
  • m

    Marat Bakeev

    10/28/2026, 11:14 PM
    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
    👀 1
    merged 1
    a
    • 2
    • 3
  • i

    Ivan Pinatti

    10/29/2026, 8:08 PM
    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
    merged 1
  • m

    Michael

    10/30/2026, 4:17 PM
    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
    merged 1