#terraform-aws-modules (2020-02)

terraform https://github.com/terraform-aws-modules/ (not cloudposse modules)

Discussions related to https://github.com/terraform-aws-modules Archive: https://archive.sweetops.com/terraform-aws-modules/

2020-02-25

Alex Kiss avatar
Alex Kiss

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 ?

cloudposse/terraform-aws-alb

Terraform module to provision a standard ALB for HTTP/HTTP traffic - cloudposse/terraform-aws-alb

David avatar
David
cloudposse/terraform-aws-alb

Terraform module to provision a standard ALB for HTTP/HTTP traffic - cloudposse/terraform-aws-alb

:--1:1
Alex Kiss avatar
Alex Kiss

or target_ids for that matter

2020-02-24

kimxogus avatar
kimxogus

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.

cloudposse/terraform-aws-cloudfront-s3-cdn

Terraform module to easily provision CloudFront CDN backed by an S3 origin - cloudposse/terraform-aws-cloudfront-s3-cdn

maxim avatar
maxim
cloudposse/terraform-aws-cloudfront-s3-cdn

Terraform module to easily provision CloudFront CDN backed by an S3 origin - cloudposse/terraform-aws-cloudfront-s3-cdn

:--1:2
kimxogus avatar
kimxogus

Thanks!

2020-02-22

antonbabenko avatar
antonbabenko

@scorebot help

2020-02-21

Erik Osterman (Cloud Posse) avatar
Erik Osterman (Cloud Posse)

@scorebot help keep tabs!

scorebot avatar
scorebot
05:50:33 PM

@scorebot has joined the channel

scorebot avatar
scorebot
05:50:34 PM

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

scorebot avatar
scorebot
05:50:36 PM

Wondering what I can do? try @scorebot help

2020-02-12

David avatar
David
add instance profile to iam-assumable-role by alemairebe · Pull Request #46 · terraform-aws-modules/terraform-aws-iam

Description I added the possibility to create an IAM instance profile to “iam-assumable-role” As it only takes 1 extra parameter, and is whitin the scope of this module (IAM)

Erik Osterman (Cloud Posse) avatar
Erik Osterman (Cloud Posse)

@maxim

maxim avatar
maxim

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

Erik Osterman (Cloud Posse) avatar
Erik Osterman (Cloud Posse)

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

Erik Osterman (Cloud Posse) avatar
Erik Osterman (Cloud Posse)

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

David avatar
David

ah gotcha, apologies!

2020-02-11

Jim Park avatar
Jim Park

@aknysh 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?

make cluster-enabled mode a bit more automated by jim80net · Pull Request #59 · cloudposse/terraform-aws-elasticache-redis

This PR adds the following functionality when setting cluster_mode_enabled == true: Automatically set number of nodes in cluster_size, enabling scale-out with a single parameter change. Automatica…

Erik Osterman (Cloud Posse) avatar
Erik Osterman (Cloud Posse)

@maxim is now helping as well review some PRs

Erik Osterman (Cloud Posse) avatar
Erik Osterman (Cloud Posse)

he can take a look at this tomorrow

maxim avatar
maxim
11:56:48 PM

@maxim has joined the channel

maxim avatar
maxim

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

Jim Park avatar
Jim Park

thanks @maxim!

Jim Park avatar
Jim Park

and thanks to all y’all

maxim avatar
maxim

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

Jim Park avatar
Jim Park

woot!

2020-02-05

Jim Park avatar
Jim Park
make cluster-enabled mode a bit more automated by jim80net · Pull Request #59 · cloudposse/terraform-aws-elasticache-redis

This PR adds the following functionality when setting cluster_mode_enabled == true: Automatically set number of nodes in cluster_size, enabling scale-out with a single parameter change. Automatica…

Erik Osterman (Cloud Posse) avatar
Erik Osterman (Cloud Posse)

@aknysh

Jim Park avatar
Jim Park

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

Adam Crews avatar
Adam Crews

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

    keyboard_arrow_up