Could I please get a review on these 4 PRs? - <htt...
# pr-reviews
i
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
👀 1
✅ 1
@Ivan Pinatti
i
Fixed