From c031c68d4bb4e5401a3f75d98dea0f20562f15f5 Mon Sep 17 00:00:00 2001 From: Chikara Takahashi Date: Wed, 1 Nov 2023 16:12:03 -0400 Subject: [PATCH] remove default tags applied to resources --- README.md | 6 +++--- variables.tf | 11 ++++------- 2 files changed, 7 insertions(+), 10 deletions(-) diff --git a/README.md b/README.md index 312b97a..4b40295 100644 --- a/README.md +++ b/README.md @@ -57,16 +57,16 @@ module "bootstrap" { | bucket\_purpose | Name to identify the bucket's purpose | `string` | `"tf-state"` | no | | dynamodb\_point\_in\_time\_recovery | Point-in-time recovery options | `bool` | `false` | no | | dynamodb\_table\_name | Name of the DynamoDB Table for locking Terraform state. | `string` | `"terraform-state-lock"` | no | -| dynamodb\_table\_tags | Tags of the DynamoDB Table for locking Terraform state. | `map(string)` | ```{ "Automation": "Terraform", "Name": "terraform-state-lock" }``` | no | +| dynamodb\_table\_tags | Tags of the DynamoDB Table for locking Terraform state. | `map(string)` | `{}` | no | | enable\_s3\_public\_access\_block | Bool for toggling whether the s3 public access block resource should be enabled. | `bool` | `true` | no | | kms\_master\_key\_id | The AWS KMS master key ID used for the SSE-KMS encryption of the state bucket. | `string` | `""` | no | -| log\_bucket\_tags | Tags to associate with the bucket storing the Terraform state bucket logs | `map(string)` | ```{ "Automation": "Terraform" }``` | no | +| log\_bucket\_tags | Tags to associate with the bucket storing the Terraform state bucket logs | `map(string)` | `{}` | no | | log\_bucket\_versioning | A string that indicates the versioning status for the log bucket. | `string` | `"Disabled"` | no | | log\_name | Log name (for backwards compatibility this can be modified to logs) | `string` | `"log"` | no | | log\_retention | Log retention of access logs of state bucket. | `number` | `90` | no | | manage\_account\_alias | Manage the account alias as a resource. Set to 'false' if this behavior is not desired. | `bool` | `true` | no | | region | AWS region. | `string` | n/a | yes | -| state\_bucket\_tags | Tags to associate with the bucket storing the Terraform state files | `map(string)` | ```{ "Automation": "Terraform" }``` | no | +| state\_bucket\_tags | Tags to associate with the bucket storing the Terraform state files | `map(string)` | `{}` | no | ## Outputs diff --git a/variables.tf b/variables.tf index 1f2081e..3652241 100644 --- a/variables.tf +++ b/variables.tf @@ -6,11 +6,8 @@ variable "dynamodb_table_name" { variable "dynamodb_table_tags" { description = "Tags of the DynamoDB Table for locking Terraform state." - default = { - Name = "terraform-state-lock" - Automation = "Terraform" - } - type = map(string) + default = {} + type = map(string) } variable "region" { @@ -53,13 +50,13 @@ variable "log_bucket_versioning" { variable "state_bucket_tags" { type = map(string) - default = { Automation : "Terraform" } + default = {} description = "Tags to associate with the bucket storing the Terraform state files" } variable "log_bucket_tags" { type = map(string) - default = { Automation : "Terraform" } + default = {} description = "Tags to associate with the bucket storing the Terraform state bucket logs" }