From 7bbf4ad7d65320d82214a82045457c1a1ab9647d Mon Sep 17 00:00:00 2001 From: Anusha Hegde Date: Fri, 24 Jan 2020 18:39:05 +0530 Subject: [PATCH] Modifying output for VM already present (#66705) Signed-off-by: Anusha Hegde --- .../modules/cloud/vmware/vmware_guest_cross_vc_clone.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/lib/ansible/modules/cloud/vmware/vmware_guest_cross_vc_clone.py b/lib/ansible/modules/cloud/vmware/vmware_guest_cross_vc_clone.py index c11b8657f63..7f6f9afb0b8 100644 --- a/lib/ansible/modules/cloud/vmware/vmware_guest_cross_vc_clone.py +++ b/lib/ansible/modules/cloud/vmware/vmware_guest_cross_vc_clone.py @@ -237,6 +237,8 @@ class CrossVCCloneManager(PyVmomi): def clone(self): # clone the vm/template on destination VC vm_folder = find_folder_by_name(content=self.destination_content, folder_name=self.params['destination_vm_folder']) + if not vm_folder: + self.module.fail_json(msg="Destination folder does not exist. Please refer to the documentation to correctly specify the folder.") vm_name = self.params['destination_vm_name'] task = self.vm_obj.Clone(folder=vm_folder, name=vm_name, spec=self.clone_spec) wait_for_task(task) @@ -268,7 +270,7 @@ class CrossVCCloneManager(PyVmomi): # Check if vm name already exists in the destination VC vm = find_vm_by_name(content=self.destination_content, vm_name=self.params['destination_vm_name']) if vm: - self.module.fail_json(msg="A VM with the given name already exists") + self.module.exit_json(changed=False, msg="A VM with the given name already exists") datastore_name = self.params['destination_datastore'] datastore_cluster = find_obj(self.destination_content, [vim.StoragePod], datastore_name)