Merge pull request #2 from data-platform-hq/fix-identity

fix: identity and free tier
data-platform-hq · Jan 5, 2023 · d2e74f3 · d2e74f3
2 parents 4faf289 + 565d489
commit d2e74f3
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/main.tf b/main.tf
@@ -5,9 +5,12 @@ resource "azurerm_static_site" "this" {
   sku_tier            = var.sku_tier
   sku_size            = var.sku_size
   tags                = var.tags
-  identity {
-    type         = var.identity_ids == null ? "SystemAssigned" : "SystemAssigned, UserAssigned"
-    identity_ids = var.identity_ids
+  dynamic "identity" {
+    for_each = var.sku_tier == "Free" ? toset([]) : toset([1])
+    content {
+      type         = var.identity_ids == null ? "SystemAssigned" : "SystemAssigned, UserAssigned"
+      identity_ids = var.identity_ids
+    }
   }
 }
 

diff --git a/variables.tf b/variables.tf
@@ -30,8 +30,9 @@ variable "name" {
 
 variable "identity_ids" {
   type        = list(string)
-  description = "List of user assigned identity IDs"
+  description = "List of user assigned identity IDs (cannot be used with Free SKU)"
   default     = null
+  # TODO: add validation after fix https://github.com/hashicorp/terraform/issues/25609
 }
 
 variable "sku_tier" {