21 messages
Pull Request Reviews for Cloud Posse Projects
Michael4 months ago
Small PR for fixing a Datadog tag parsing misinterpretation: https://github.com/cloudposse-terraform-components/aws-eks-datadog-agent/pull/74
Michael4 months ago
Small PR fix for Cognito component's failing validation: https://github.com/cloudposse-terraform-components/aws-cognito/pull/50
MP4 months ago
Hello 👋
Would I be able to get a review for this PR => https://github.com/cloudposse/atmos/pull/2058 ?
Reference: https://sweetops.slack.com/archives/C031919U8A0/p1770405166302539
Would I be able to get a review for this PR => https://github.com/cloudposse/atmos/pull/2058 ?
Reference: https://sweetops.slack.com/archives/C031919U8A0/p1770405166302539
Michael4 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
david4 months ago
May I get a review please https://github.com/cloudposse-terraform-components/aws-api-gateway-rest-api/pull/46
Jakub Wądołowski4 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
RB4 months ago
Please review setting bucket key to true default https://github.com/cloudposse/terraform-aws-s3-bucket/pull/285
Jakub Wądołowski4 months ago
I've just created two straightforward PRs that add
• 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?
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?
RB3 months ago
Another set of eyes here pleae https://github.com/cloudposse/terraform-aws-dynamodb/pull/141
Ivan Pinatti3 months ago
Ivan Pinatti3 months ago
PR adding maintenance and snapshot window options for Elasticache Redis
• https://github.com/cloudposse-terraform-components/aws-elasticache-redis/pull/64
• https://github.com/cloudposse-terraform-components/aws-elasticache-redis/pull/64
Ivan Pinatti3 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/
• https://github.com/cloudposse-terraform-components/aws-elasticache-redis/pull/62/
Ivan Pinatti3 months ago
Another one, expose
• https://github.com/cloudposse-terraform-components/aws-elasticache-redis/pull/63/
num_shards and replicas_per_shard as component-level variables• https://github.com/cloudposse-terraform-components/aws-elasticache-redis/pull/63/
RB3 months ago
Expose the rest of elasticache redis inputs
https://github.com/cloudposse-terraform-components/aws-elasticache-redis/pull/68
https://github.com/cloudposse-terraform-components/aws-elasticache-redis/pull/68
aairey3 months ago
Hi can anyone please review:
• https://github.com/cloudposse/terraform-aws-transfer-sftp/pull/81
• https://github.com/cloudposse/terraform-aws-transfer-sftp/pull/81
RB3 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
RB3 months ago
Please review adding global engine_version default for all redis clusters https://github.com/cloudposse-terraform-components/aws-elasticache-redis/pull/80
RB3 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
RB3 months ago
Please review setting default instance type for all redis clusters
https://github.com/cloudposse-terraform-components/aws-elasticache-redis/pull/82
https://github.com/cloudposse-terraform-components/aws-elasticache-redis/pull/82