From bcacd331c365b2b9cd656f9b58b36044892c28cc Mon Sep 17 00:00:00 2001 From: Megan Marsh Date: Fri, 29 Mar 2019 15:12:13 -0700 Subject: [PATCH] change cpu to cpus and ram_size to memory in order to bring hyper v into line with other builder defaults --- builder/hyperv/iso/builder.go | 8 ++++---- builder/hyperv/vmcx/builder.go | 4 ++-- .../source/docs/builders/hyperv-iso.html.md.erb | 16 ++++++++-------- 3 files changed, 14 insertions(+), 14 deletions(-) diff --git a/builder/hyperv/iso/builder.go b/builder/hyperv/iso/builder.go index 61add6777..77f5a9255 100644 --- a/builder/hyperv/iso/builder.go +++ b/builder/hyperv/iso/builder.go @@ -68,7 +68,7 @@ type Config struct { // The size, in megabytes, of the computer memory in the VM. // By default, this is 1024 (about 1 GB). - RamSize uint `mapstructure:"ram_size"` + RamSize uint `mapstructure:"memory"` // SecondaryDvdImages []string `mapstructure:"secondary_iso_images"` @@ -87,7 +87,7 @@ type Config struct { SwitchVlanId string `mapstructure:"switch_vlan_id"` MacAddress string `mapstructure:"mac_address"` VlanId string `mapstructure:"vlan_id"` - Cpu uint `mapstructure:"cpu"` + Cpu uint `mapstructure:"cpus"` Generation uint `mapstructure:"generation"` EnableMacSpoofing bool `mapstructure:"enable_mac_spoofing"` UseLegacyNetworkAdapter bool `mapstructure:"use_legacy_network_adapter"` @@ -593,10 +593,10 @@ func (b *Builder) checkRamSize() error { log.Println(fmt.Sprintf("%s: %v", "RamSize", b.config.RamSize)) if b.config.RamSize < MinRamSize { - return fmt.Errorf("ram_size: Virtual machine requires memory size >= %v MB, but defined: %v", + return fmt.Errorf("memory: Virtual machine requires memory size >= %v MB, but defined: %v", MinRamSize, b.config.RamSize) } else if b.config.RamSize > MaxRamSize { - return fmt.Errorf("ram_size: Virtual machine requires memory size <= %v MB, but defined: %v", + return fmt.Errorf("memory: Virtual machine requires memory size <= %v MB, but defined: %v", MaxRamSize, b.config.RamSize) } diff --git a/builder/hyperv/vmcx/builder.go b/builder/hyperv/vmcx/builder.go index 9ecca1fe5..a611a300d 100644 --- a/builder/hyperv/vmcx/builder.go +++ b/builder/hyperv/vmcx/builder.go @@ -50,7 +50,7 @@ type Config struct { // The size, in megabytes, of the computer memory in the VM. // By default, this is 1024 (about 1 GB). - RamSize uint `mapstructure:"ram_size"` + RamSize uint `mapstructure:"memory"` // SecondaryDvdImages []string `mapstructure:"secondary_iso_images"` @@ -85,7 +85,7 @@ type Config struct { SwitchVlanId string `mapstructure:"switch_vlan_id"` MacAddress string `mapstructure:"mac_address"` VlanId string `mapstructure:"vlan_id"` - Cpu uint `mapstructure:"cpu"` + Cpu uint `mapstructure:"cpus"` Generation uint `mapstructure:"generation"` EnableMacSpoofing bool `mapstructure:"enable_mac_spoofing"` EnableDynamicMemory bool `mapstructure:"enable_dynamic_memory"` diff --git a/website/source/docs/builders/hyperv-iso.html.md.erb b/website/source/docs/builders/hyperv-iso.html.md.erb index 72e983706..c739f077c 100644 --- a/website/source/docs/builders/hyperv-iso.html.md.erb +++ b/website/source/docs/builders/hyperv-iso.html.md.erb @@ -78,7 +78,7 @@ builder. - `configuration_version` (string) - This allows you to set the vm version when calling New-VM to generate the vm. -- `cpu` (number) - The number of CPUs the virtual machine should use. If +- `cpus` (number) - The number of CPUs the virtual machine should use. If this isn't specified, the default is 1 CPU. - `differencing_disk` (boolean) - If true enables differencing disks. Only @@ -179,7 +179,7 @@ builder. created, must be empty prior to running the builder. By default this is "output-BUILDNAME" where "BUILDNAME" is the name of the build. -- `ram_size` (number) - The amount, in megabytes, of RAM to assign to the +- `memory` (number) - The amount, in megabytes, of RAM to assign to the VM. By default, this is 1 GB. - `secondary_iso_images` (array of strings) - A list of ISO paths to @@ -379,8 +379,8 @@ Packer config: "winrm_password": "vagrant", "winrm_timeout" : "4h", "shutdown_command": "f:\\run-sysprep.cmd", - "ram_size": 4096, - "cpu": 4, + "memory": 4096, + "cpus": 4, "generation": 2, "switch_name":"LAN", "enable_secure_boot":true @@ -887,8 +887,8 @@ virtual switch with an `External` connection type. { "variables": { "vm_name": "ubuntu-xenial", - "cpu": "2", - "ram_size": "1024", + "cpus": "2", + "memory": "1024", "disk_size": "21440", "iso_url": "http://releases.ubuntu.com/16.04/ubuntu-16.04.1-server-amd64.iso", "iso_checksum_type": "sha1", @@ -923,8 +923,8 @@ virtual switch with an `External` connection type. "boot" ], "shutdown_command": "echo 'packer' | sudo -S -E shutdown -P now", - "ram_size": "{{user `ram_size`}}", - "cpu": "{{user `cpu`}}", + "memory": "{{user `memory`}}", + "cpus": "{{user `cpu`}}", "generation": 2, "enable_secure_boot": false }