Skip to content

Commit

Permalink
remove backup global settings resource (#206)
Browse files Browse the repository at this point in the history
  • Loading branch information
edbentho authored Feb 11, 2025
1 parent 053814a commit 9feac6b
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 15 deletions.
2 changes: 0 additions & 2 deletions modules/aws/backup_vault/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ No modules.

| Name | Type |
|------|------|
| [aws_backup_global_settings.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/backup_global_settings) | resource |
| [aws_backup_vault.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/backup_vault) | resource |
| [aws_backup_vault_policy.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/backup_vault_policy) | resource |

Expand All @@ -27,7 +26,6 @@ No modules.
|------|-------------|------|---------|:--------:|
| <a name="input_backup_vault_kms_key_arn"></a> [backup\_vault\_kms\_key\_arn](#input\_backup\_vault\_kms\_key\_arn) | ARN of the KMS key used to protect the AWS Backup vault. | `string` | `null` | no |
| <a name="input_backup_vault_policy_json"></a> [backup\_vault\_policy\_json](#input\_backup\_vault\_policy\_json) | Resource Policy JSON for the AWS Backup vault. | `string` | `""` | no |
| <a name="input_enable_cross_account_backup"></a> [enable\_cross\_account\_backup](#input\_enable\_cross\_account\_backup) | Enable cross account backup? | `bool` | `false` | no |
| <a name="input_name"></a> [name](#input\_name) | Name of the Backup Vault | `string` | n/a | yes |
| <a name="input_tags"></a> [tags](#input\_tags) | Resource tags | `map(string)` | n/a | yes |

Expand Down
7 changes: 0 additions & 7 deletions modules/aws/backup_vault/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,3 @@ resource "aws_backup_vault_policy" "this" {
backup_vault_name = aws_backup_vault.this.name
policy = var.backup_vault_policy_json
}

resource "aws_backup_global_settings" "this" {
count = var.enable_cross_account_backup == true ? 1 : 0
global_settings = {
"isCrossAccountBackupEnabled" = "true"
}
}
6 changes: 0 additions & 6 deletions modules/aws/backup_vault/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,6 @@ variable "backup_vault_kms_key_arn" {
default = null
}

variable "enable_cross_account_backup" {
description = "Enable cross account backup?"
type = bool
default = false
}

variable "tags" {
description = "Resource tags"
type = map(string)
Expand Down

0 comments on commit 9feac6b

Please sign in to comment.