#pr-reviews - February 2026

21 messages

Pull Request Reviews for Cloud Posse Projects

February 2026
Michael
Michael9 months ago
Small PR for fixing a Datadog tag parsing misinterpretation: https://github.com/cloudposse-terraform-components/aws-eks-datadog-agent/pull/74
Michael
Michael9 months ago
Small PR fix for Cognito component's failing validation: https://github.com/cloudposse-terraform-components/aws-cognito/pull/50
MP
MP9 months ago
Michael
Michael9 months ago
Sorry, quick patch fix for Datadog tags in the EKS agent: https://github.com/cloudposse-terraform-components/aws-eks-datadog-agent/pull/75
david
david9 months ago
Jakub Wądołowski
Jakub Wądołowski9 months ago
a small enhancement for the ECS web app module: https://github.com/cloudposse/terraform-aws-ecs-web-app/pull/305. Would appreciate a review when you get a chance
RB (pat leave again)
RB (pat leave again)9 months ago
Please review setting bucket key to true default https://github.com/cloudposse/terraform-aws-s3-bucket/pull/285
Jakub Wądołowski
Jakub Wądołowski9 months ago
I've just created two straightforward PRs that add response_completion_timeout support to both CDN modules:
• https://github.com/cloudposse/terraform-aws-cloudfront-cdn/pull/159
• https://github.com/cloudposse/terraform-aws-cloudfront-s3-cdn/pull/369
Could someone take a look please?
RB (pat leave again)
RB (pat leave again)9 months ago
Ivan Pinatti
Ivan Pinatti9 months ago
Ivan Pinatti
Ivan Pinatti8 months ago
PR adding maintenance and snapshot window options for Elasticache Redis
• https://github.com/cloudposse-terraform-components/aws-elasticache-redis/pull/64
Ivan Pinatti
Ivan Pinatti8 months ago
One more PR, now to enable Slow and Engine logs options for Redis
• https://github.com/cloudposse-terraform-components/aws-elasticache-redis/pull/62/
Ivan Pinatti
Ivan Pinatti8 months ago
Another one, expose num_shards and replicas_per_shard as component-level variables
• https://github.com/cloudposse-terraform-components/aws-elasticache-redis/pull/63/
RB (pat leave again)
RB (pat leave again)8 months ago
aairey
aairey8 months ago
RB (pat leave again)
RB (pat leave again)8 months ago
Please review allowing dns_subdomain to be overridden per cluster in redis clusters https://github.com/cloudposse-terraform-components/aws-elasticache-redis/pull/79
RB (pat leave again)
RB (pat leave again)8 months ago
Please review adding global engine_version default for all redis clusters https://github.com/cloudposse-terraform-components/aws-elasticache-redis/pull/80
RB (pat leave again)
RB (pat leave again)8 months ago
Please also review bumping the minimum aws provider for the elasticache redis component https://github.com/cloudposse-terraform-components/aws-elasticache-redis/pull/81
RB (pat leave again)
RB (pat leave again)8 months ago
Please review setting default instance type for all redis clusters

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