Fix versioning for S3 bucket

modules/vault-cluster/main.tf

@@ -280,10 +280,6 @@ resource "aws_s3_bucket" "vault_storage" {
     var.s3_bucket_tags,
   )
 
-  versioning {
-    enabled = var.enable_s3_bucket_versioning
-  }
-
   # aws_launch_configuration.launch_configuration in this module sets create_before_destroy to true, which means
   # everything it depends on, including this resource, must set it as well, or you'll get cyclic dependency errors
   # when you try to do a terraform destroy.
@@ -292,6 +288,14 @@ resource "aws_s3_bucket" "vault_storage" {
   }
 }
 
+resource "aws_s3_bucket_versioning" "vault_storage" {
+  count  = var.enable_s3_backend ? 1 : 0
+  bucket = aws_s3_bucket.vault_storage[count.index].id
+  versioning_configuration {
+    status = var.enable_s3_bucket_versioning
+  }
+}
+
 resource "aws_iam_role_policy" "vault_s3" {
   count = var.enable_s3_backend ? 1 : 0
   name  = "vault_s3"