fix(storage_account): Reaplce identity with enable_identity in storag…

…e_account (#67)

reaplce identity with enable_identity in storage_account

storage_account/README.md

@@ -166,7 +166,7 @@ No modules.
 | <a name="input_cross_tenant_replication_enabled"></a> [cross\_tenant\_replication\_enabled](#input\_cross\_tenant\_replication\_enabled) | (Optional) Should cross Tenant replication be enabled? Defaults to false. | `bool` | `false` | no |
 | <a name="input_custom_domain"></a> [custom\_domain](#input\_custom\_domain) | Custom domain for accessing blob data | <pre>object({<br>    name          = string<br>    use_subdomain = bool<br>  })</pre> | <pre>{<br>  "name": null,<br>  "use_subdomain": false<br>}</pre> | no |
 | <a name="input_domain"></a> [domain](#input\_domain) | (Optional) Specifies the domain of the Storage Account. | `string` | `null` | no |
-| <a name="input_enable_identity"></a> [enable\_identity](#input\_enable\_identity) | (Optional)  Specifies the type of Managed Service Identity that should be configured on this Storage Account. Possible values are SystemAssigned, UserAssigned, SystemAssigned, UserAssigned  (to enable both). | `string` | `null` | no |
+| <a name="input_enable_identity"></a> [enable\_identity](#input\_enable\_identity) | (Optional) If true, set the identity as SystemAssigned | `bool` | `false` | no |
 | <a name="input_enable_low_availability_alert"></a> [enable\_low\_availability\_alert](#input\_enable\_low\_availability\_alert) | Enable the Low Availability alert. Default is true | `bool` | `true` | no |
 | <a name="input_error_404_document"></a> [error\_404\_document](#input\_error\_404\_document) | The absolute path to a custom webpage that should be used when a request is made which does not correspond to an existing file. | `string` | `null` | no |
 | <a name="input_index_document"></a> [index\_document](#input\_index\_document) | The webpage that Azure Storage serves for requests to the root of a website or any subfolder. For example, index.html. The value is case-sensitive. | `string` | `null` | no |

storage_account/main.tf

@@ -64,9 +64,9 @@ resource "azurerm_storage_account" "this" {
   }
 
   dynamic "identity" {
-    for_each = (var.enable_identity != null ? [1] : [])
+    for_each = (var.enable_identity ? [1] : [])
     content {
-      type = var.identity_type
+      type = "SystemAssigned"
     }
   }
 

storage_account/variables.tf

@@ -81,9 +81,9 @@ variable "cross_tenant_replication_enabled" {
 }
 
 variable "enable_identity" {
-  description = "(Optional)  Specifies the type of Managed Service Identity that should be configured on this Storage Account. Possible values are SystemAssigned, UserAssigned, SystemAssigned, UserAssigned  (to enable both)."
-  type        = string
-  default     = null
+  description = "(Optional) If true, set the identity as SystemAssigned"
+  type        = bool
+  default     = false
 }
 
 # Note: If specifying network_rules,