Fix conditionals on federated role assignments

appvia · Oct 26, 2023 · 8c8613e · 8c8613e
1 parent 9728ed6
commit 8c8613e
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/modules/cloudaccess/wf_cluster_manager.tf b/modules/cloudaccess/wf_cluster_manager.tf
@@ -41,7 +41,7 @@ resource "azurerm_role_assignment" "clustermanager" {
 }
 
 resource "azurerm_role_assignment" "clustermanager_federated" {
-  count = var.enable_cloud_info && (var.from_aws || var.from_gcp) ? 1 : 0
+  count = var.enable_cluster_manager && (var.from_aws || var.from_gcp) ? 1 : 0
 
   scope                = data.azurerm_subscription.primary.id
   role_definition_name = azurerm_role_definition.clustermanager[0].name

diff --git a/modules/cloudaccess/wf_dnszone_manager.tf b/modules/cloudaccess/wf_dnszone_manager.tf
@@ -41,7 +41,7 @@ resource "azurerm_role_assignment" "dnszonemanager" {
 }
 
 resource "azurerm_role_assignment" "dnszonemanager_federated" {
-  count = var.enable_cloud_info && (var.from_aws || var.from_gcp) ? 1 : 0
+  count = var.enable_dns_zone_manager && (var.from_aws || var.from_gcp) ? 1 : 0
 
   scope                = data.azurerm_subscription.primary.id
   role_definition_name = azurerm_role_definition.dnszonemanager[0].name

diff --git a/modules/cloudaccess/wf_network_manager.tf b/modules/cloudaccess/wf_network_manager.tf
@@ -41,7 +41,7 @@ resource "azurerm_role_assignment" "networkmanager" {
 }
 
 resource "azurerm_role_assignment" "networkmanager_federated" {
-  count = var.enable_cloud_info && (var.from_aws || var.from_gcp) ? 1 : 0
+  count = var.enable_network_manager && (var.from_aws || var.from_gcp) ? 1 : 0
 
   scope                = data.azurerm_subscription.primary.id
   role_definition_name = azurerm_role_definition.networkmanager[0].name