#terraform-aws-modules (2020-02)

https://github.com/cloudposse/terraform-aws-elasticache-redis/pull/59

@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!

@scorebot help keep tabs!

@scorebot has joined the channel

Thanks for adding me emojis used in this channel are now worth points.

Wondering what I can do? try @scorebot help

@scorebot help

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:sob:. 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