fix: Velero storage account kind (#163)

pagopa · Sep 27, 2023 · a9fb3c2 · a9fb3c2
1 parent 27835c8
commit a9fb3c2
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/kubernetes_cluster_velero/01_main.tf b/kubernetes_cluster_velero/01_main.tf
@@ -11,7 +11,7 @@ module "velero_storage_account" {
   source = "git::https://github.com/pagopa/terraform-azurerm-v3.git//storage_account?ref=v7.2.0"
 
   name                            = "${local.sa_prefix}velerosa"
-  account_kind                    = "BlobStorage"
+  account_kind                    = var.storage_account_kind
   account_tier                    = var.storage_account_tier
   account_replication_type        = var.storage_account_replication_type
   blob_versioning_enabled         = true

diff --git a/kubernetes_cluster_velero/99_variables.tf b/kubernetes_cluster_velero/99_variables.tf
@@ -80,3 +80,9 @@ variable "storage_account_replication_type" {
   description = "(Optional) Replication type used for the backup storage account"
   default     = "ZRS"
 }
+
+variable "storage_account_kind" {
+  type        = string
+  default     = "StorageV2"
+  description = "(Optional) Defines the Kind of account. Valid options are BlobStorage, BlockBlobStorage, FileStorage, Storage and StorageV2. Defaults to StorageV2"
+}
diff --git a/kubernetes_cluster_velero/README.md b/kubernetes_cluster_velero/README.md
@@ -88,6 +88,7 @@ This is achievable using the utility script `k8setup.sh` included in the aks-set
 | <a name="input_prefix"></a> [prefix](#input\_prefix) | (Required) Prefix used in the Velero dedicated resource names | `string` | n/a | yes |
 | <a name="input_private_endpoint_subnet_id"></a> [private\_endpoint\_subnet\_id](#input\_private\_endpoint\_subnet\_id) | (Optional) Subnet id where to create the private endpoint for backups storage account | `string` | `null` | no |
 | <a name="input_resource_group_name"></a> [resource\_group\_name](#input\_resource\_group\_name) | (Required) Name of the resource group in which the backup storage account is located | `string` | n/a | yes |
+| <a name="input_storage_account_kind"></a> [storage\_account\_kind](#input\_storage\_account\_kind) | (Optional) Defines the Kind of account. Valid options are BlobStorage, BlockBlobStorage, FileStorage, Storage and StorageV2. Defaults to StorageV2 | `string` | `"StorageV2"` | no |
 | <a name="input_storage_account_private_dns_zone_id"></a> [storage\_account\_private\_dns\_zone\_id](#input\_storage\_account\_private\_dns\_zone\_id) | (Optional) Storage account private dns zone id, used in the private endpoint creation | `string` | `null` | no |
 | <a name="input_storage_account_replication_type"></a> [storage\_account\_replication\_type](#input\_storage\_account\_replication\_type) | (Optional) Replication type used for the backup storage account | `string` | `"ZRS"` | no |
 | <a name="input_storage_account_tier"></a> [storage\_account\_tier](#input\_storage\_account\_tier) | (Optional) Tier used for the backup storage account | `string` | `"Standard"` | no |