diff --git a/README.md b/README.md index 480075b..903ed5e 100644 --- a/README.md +++ b/README.md @@ -47,8 +47,6 @@ module "bastion_linux" { # s3 - used for logs and user public keys bucket_name = "bastion" - bucket_versioning = true - bucket_force_destroy = true # public keys public_key_data = local.public_key_data.keys[local.environment] # logs @@ -156,9 +154,7 @@ In order to prevent older versions from being retained forever, in addition to t | [allow\_ssh\_commands](#input\_allow\_ssh\_commands) | Allow SSH commands to be specified | `bool` | n/a | yes | | [app\_name](#input\_app\_name) | Name of application | `string` | n/a | yes | | [autoscaling\_cron](#input\_autoscaling\_cron) | Cron expressions for scale up and scale down | `map(string)` |
{| no | -| [bucket\_force\_destroy](#input\_bucket\_force\_destroy) | The bucket and all objects should be destroyed when using true | `bool` | n/a | yes | | [bucket\_name](#input\_bucket\_name) | Bucket used for bucket log storage and user public keys | `string` | n/a | yes | -| [bucket\_versioning](#input\_bucket\_versioning) | Enable bucket versioning or not | `bool` | n/a | yes | | [business\_unit](#input\_business\_unit) | Fixed variable to specify business-unit for RAM shared subnets | `string` | n/a | yes | | [environment](#input\_environment) | application environment | `string` | n/a | yes | | [extra\_user\_data\_content](#input\_extra\_user\_data\_content) | Extra user data content for Bastion ec2 | `string` | `""` | no | diff --git a/test/unit-test/main.tf b/test/unit-test/main.tf index f96dde0..c27170f 100644 --- a/test/unit-test/main.tf +++ b/test/unit-test/main.tf @@ -15,9 +15,7 @@ module "bastion_linux" { instance_name = "bastion_linux_${count.index + 1}" # s3 - used for logs and user ssh public keys - bucket_name = "bastion-${count.index + 1}" - bucket_versioning = true - bucket_force_destroy = true + bucket_name = "bastion-${count.index + 1}" # public keys public_key_data = local.public_key_data.keys[local.environment] # logs diff --git a/variables.tf b/variables.tf index 9197d55..213b93e 100644 --- a/variables.tf +++ b/variables.tf @@ -65,24 +65,6 @@ variable "bucket_name" { } } -variable "bucket_versioning" { - type = bool - description = "Enable bucket versioning or not" - validation { - condition = (var.bucket_versioning == true || var.bucket_versioning == false) - error_message = "Variable bucket_versioning must be boolean." - } -} - -variable "bucket_force_destroy" { - type = bool - description = "The bucket and all objects should be destroyed when using true" - validation { - condition = (var.bucket_force_destroy == true || var.bucket_force_destroy == false) - error_message = "Variable bucket_force_destroy must be boolean." - } -} - #### Logs variable "log_auto_clean" { type = string
"down": "0 20 * * *",
"up": "0 5 * * *"
}