#pr-reviews - February 2021

Could someone please take a look at this PR? https://github.com/cloudposse/terraform-aws-ecs-container-definition/pull/118

Regarding - https://github.com/cloudposse/terraform-aws-eks-node-group/pull/54 - there are some open questions that I am not able to answer ( in terns of aws-provider requirements and cloudposse customers ) - anyone that could chime in and provide a path forward? Would be amazing to have more than a single instance type in a spot instane node group

Hi all, https://github.com/cloudposse/terraform-aws-iam-system-user/pull/47

Can someone take a look at this one? https://github.com/cloudposse/terraform-aws-ecs-web-app/pull/113 Thanks!

@Erik Osterman (Cloud Posse)
Hi all,

I am getting the following from the ECS web app module using webhooks.
I am guessing its coming from the webhooks module.
It seems GitHub there are breaking changes with the GitHub provider.

Warning: Additional provider information from registry

The remote registry returned warnings for
<http://registry.terraform.io/hashicorp/github|registry.terraform.io/hashicorp/github>:
- For users on Terraform 0.13 or greater, this provider has moved to
integrations/github. Please update your source in required_providers.



Error: Failed to query available provider packages

Could not retrieve the list of available versions for provider
hashicorp/github: no available releases match the given constraints ~> 2.8.0,
3.0.0

@Leia Renée has joined the channel

Hi everyone,

Can someone take a look at this one please?
https://github.com/cloudposse/terraform-aws-vpc/pull/81

it's Friday, quick one https://github.com/cloudposse/terraform-aws-cicd/pull/87

Hi all,

Would someone take a look please?
https://github.com/cloudposse/terraform-aws-ecs-web-app/pull/122

Hi folks, seeing a bit of an odd issue on this PR. The build harness readme/build target seems to be non-deterministic from run to run, causing CI to fail.
https://github.com/cloudposse/terraform-aws-codebuild/pull/78

Just wrote up https://github.com/cloudposse/charts/issues/266

No PR yet, but I can work on one later if nobody gets to it first

Hi all,
Can someone take a look at this one please?
https://github.com/cloudposse/terraform-aws-vpc/pull/81

Hi all, I enabled support for the s3 deep archive storage class:

can you have a look ?

Thx guys !

Hi all, added GitHub Webhooks to the CI/CD module, would someone have a look?
https://github.com/cloudposse/terraform-aws-cicd/pull/89

if anyone has a chance, i opened up 5 PRs here

https://github.com/cloudposse/terraform-aws-ecs-alb-service-task/pulls

Hi there, I don't know how it works with pending PR, will post here for you 🙂
https://github.com/cloudposse/terraform-aws-rds/pull/106

review plz

• https://github.com/cloudposse/terraform-aws-ecs-container-definition/pull/123
• https://github.com/cloudposse/terraform-aws-elasticache-redis/pull/110