Fix proxmox_storage does not add PBS parameters to API payload (#170) (#171)

* Fix proxmox_storage does not add PBS parameters to API payload (#170)

* Update proxmox_storage.py

Fix whitespacing to fix sanity test

* chore: Add changelog fragment for #171

* Update changelogs/fragments/171.bugfix.yml

---------

Co-authored-by: Carlo Luzi <[email protected]>
Co-authored-by: Jeffrey van Pelt <[email protected]>

Author: 3 people

changelogs/fragments/171.bugfix.yml

@@ -0,0 +1,2 @@
+bugfixes:
+  - proxmox_storage - fixed adding PBS-type storage by ensuring its parameters (server, datastore, etc.) are correctly sent to the Proxmox API (https://github.com/ansible-collections/community.proxmox/pull/171).

plugins/modules/proxmox_storage.py

@@ -350,7 +350,14 @@ def add_storage(self):
             datastore = pbs_options.get('datastore')
             fingerprint = pbs_options.get('fingerprint')
             if not all([server, datastore, username, password]):
-                self.module.fail_json(msg="PBS storage requires 'server', 'username', 'password', 'datastore' and 'fingerprint' parameters.")
+                self.module.fail_json(msg="PBS storage requires 'server', 'username', 'password' and 'datastore' parameters.")
+            else:
+                payload['server'] = server
+                payload['username'] = username
+                payload['password'] = password
+                payload['datastore'] = datastore
+                if fingerprint:
+                    payload['fingerprint'] = fingerprint
 
         # Check Mode validation
         if self.module.check_mode: