[PR #6554/29790df5 backport][stable-6] Don't require api_password whe…

…n api_token_id is used in proxmox_tasks_info (#6561) Don't require api_password when api_token_id is used in proxmox_tasks_info (#6554) * Don't require api_password when api_token_id is used in proxmox_tasks_info * Add changelog fragment * Fix casing. --------- Co-authored-by: Felix Fontein <[email protected]> (cherry picked from commit 29790df) Co-authored-by: Sergei Antipov <[email protected]>
ansible-collections · May 22, 2023 · df01cde · df01cde
1 parent cdd9ced
commit df01cde
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/changelogs/fragments/6554-proxmox-tasks-info-fix-required-password.yaml b/changelogs/fragments/6554-proxmox-tasks-info-fix-required-password.yaml
@@ -0,0 +1,3 @@
+---
+bugfixes:
+  - proxmox_tasks_info - remove ``api_user`` + ``api_password`` constraint from ``required_together`` as it causes to require ``api_password`` even when API token param is used (https://github.com/ansible-collections/community.general/issues/6201).
diff --git a/plugins/modules/proxmox_tasks_info.py b/plugins/modules/proxmox_tasks_info.py
@@ -160,8 +160,7 @@ def main():
 
     module = AnsibleModule(
         argument_spec=module_args,
-        required_together=[('api_token_id', 'api_token_secret'),
-                           ('api_user', 'api_password')],
+        required_together=[('api_token_id', 'api_token_secret')],
         required_one_of=[('api_password', 'api_token_id')],
         supports_check_mode=True)
     result = dict(changed=False)