allow to add additional attributes and tags to autoscaler (#52)

* allow to add additional attributes and tags to autoscaler This is to avoid adding a region prefix to dynamodb tables because the autoscaler IAM role requires additional region indicator in its names * use map type for tags variable * remove markdown comments from diff * Updated README.md Co-authored-by: actions-bot <[email protected]>
cloudposse · Oct 9, 2020 · 622ce1e · 622ce1e
1 parent ed229a0
commit 622ce1e
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -224,6 +224,8 @@ Available targets:
 | autoscale\_min\_write\_capacity | DynamoDB autoscaling min write capacity | `number` | `5` | no |
 | autoscale\_read\_target | The target value (in %) for DynamoDB read autoscaling | `number` | `50` | no |
 | autoscale\_write\_target | The target value (in %) for DynamoDB write autoscaling | `number` | `50` | no |
+| autoscaler\_attributes | Additional attributes for the autoscaler module | `list(string)` | `[]` | no |
+| autoscaler\_tags | Additional resource tags for the autoscaler module | `map(string)` | `{}` | no |
 | billing\_mode | DynamoDB Billing mode. Can be PROVISIONED or PAY\_PER\_REQUEST | `string` | `"PROVISIONED"` | no |
 | delimiter | Delimiter to be used between `namespace`, `environment`, `stage`, `name` and `attributes` | `string` | `"-"` | no |
 | dynamodb\_attributes | Additional DynamoDB attributes in the form of a list of mapped values | <pre>list(object({<br>    name = string<br>    type = string<br>  }))</pre> | `[]` | no |

diff --git a/docs/terraform.md b/docs/terraform.md
@@ -25,6 +25,8 @@
 | autoscale\_min\_write\_capacity | DynamoDB autoscaling min write capacity | `number` | `5` | no |
 | autoscale\_read\_target | The target value (in %) for DynamoDB read autoscaling | `number` | `50` | no |
 | autoscale\_write\_target | The target value (in %) for DynamoDB write autoscaling | `number` | `50` | no |
+| autoscaler\_attributes | Additional attributes for the autoscaler module | `list(string)` | `[]` | no |
+| autoscaler\_tags | Additional resource tags for the autoscaler module | `map(string)` | `{}` | no |
 | billing\_mode | DynamoDB Billing mode. Can be PROVISIONED or PAY\_PER\_REQUEST | `string` | `"PROVISIONED"` | no |
 | delimiter | Delimiter to be used between `namespace`, `environment`, `stage`, `name` and `attributes` | `string` | `"-"` | no |
 | dynamodb\_attributes | Additional DynamoDB attributes in the form of a list of mapped values | <pre>list(object({<br>    name = string<br>    type = string<br>  }))</pre> | `[]` | no |

diff --git a/main.tf b/main.tf
@@ -127,8 +127,8 @@ module "dynamodb_autoscaler" {
   environment                  = var.environment
   name                         = var.name
   delimiter                    = var.delimiter
-  attributes                   = var.attributes
-  tags                         = module.dynamodb_label.tags
+  attributes                   = concat(var.attributes, var.autoscaler_attributes)
+  tags                         = merge(module.dynamodb_label.tags, var.autoscaler_tags)
   dynamodb_table_name          = join("", aws_dynamodb_table.default.*.id)
   dynamodb_table_arn           = join("", aws_dynamodb_table.default.*.arn)
   dynamodb_indexes             = null_resource.global_secondary_index_names.*.triggers.name

diff --git a/variables.tf b/variables.tf
@@ -147,6 +147,18 @@ variable "enable_autoscaler" {
   description = "Flag to enable/disable DynamoDB autoscaling"
 }
 
+variable "autoscaler_attributes" {
+  type        = list(string)
+  default     = []
+  description = "Additional attributes for the autoscaler module"
+}
+
+variable "autoscaler_tags" {
+  type        = map(string)
+  default     = {}
+  description = "Additional resource tags for the autoscaler module"
+}
+
 variable "dynamodb_attributes" {
   type = list(object({
     name = string