Skip to content

Commit

Permalink
removed the shared storage veriable.
Browse files Browse the repository at this point in the history
  • Loading branch information
james-annages authored Feb 18, 2025
1 parent 992eebc commit 15c8f98
Show file tree
Hide file tree
Showing 5 changed files with 2 additions and 17 deletions.
6 changes: 0 additions & 6 deletions templates/workspaces/base/parameters.json
Original file line number Diff line number Diff line change
Expand Up @@ -165,12 +165,6 @@
"source": {
"env": "ENABLE_BACKUP"
}
},
{
"name": "shared_storage_name",
"source": {
"env": "SHARED_STORAGE_NAME"
}
}
]
}
4 changes: 0 additions & 4 deletions templates/workspaces/base/porter.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -130,10 +130,6 @@ parameters:
type: boolean
default: true
description: "Enable backups for the workspace, including the vm's & shared storage."
- name: shared_storage_name
type: string
default: "vm-shared-storage"
description: "The name of the shared storage used for the workspace VMs"
- name: backup_vault_name
type: string
default: ""
Expand Down
1 change: 1 addition & 0 deletions templates/workspaces/base/terraform/locals.tf
Original file line number Diff line number Diff line change
Expand Up @@ -10,4 +10,5 @@ locals {
}
kv_encryption_key_name = "tre-encryption-${local.workspace_resource_name_suffix}"
encryption_identity_name = "id-encryption-${var.tre_id}-${local.short_workspace_id}"
shared_storage_name = "vm-shared-storage"
}
6 changes: 0 additions & 6 deletions templates/workspaces/base/terraform/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -81,12 +81,6 @@ variable "enable_backup"{
description = "Enable backups for the workspace"
}

variable "shared_storage_name" {
type = string
default = "vm-shared-storage"
description = "Name of the VM Shared Storage"
}

# These variables are only passed in if you are not registering an AAD
# application as they need passing back out
variable "app_role_id_workspace_owner" {
Expand Down
2 changes: 1 addition & 1 deletion templates/workspaces/base/terraform/workspace.tf
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,7 @@ module "backup" {
azurerm_storage_account_id = azurerm_storage_account.stg.id
enable_local_debugging = var.enable_local_debugging
enable_cmk_encryption = var.enable_cmk_encryption
shared_storage_name = var.shared_storage_name
shared_storage_name = local.shared_storage_name


depends_on = [
Expand Down

0 comments on commit 15c8f98

Please sign in to comment.