Rename variable to template_storage_policy_id

Signed-off-by: Stéphane Este-Gracias <sestegra@gmail.com>

Author: sestegra

main.tf

@@ -115,7 +115,7 @@ resource "vsphere_virtual_machine" "Linux" {
       thin_provisioned  = data.vsphere_virtual_machine.template.disks[template_disks.key].thin_provisioned
       eagerly_scrub     = data.vsphere_virtual_machine.template.disks[template_disks.key].eagerly_scrub
       datastore_id      = var.disk_datastore != "" ? data.vsphere_datastore.disk_datastore[0].id : null
-      storage_policy_id = length(var.disk_storage_policy_id) > 0 ? var.disk_storage_policy_id[template_disks.key] : null
+      storage_policy_id = length(var.template_storage_policy_id) > 0 ? var.template_storage_policy_id[template_disks.key] : null
     }
   }
   // Additional disks defined by Terraform config
@@ -231,7 +231,7 @@ resource "vsphere_virtual_machine" "Windows" {
       thin_provisioned  = data.vsphere_virtual_machine.template.disks[template_disks.key].thin_provisioned
       eagerly_scrub     = data.vsphere_virtual_machine.template.disks[template_disks.key].eagerly_scrub
       datastore_id      = var.disk_datastore != "" ? data.vsphere_datastore.disk_datastore[0].id : null
-      storage_policy_id = length(var.disk_storage_policy_id) > 0 ? var.disk_storage_policy_id[template_disks.key] : null
+      storage_policy_id = length(var.template_storage_policy_id) > 0 ? var.template_storage_policy_id[template_disks.key] : null
     }
   }
 

tests/sanity/main.tf

@@ -17,38 +17,38 @@ resource "vsphere_tag" "tag" {
 
 variable "vm" {
   type = map(object({
-    vmname                 = string
-    vmtemp                 = string
-    dc                     = string
-    vmrp                   = string
-    vmfolder               = string
-    datastore              = string
-    disk_storage_policy_id = list(string)
-    is_windows_image       = bool
-    tags                   = map(string)
-    instances              = number
-    network                = map(list(string))
-    vmgateway              = string
-    dns_servers            = list(string)
-    data_disk              = map(map(string))
+    vmname                     = string
+    vmtemp                     = string
+    dc                         = string
+    vmrp                       = string
+    vmfolder                   = string
+    datastore                  = string
+    template_storage_policy_id = list(string)
+    is_windows_image           = bool
+    tags                       = map(string)
+    instances                  = number
+    network                    = map(list(string))
+    vmgateway                  = string
+    dns_servers                = list(string)
+    data_disk                  = map(map(string))
   }))
 }
 
 module "example-server-basic" {
-  source                 = "../../"
-  for_each               = var.vm
-  tag_depends_on         = [vsphere_tag.tag.id]
-  tags                   = each.value.tags
-  vmtemp                 = each.value.vmtemp
-  is_windows_image       = each.value.is_windows_image
-  instances              = each.value.instances
-  vmname                 = each.value.vmname
-  vmrp                   = each.value.vmrp
-  vmfolder               = each.value.vmfolder
-  network                = each.value.network
-  vmgateway              = each.value.vmgateway
-  dc                     = each.value.dc
-  datastore              = each.value.datastore
-  disk_storage_policy_id = each.value.disk_storage_policy_id
-  data_disk              = each.value.data_disk
+  source                     = "../../"
+  for_each                   = var.vm
+  tag_depends_on             = [vsphere_tag.tag.id]
+  tags                       = each.value.tags
+  vmtemp                     = each.value.vmtemp
+  is_windows_image           = each.value.is_windows_image
+  instances                  = each.value.instances
+  vmname                     = each.value.vmname
+  vmrp                       = each.value.vmrp
+  vmfolder                   = each.value.vmfolder
+  network                    = each.value.network
+  vmgateway                  = each.value.vmgateway
+  dc                         = each.value.dc
+  datastore                  = each.value.datastore
+  template_storage_policy_id = each.value.template_storage_policy_id
+  data_disk                  = each.value.data_disk
 }

variables.tf

@@ -52,7 +52,7 @@ variable "disk_datastore" {
   default     = ""
 }
 
-variable "disk_storage_policy_id" {
+variable "template_storage_policy_id" {
   description = "List of UUIDs of the storage policy to assign to the template disk."
   type        = list(any)
   default     = []