Skip to content

Commit 1baebf1

Browse files
committed
fix(m365): change keyvault name on serial number change
1 parent 278794a commit 1baebf1

3 files changed

Lines changed: 4 additions & 4 deletions

File tree

m365/terraform/main.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ module "app" {
4444
source = "./modules/app"
4545
resource_group_name = azurerm_resource_group.rg.name
4646
location = var.location
47-
resource_prefix = local.name
47+
kv_prefix = "${local.name}-${var.serial_number}"
4848
app_name = var.app_name
4949
image_path = var.image_path
5050
create_app = var.create_app

m365/terraform/modules/app/main.tf

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ data "azurerm_client_config" "current" {}
22
data "azuread_client_config" "current" {}
33

44
locals {
5-
kv_prefix = "${var.resource_prefix}-kv-"
5+
kv_prefix = "${var.kv_prefix}-kv-"
66
kv_unique_id = substr(replace((var.create_app ? azuread_application.app[0].client_id : data.azuread_application.app[0].client_id), "-", ""), 0, 24 - length(local.kv_prefix))
77
}
88

@@ -85,7 +85,7 @@ resource "time_rotating" "cert_rotation" {
8585
# Generate the app registration certificate
8686
resource "azurerm_key_vault_certificate" "cert" {
8787
# Name change forces recreating certificate
88-
name = "${var.resource_prefix}-app-cert-${formatdate("YYYY-MM-DD", time_rotating.cert_rotation.rfc3339)}"
88+
name = "${var.app_name}-app-cert-${formatdate("YYYY-MM-DD", time_rotating.cert_rotation.rfc3339)}"
8989
key_vault_id = azurerm_key_vault.vault.id
9090

9191
certificate_policy {

m365/terraform/modules/app/variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ variable "app_name" {
33
description = "App name. Displayed in Azure console on installed tenants"
44
}
55

6-
variable "resource_prefix" {
6+
variable "kv_prefix" {
77
type = string
88
description = "Prefix to use in resource names"
99
}

0 commit comments

Comments
 (0)