#pr-reviews - January 2023

Simple upgrade, could someone have a look ?

pretty please https://github.com/cloudposse/terraform-aws-alb/pull/127

and one more https://github.com/cloudposse/terraform-aws-ec2-autoscale-group/pull/101

quick one - if someone can start the tests https://github.com/cloudposse/terraform-aws-ec2-autoscale-group/pull/102

Hello.
I’m using the “terraform-aws-cloudtrail-s3-bucket” module, but it lacks 2 variables (object_lock_configuration & bucket_key_enabled), I added them to the module:
1) https://github.com/cloudposse/terraform-aws-cloudtrail-s3-bucket/pull/69

2) https://github.com/cloudposse/terraform-aws-cloudtrail-s3-bucket/pull/70

Since the module refers to “terraform-aws-s3-log-storage”, I had to add a variable in it:
3) https://github.com/cloudposse/terraform-aws-s3-log-storage/pull/84

Note that the final module “cloudposse/s3-bucket/aws” already has these variables.
Please approve these changes as soon as possible, the code is tested and works.

Other topic
------------

Hello, I have fixed the bug https://github.com/cloudposse/terraform-aws-cloudwatch-logs/pull/38 Please review as soon as possible. The code has been tested and works.