#terraform-aws-modules - February 2020

@Andriy Knysh (Cloud Posse) ☝️

looks like test failed with a formatting error (I’m bad at those). commit coming with fix.

I’d love eyes on this PR too when there is time: https://github.com/cloudposse/terraform-aws-rds-cluster/pull/59

@Andriy Knysh (Cloud Posse) i’m not able to reproduce the failing test in https://github.com/cloudposse/terraform-aws-elasticache-redis/pull/59, advice for how to proceed?

@Maxim Mironenko (Cloud Posse) is now helping as well review some PRs

he can take a look at this tomorrow

@Maxim Mironenko (Cloud Posse) has joined the channel

@Jim Park looks like there was no issues with code, just test gets broken. will make a release shortly. stay tuned

thanks @Maxim Mironenko (Cloud Posse)!

and thanks to all y’all

@Jim Park here it is: <https://github.com/cloudposse/terraform-aws-elasticache-redis/releases/tag/0.15.0>

woot!

Friendly ping on https://github.com/terraform-aws-modules/terraform-aws-iam/pull/46 🙂

@Maxim Mironenko (Cloud Posse) ☝️

@Erik Osterman (Cloud Posse) @David this is not a Cloudposse repo, sorry 😞

Ah yes, @David this channel is unmonitored by the folks in the “terraform-aws-modules” organization

It’s been confusing that this channel is related to that organization, so I have given up and just conceded we can have conversations related to general terraform modules for AWS. :-)

ah gotcha, apologies!

@UUB28NLDS help keep tabs! 😉

Can you release https://github.com/cloudposse/terraform-aws-cloudfront-s3-cdn including https://github.com/cloudposse/terraform-aws-cloudfront-s3-cdn/pull/75 (maybe 0.21.1)?
Without that fix, module generates wrong origin domain with website_enabled = true in my region(ap-northeast-2), it took almost 2 hours to figure out what's wrong😭.
Now I'm using it with source = "git::<https://github.com/cloudposse/terraform-aws-cloudfront-s3-cdn.git?ref=41c6c331964c76184c796f1857b779fb88c51f44>" (which points to current master head) as workaround, but it would be better there's an official release and note for that fix.

Hi guys, I am using https://github.com/cloudposse/terraform-aws-alb - target_id is a list that describes the targets to be attached, is there any reason not to rename this to targets_id ?

or target_ids for that matter