vm_name now not a required variable

changelogs/fragments/93-bugfix-vm_name-required.yml

@@ -0,0 +1,3 @@
+---
+minor_changes:
+  - info - Made vm_name variable required only when state is set to present in content_template module

plugins/modules/content_template.py

@@ -36,8 +36,8 @@
     vm_name:
         description:
         - The name of the VM to be used to create template.
+        - This attribute is required only when the state attribute is set to present
         type: str
-        required: true
     host:
         description:
         - Host onto which the virtual machine template should be placed.
@@ -206,7 +206,7 @@ def main():
     argument_spec.update(
         template=dict(type='str', required=True),
         library=dict(type='str', required=True),
-        vm_name=dict(type='str', required=True),
+        vm_name=dict(type='str'),
         host=dict(type='str'),
         cluster=dict(type='str'),
         resource_pool=dict(type='str'),
@@ -217,6 +217,7 @@ def main():
         argument_spec=argument_spec,
         supports_check_mode=True,
         required_one_of=[('host', 'resource_pool', 'cluster')],
+        required_if=[('state', 'present', ["vm_name"])]
     )
 
     result = {'failed': False, 'changed': False}

tests/integration/targets/vmware_content_template/tasks/main.yml

@@ -69,7 +69,6 @@
         port: "{{ vcenter_port }}"
         template: "{{ template_name }}"
         library: "{{ library }}"
-        vm_name: "{{ vm }}"
         host: "{{ template_host }}"
         state: absent
       when: not run_on_simulator