proxmox_vm_info: Do not throw exception when iterating through machines and optional api results are missing (#191)

* proxmox_vm_info: Do not throw exception when iterating through machines and optional api results are missing

* changelog fragment

Author: koalajoe23

changelogs/fragments/191-vm_info-optional-var-access.yml

@@ -0,0 +1,2 @@
+bugfixes:
+  - proxmox_vm_info - do not throw exception when iterating through machines and optional api results are missing (https://github.com/ansible-collections/community.proxmox/pull/191)

plugins/modules/proxmox_vm_info.py

@@ -177,13 +177,13 @@ def get_vms_from_nodes(self, cluster_machines, type, vmid=None, name=None, node=
         filtered_vms = {
             vm: info for vm, info in cluster_machines.items() if not (
                 type != info["type"]
-                or (node and info["node"] != node)
-                or (vmid and int(info["vmid"]) != vmid)
-                or (name is not None and info["name"] != name)
+                or (node and info.get("node") != node)
+                or (vmid and int(info.get("vmid", -1)) != vmid)
+                or (name is not None and info.get("name") != name)
             )
         }
         # Get list of unique node names and loop through it to get info about machines.
-        nodes = frozenset([info["node"] for vm, info in filtered_vms.items()])
+        nodes = frozenset([info["node"] for vm, info in filtered_vms.items() if "node" in info])
         for this_node in nodes:
             # "type" is mandatory and can have only values of "qemu" or "lxc". Seems that use of reflection is safe.
             call_vm_getter = getattr(self.proxmox_api.nodes(this_node), type)