#terraform-aws-modules - May 2021

its corelogic
its corelogicover 5 years ago
👋 I’m here! What’d I miss?
michael sew
michael sewover 5 years ago
Minor Documentation question: https://docs.cloudposse.com/reference/best-practices/terraform-tips-tricks/ -- the S3 bucket lifecycle rules section seems like its missing a code backtick ( `). I'm not sure if this is powered by git in the background, so I'm just pointing it out here. =]
Karl Webster
Karl Websterover 5 years ago
Hey just wanted to check this is in fact a bug? https://github.com/cloudposse/terraform-aws-rds/issues/114
Karl Webster
Karl Websterover 5 years ago(edited)
Can we get this merged in pls?
Alex Jurkiewicz
Alex Jurkiewiczover 5 years ago
this small PR fixes a bug where resources are still created when var.enabled = false:
https://github.com/cloudposse/terraform-aws-dynamodb-autoscaler/pull/40
Please take a look, because once :merged: I will then update the downstream module:
https://github.com/cloudposse/terraform-aws-dynamodb
🙏
Alex Jurkiewicz
Alex Jurkiewiczover 5 years ago
thanks, this now unblocks my dynamodb bugfix too: https://github.com/cloudposse/terraform-aws-dynamodb/pull/95
Alex Jurkiewicz
Alex Jurkiewiczover 5 years ago
is anyone available to review a PR for null-label:
https://github.com/cloudposse/terraform-null-label/pull/118
and a RFC which I would like to implement?
https://github.com/cloudposse/terraform-null-label/issues/128
🙏