Skip to content

Commit

Permalink
Use cloudidentity name local
Browse files Browse the repository at this point in the history
  • Loading branch information
Adam Smith authored and Adam Smith committed Sep 10, 2024
1 parent 70c6cd8 commit ee353c4
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
2 changes: 2 additions & 0 deletions main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,8 @@ locals {

# if not provided, use the same resource group as the AKS cluster
private_link_resourcegroup = var.private_link_resourcegroup == "" ? var.resource_group_name : var.private_link_resourcegroup

cloudidentity_name = "cloudidentity-azure"

Check warning on line 20 in main.tf

View workflow job for this annotation

GitHub Actions / Root Module Validation / Terraform Lint

local.cloudidentity_name is declared but not used
}

resource "time_sleep" "after_azurerm_role_definition_main" {
Expand Down
8 changes: 4 additions & 4 deletions wayfinder-cloudaccess.tf
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ resource "kubectl_manifest" "wayfinder_cloud_identity_main" {
depends_on = [helm_release.wayfinder]

yaml_body = templatefile("${path.module}/manifests/wayfinder-cloud-identity.yml.tpl", {
name = "cloudidentity-azure"
name = local.cloudidenitiy_name
description = "Cloud managed identity"
client_id = azurerm_user_assigned_identity.wayfinder_main.client_id
tenant_id = data.azurerm_subscription.current.tenant_id
Expand All @@ -34,7 +34,7 @@ resource "kubectl_manifest" "wayfinder_azure_cloudinfo_cloudaccessconfig" {
region = var.location
subscription_id = data.azurerm_subscription.current.subscription_id
tenant_id = data.azurerm_subscription.current.tenant_id
identity = "cloudidentity-azure"
identity = local.cloudidenitiy_name
})
}

Expand All @@ -46,7 +46,7 @@ resource "kubectl_manifest" "wayfinder_azure_privatelinkmanager_cloudaccessconfi
region = var.location
subscription_id = data.azurerm_subscription.current.subscription_id
tenant_id = data.azurerm_subscription.current.tenant_id
identity = "cloudidentity-azure"
identity = local.cloudidenitiy_name
})
}

Expand All @@ -58,6 +58,6 @@ resource "kubectl_manifest" "wayfinder_azure_dnszonemanagement_cloudaccessconfig
region = var.location
subscription_id = data.azurerm_subscription.current.subscription_id
tenant_id = data.azurerm_subscription.current.tenant_id
identity = "cloudidentity-azure"
identity = local.cloudidenitiy_name
})
}

0 comments on commit ee353c4

Please sign in to comment.