fix(proxmox): move hostname extraction logic before exception handling

- Move hostname parsing logic inside try block for cleaner error handling
- Maintain fallback to vm_name when API call fails
- Improve code readability and exception flow

Co-authored-by: sarah-witt <[email protected]>
Signed-off-by: puretension <[email protected]>
Signed-off-by: sarah-witt <[email protected]>

proxmox/datadog_checks/proxmox/check.py

@@ -145,6 +145,7 @@ def _get_vm_hostname(self, vm_id, vm_name, node):
             url = f"{self.config.proxmox_server}/nodes/{node}/qemu/{vm_id}/agent/get-host-name"
             hostname_response = self.http.get(url)
             hostname_json = hostname_response.json()
+            hostname = hostname_json.get("data", {}).get("result", {}).get("host-name", vm_name)
         except (HTTPError, InvalidURL, ConnectionError, Timeout, JSONDecodeError, AttributeError) as e:
             self.log.info(
                 "Failed to get hostname for vm %s on node %s; endpoint: %s; %s",
@@ -154,8 +155,6 @@ def _get_vm_hostname(self, vm_id, vm_name, node):
                 e,
             )
             hostname = vm_name
-            return hostname
-        hostname = hostname_json.get("data", {}).get("result", {}).get("host-name", vm_name)
         return hostname
 
     def _create_dd_event_for_task(self, task, node_name):