#pr-reviews - May 2026

20 messages

Pull Request Reviews for Cloud Posse Projects

Ben Smith (Benbentwo)
Ben Smith (Benbentwo)6 months ago
Josh Simmonds
Josh Simmonds6 months ago
👋 Hi friends! PR here to bolster the native CI rewrite and fix issue 2367 (native CI not posting to PRs): https://github.com/cloudposse/atmos/pull/2405
Jakub Wądołowski
Jakub Wądołowski6 months ago
hey! I just submitted a minor monitoring improvement to both cloudposse/cloudfront-cdn/aws and cloudposse/cloudfront-s3-cdn/aws modules:
• https://github.com/cloudposse/terraform-aws-cloudfront-cdn/pull/162
• https://github.com/cloudposse/terraform-aws-cloudfront-s3-cdn/pull/372
Tyrone Meijn
Tyrone Meijn6 months ago
Hi 👋 I submitted a fix for the classic plan time for_each issue: https://github.com/cloudposse/terraform-aws-iam-role/pull/92
Jonathan
Jonathan6 months ago
May I get a review of https://github.com/cloudposse/atmos/pull/2430 please?
Jonathan
Jonathan6 months ago(edited)
Michael
Michael6 months ago
Would I be able to get a PR that adds deletion_protection for EKS clusters?

https://github.com/cloudposse/terraform-aws-eks-cluster/pull/260
Jonathan
Jonathan5 months ago
May I get a review of https://github.com/cloudposse/atmos/pull/2484 please?
Jonathan
Jonathan5 months ago
May I get a review of https://github.com/cloudposse/atmos/pull/2486 please?
Marat Bakeev
Marat Bakeev5 months ago
Hi team, can someone have a look at this PR that adds support for custom MAIL FROM domain in the ses component?
https://github.com/cloudposse-terraform-components/aws-ses/pull/69
Marat Bakeev
Marat Bakeev5 months ago
And another request - support for making ArgoCD internal, while still exposing its webhook receiver publicly.
https://github.com/cloudposse-terraform-components/aws-eks-argocd/pull/64
Marat Bakeev
Marat Bakeev5 months ago
N.B. - the pipeline seems unhappy

│ Error: Failed to query available provider packages
│
│ Could not retrieve the list of available versions for provider
│ cloudposse/utils: no available releases match the given constraints >=
│ 0.3.0, >= 1.7.1, >= 2.0.0, < 2.0.0, < 3.0.0
erik
erik5 months ago
@Igor Rodionov
Arnoldas Brazys
Arnoldas Brazys5 months ago
Hi team,
datadog integration been dragging with deprecation warnings since v2, now v4 already available, who could look into it?
https://github.com/cloudposse/terraform-aws-datadog-integration/pull/73
PePe Amengual
PePe Amengual5 months ago
Hi CP I have a PR for review https://github.com/cloudposse/terraform-aws-transit-gateway/pull/67 to add asn to the TGW component
Ben Smith (Benbentwo)
Ben Smith (Benbentwo)5 months ago
I have a relatively simple PR https://github.com/cloudposse-terraform-components/aws-managed-grafana-dashboard/pull/47 (new var input directly into resource) it fails with the issue of
│ Could not retrieve the list of available versions for provider
│ cloudposse/utils: no available releases match the given constraints >=
│ 0.3.0, >= 1.7.1, >= 2.0.0, < 2.0.0, < 3.0.0

do you want a new PR to update to Remote-state 2.0.0? Same PR? seems like this is an issue across many components, perhaps the range >= 1.7.1, < 3.0.0 would work (more forgiving for 2.X.Y releases that IIRC didn't have breaking changes)
Jonathan
Jonathan5 months ago
May I get a review of https://github.com/cloudposse/atmos/pull/2533 please?