From 7dfc0a6d1e1702e3e9fb6d4e58cbc741eb37482a Mon Sep 17 00:00:00 2001 From: Adam Heeren Date: Wed, 23 Mar 2016 16:46:33 -0400 Subject: [PATCH] Added windows clone options in vsphere and documented them --- .../resource_vsphere_virtual_machine.go | 163 ++++++++++++++---- .../vsphere/r/virtual_machine.html.markdown | 11 +- 2 files changed, 136 insertions(+), 38 deletions(-) diff --git a/builtin/providers/vsphere/resource_vsphere_virtual_machine.go b/builtin/providers/vsphere/resource_vsphere_virtual_machine.go index a26afde3d..cd0ce3ea1 100644 --- a/builtin/providers/vsphere/resource_vsphere_virtual_machine.go +++ b/builtin/providers/vsphere/resource_vsphere_virtual_machine.go @@ -44,25 +44,35 @@ type hardDisk struct { initType string } +//Additional options Vsphere can use clones of windows machines +type windowsOptConfig struct { + productKey string + adminPassword string + domainUser string + domain string + domainUserPassword string +} + type virtualMachine struct { - name string - folder string - datacenter string - cluster string - resourcePool string - datastore string - linkedClone bool - vcpu int - memoryMb int64 - template string - networkInterfaces []networkInterface - hardDisks []hardDisk - gateway string - domain string - timeZone string - dnsSuffixes []string - dnsServers []string - customConfigurations map[string](types.AnyType) + name string + folder string + datacenter string + cluster string + resourcePool string + datastore string + linkedClone bool + vcpu int + memoryMb int64 + template string + networkInterfaces []networkInterface + hardDisks []hardDisk + gateway string + domain string + timeZone string + dnsSuffixes []string + dnsServers []string + windowsOptionalConfig windowsOptConfig + customConfigurations map[string](types.AnyType) } func (v virtualMachine) Path() string { @@ -171,6 +181,44 @@ func resourceVSphereVirtualMachine() *schema.Resource { Optional: true, ForceNew: true, }, + "windows_opt_config": &schema.Schema{ + Type: schema.TypeList, + Optional: true, + ForceNew: true, + Elem: &schema.Resource{ + Schema: map[string]*schema.Schema{ + "product_key": &schema.Schema{ + Type: schema.TypeString, + Required: true, + ForceNew: true, + }, + + "admin_password": &schema.Schema{ + Type: schema.TypeString, + Optional: true, + ForceNew: true, + }, + + "domain_user": &schema.Schema{ + Type: schema.TypeString, + Optional: true, + ForceNew: true, + }, + + "domain": &schema.Schema{ + Type: schema.TypeString, + Optional: true, + ForceNew: true, + }, + + "domain_user_password": &schema.Schema{ + Type: schema.TypeString, + Optional: true, + ForceNew: true, + }, + }, + }, + }, "network_interface": &schema.Schema{ Type: schema.TypeList, @@ -386,6 +434,28 @@ func resourceVSphereVirtualMachineCreate(d *schema.ResourceData, meta interface{ log.Printf("[DEBUG] network_interface init: %v", networks) } + if vL, ok := d.GetOk("windows_opt_config"); ok { + var winOpt windowsOptConfig + custom_configs := (vL.([]interface{}))[0].(map[string]interface{}) + if v, ok := custom_configs["admin_password"].(string); ok && v != "" { + winOpt.adminPassword = v + } + if v, ok := custom_configs["domain"].(string); ok && v != "" { + winOpt.domain = v + } + if v, ok := custom_configs["domain_user"].(string); ok && v != "" { + winOpt.domainUser = v + } + if v, ok := custom_configs["product_key"].(string); ok && v != "" { + winOpt.productKey = v + } + if v, ok := custom_configs["domain_user_password"].(string); ok && v != "" { + winOpt.domainUserPassword = v + } + vm.windowsOptionalConfig = winOpt + log.Printf("[DEBUG] windows config init: %v", winOpt) + } + if vL, ok := d.GetOk("disk"); ok { disks := make([]hardDisk, len(vL.([]interface{}))) for i, v := range vL.([]interface{}) { @@ -1207,27 +1277,46 @@ func (vm *virtualMachine) deployVirtualMachine(c *govmomi.Client) error { var timeZone int timeZone, err := strconv.Atoi(vm.timeZone) if err != nil { - return fmt.Errorf("Error reading base VM properties: %s", err) + return fmt.Errorf("Error converting TimeZone: %s", err) } + + guiUnattended := types.CustomizationGuiUnattended{ + AutoLogon: false, + AutoLogonCount: 1, + TimeZone: timeZone, + } + + customIdentification := types.CustomizationIdentification{} + + userData := types.CustomizationUserData{ + ComputerName: &types.CustomizationFixedName{ + Name: strings.Split(vm.name, ".")[0], + }, + ProductId: vm.windowsOptionalConfig.productKey, + FullName: "terraform", + OrgName: "terraform", + } + + if vm.windowsOptionalConfig.domainUserPassword != "" && vm.windowsOptionalConfig.domainUser != "" && vm.windowsOptionalConfig.domain != "" { + customIdentification.DomainAdminPassword = &types.CustomizationPassword{ + PlainText: true, + Value: vm.windowsOptionalConfig.domainUserPassword, + } + customIdentification.DomainAdmin = vm.windowsOptionalConfig.domainUser + customIdentification.JoinDomain = vm.windowsOptionalConfig.domain + } + + if vm.windowsOptionalConfig.adminPassword != "" { + guiUnattended.Password = &types.CustomizationPassword{ + PlainText: true, + Value: vm.windowsOptionalConfig.adminPassword, + } + } + identity_options = &types.CustomizationSysprep{ - GuiUnattended: types.CustomizationGuiUnattended{ - AutoLogon: false, - AutoLogonCount: 1, - Password: &types.CustomizationPassword{ - PlainText: true, - Value: "NULL", - }, - TimeZone: timeZone, - }, - Identification: types.CustomizationIdentification{}, - UserData: types.CustomizationUserData{ - ComputerName: &types.CustomizationFixedName{ - Name: strings.Split(vm.name, ".")[0], - }, - FullName: "LSTTE", - OrgName: "LSTTE", - ProductId: "ruh roh", - }, + GuiUnattended: guiUnattended, + Identification: customIdentification, + UserData: userData, } } else { identity_options = &types.CustomizationLinuxPrep{ diff --git a/website/source/docs/providers/vsphere/r/virtual_machine.html.markdown b/website/source/docs/providers/vsphere/r/virtual_machine.html.markdown index 9812a0aed..92b038ecd 100644 --- a/website/source/docs/providers/vsphere/r/virtual_machine.html.markdown +++ b/website/source/docs/providers/vsphere/r/virtual_machine.html.markdown @@ -41,12 +41,14 @@ The following arguments are supported: * `resource_pool` (Optional) The name of a Resource Pool in which to launch the virtual machine * `gateway` - (Optional) Gateway IP address to use for all network interfaces * `domain` - (Optional) A FQDN for the virtual machine; defaults to "vsphere.local" -* `time_zone` - (Optional) The [time zone](https://www.vmware.com/support/developer/vc-sdk/visdk41pubs/ApiReference/timezone.html) to set on the virtual machine. Defaults to "Etc/UTC" +* `time_zone` - (Optional) The [Linux](https://www.vmware.com/support/developer/vc-sdk/visdk41pubs/ApiReference/timezone.html) or [Windows](https://msdn.microsoft.com/en-us/library/ms912391.aspx) time zone to set on the virtual machine. Defaults to "Etc/UTC" * `dns_suffixes` - (Optional) List of name resolution suffixes for the virtual network adapter * `dns_servers` - (Optional) List of DNS servers for the virtual network adapter; defaults to 8.8.8.8, 8.8.4.4 * `network_interface` - (Required) Configures virtual network interfaces; see [Network Interfaces](#network-interfaces) below for details. * `disk` - (Required) Configures virtual disks; see [Disks](#disks) below for details * `boot_delay` - (Optional) Time in seconds to wait for machine network to be ready. +* `windows_opt_config` - (Optional) Extra options for clones of Windows machines. +* `linkedClone` - (Optional) Specifies if the new machine is a [linked clone](https://www.vmware.com/support/ws5/doc/ws_clone_overview.html#wp1036396) of another machine or not. * `custom_configuration_parameters` - (Optional) Map of values that is set as virtual machine custom configurations. The `network_interface` block supports: @@ -61,6 +63,13 @@ removed in a future version: * `ip_address` - __Deprecated, please use `ipv4_address` instead_. * `subnet_mask` - __Deprecated, please use `ipv4_prefix_length` instead_. +The `windows_opt_config` block supports: + +* `product_key` - (Optional) Serial number for new installation of Windows. This serial number is ignored if the original guest operating system was installed using a volume-licensed CD. +* `admin_password` - (Optional) The password for the new `administrator` account. Omit for passwordless admin (using `""` does not work). +* `domain` - (Optional) Domain that the new machine will be placed into. If `domain`, `domain_user`, and `domain_user_password` are not all set, all three will be ignored. +* `domain_user` - (Optional) User that is a member of the specified domain. +* `domain_user_password` - (Optional) Password for domain user, in plain text. The `disk` block supports: