Fix boolean settings for PVE storage module

Author: lae

library/proxmox_role.py

@@ -186,4 +186,4 @@ def main():
     module.exit_json(**result)
 
 if __name__ == '__main__':
-    main()
+    main()

library/proxmox_storage.py

@@ -202,9 +202,9 @@ def lookup(self):
         for item in self.existing_storages:
             if item['storage'] == self.name:
                 # pvesh doesn't return the disable param value if it's false,
-                # so we set it to False.
+                # so we set it to 0, which is what PVE would normally use.
                 if item.get('disable') is None:
-                    item['disable'] = False
+                    item['disable'] = 0
                 return item
         return None
 
@@ -222,9 +222,7 @@ def prepare_storage_args(self):
         if self.nodes is not None:
             args['nodes'] = ','.join(self.nodes)
         if self.disable is not None:
-            args['disable'] = self.disable
-        else:
-            args['disable'] = False
+            args['disable'] = 1 if self.disable else 0
         if self.path is not None:
             args['path'] = self.path
         if self.pool is not None:
@@ -234,7 +232,7 @@ def prepare_storage_args(self):
         if self.username is not None:
             args['username'] = self.username
         if self.krbd is not None:
-            args['krbd'] = self.krbd
+            args['krbd'] = 1 if self.krbd else 0
         if self.maxfiles is not None:
             args['maxfiles'] = self.maxfiles
         if self.server is not None:
@@ -248,7 +246,7 @@ def prepare_storage_args(self):
         if self.thinpool is not None:
             args['thinpool'] = self.thinpool
         if self.sparse is not None:
-            args['sparse'] = self.sparse
+            args['sparse'] = 1 if self.sparse else 0
 
         if self.maxfiles is not None and 'backup' not in self.content:
             self.module.fail_json(msg="maxfiles is not allowed when there is no 'backup' in content")

tests/vagrant/group_vars/all

@@ -58,6 +58,11 @@ pve_storages:
     type: dir
     content: [ "iso", "vztmpl", "backup" ]
     path: /testpool/zfs2
+  - name: disabled-dir
+    type: dir
+    disable: yes
+    content: [ "images" ]
+    path: /tmp/fakedir
 pve_zfs_create_volumes:
   - testpool/zfs2
 pve_ceph_osds: