Skip to content

Commit

Permalink
Fix Azure Mgmt Group id format (#14)
Browse files Browse the repository at this point in the history
  • Loading branch information
ravinadhruve10 authored Nov 30, 2023
1 parent 44a1c8b commit 2c2ca5b
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions modules/services/service-principal/organizational.tf
Original file line number Diff line number Diff line change
Expand Up @@ -2,13 +2,14 @@
# Fetch the management groups for customer tenant and onboard subscriptions under them
#---------------------------------------------------------------------------------------------
data "azurerm_management_group" "root_management_group" {
count = var.is_organizational ? 1 : 0
count = var.is_organizational && length(var.management_group_ids) == 0 ? 1 : 0
display_name = "Tenant Root Group"
}

locals {
# when empty, this will be the root management group whose default display name is "Tenant root group"
management_groups = var.is_organizational && length(var.management_group_ids) == 0 ? [data.azurerm_management_group.root_management_group[0].id] : toset(var.management_group_ids)
management_groups = var.is_organizational && length(var.management_group_ids) == 0 ? [data.azurerm_management_group.root_management_group[0].id] : toset(
[for m in var.management_group_ids : format("%s/%s", "/providers/Microsoft.Management/managementGroups",m)])
}

#---------------------------------------------------------------------------------------------
Expand Down Expand Up @@ -42,4 +43,4 @@ resource "azurerm_role_assignment" "sysdig_vm_user_for_tenant" {
scope = each.key
role_definition_name = "Virtual Machine User Login"
principal_id = azuread_service_principal.sysdig_sp.object_id
}
}

0 comments on commit 2c2ca5b

Please sign in to comment.