From b851321b657b0718a98b1a4822fe7a27170396da Mon Sep 17 00:00:00 2001 From: mbusbee Date: Thu, 9 Aug 2018 13:20:46 -0400 Subject: [PATCH] VMware: datastore selection (#35812) This fix adds additional parameter 'datastore'. This parameter provides a flexibility to specify custom datastore or datastore cluster to deploy new virtual machine from template (which is located in different datastore or datastore cluster from virtual machine's datastore or datastore cluster). Signed-off-by: Abhijeet Kasurde --- .../modules/cloud/vmware/vmware_guest.py | 39 ++++++++++++++++++- 1 file changed, 38 insertions(+), 1 deletion(-) diff --git a/lib/ansible/modules/cloud/vmware/vmware_guest.py b/lib/ansible/modules/cloud/vmware/vmware_guest.py index 0fb44d3545b..58270a23f66 100644 --- a/lib/ansible/modules/cloud/vmware/vmware_guest.py +++ b/lib/ansible/modules/cloud/vmware/vmware_guest.py @@ -325,6 +325,14 @@ options: - This parameter is case sensitive. - If set, then overrides C(customization) parameter values. version_added: '2.6' + datastore: + description: + - Specify datastore or datastore cluster to provision virtual machine. + - 'This will take precendence over "disk.datastore" parameter.' + - This parameter is useful to override datastore or datastore cluster setting. + - For example, when user has different datastore or datastore cluster for templates and virtual machines. + - Please see example for more usage. + version_added: '2.7' extends_documentation_fragment: vmware.documentation ''' @@ -508,6 +516,22 @@ EXAMPLES = r''' uuid: "{{ vm_uuid }}" state: poweredoff delegate_to: localhost + +- name: Deploy a virtual machine in a datastore different from the datastore of the template + vmware_guest: + hostname: "{{ vcenter_hostname }}" + username: "{{ vcenter_username }}" + password: "{{ vcenter_password }}" + name: "{{ vm_name }}" + state: present + template: "{{ template_name }}" + # Here datastore can be different which holds template + datastore: "{{ virtual_machine_datastore }}" + hardware: + memory_mb: 512 + num_cpus: 2 + scsi: paravirtual + delegate_to: localhost ''' RETURN = r''' @@ -1987,7 +2011,19 @@ class PyVmomiHelper(PyVmomi): resource_pool = self.get_resource_pool() # set the destination datastore for VM & disks - (datastore, datastore_name) = self.select_datastore(vm_obj) + if self.params['datastore']: + # Give precendence to datastore value provided by user + # User may want to deploy VM to specific datastore. + datastore_name = self.params['datastore'] + # Check if user has provided datastore cluster first + datastore_cluster = self.cache.find_obj(self.content, [vim.StoragePod], datastore_name) + if datastore_cluster: + # If user specified datastore cluster so get recommended datastore + datastore_name = self.get_recommended_datastore(datastore_cluster_obj=datastore_cluster) + # Check if get_recommended_datastore or user specified datastore exists or not + datastore = self.cache.find_obj(self.content, [vim.Datastore], datastore_name) + else: + (datastore, datastore_name) = self.select_datastore(vm_obj) self.configspec = vim.vm.ConfigSpec() self.configspec.deviceChange = [] @@ -2295,6 +2331,7 @@ def main(): customization=dict(type='dict', default={}, no_log=True), customization_spec=dict(type='str', default=None), vapp_properties=dict(type='list', default=[]), + datastore=dict(type='str'), ) module = AnsibleModule(argument_spec=argument_spec,