Skip to content

Commit

Permalink
Merge pull request #115 from trussworks/remove-tags
Browse files Browse the repository at this point in the history
remove default tags applied to resources
  • Loading branch information
chtakahashi authored Nov 1, 2023
2 parents 5f1973f + c031c68 commit 002a627
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 10 deletions.
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -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

Expand Down
11 changes: 4 additions & 7 deletions variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -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" {
Expand Down Expand Up @@ -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"
}

Expand Down

0 comments on commit 002a627

Please sign in to comment.