Skip to content

Commit 5378bd4

Browse files
authored
Fix: resolve duplicate Key Vault role assignment conflict in the workload zone (#998)
1 parent 79348bf commit 5378bd4

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

deploy/terraform/terraform-units/modules/sap_landscape/key_vault_sap_landscape.tf

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ resource "azurerm_management_lock" "key_vault" {
100100

101101
resource "azurerm_role_assignment" "role_assignment_msi" {
102102
provider = azurerm.deployer
103-
count = var.key_vault.enable_rbac_authorization && var.options.assign_permissions ? 1 : 0
103+
count = var.key_vault.user.exists && var.key_vault.enable_rbac_authorization && var.options.assign_permissions ? 1 : 0
104104
scope = var.key_vault.user.exists ? (
105105
data.azurerm_key_vault.kv_user[0].id) : (
106106
azurerm_key_vault.kv_user[0].id
@@ -132,7 +132,7 @@ resource "azurerm_role_assignment" "role_assignment_vault_ssi" {
132132

133133
resource "azurerm_role_assignment" "role_assignment_msi_officer" {
134134
provider = azurerm.deployer
135-
count = var.key_vault.enable_rbac_authorization && var.options.assign_permissions ? 1 : 0
135+
count = var.key_vault.user.exists && var.key_vault.enable_rbac_authorization && var.options.assign_permissions ? 1 : 0
136136
scope = var.key_vault.user.exists ? (
137137
data.azurerm_key_vault.kv_user[0].id) : (
138138
azurerm_key_vault.kv_user[0].id

0 commit comments

Comments
 (0)