Merge pull request #369 from gyptazy/fix/368-crash-including-storage-in-pools

fix(pools): Fixes a crash during PVE resource pool enumeration by ski…
This commit is contained in:
gyptazy
2025-11-18 13:15:53 +01:00
committed by GitHub
2 changed files with 8 additions and 0 deletions

View File

@@ -0,0 +1,2 @@
fixed:
- Fixed a crash during PVE resource pool enumeration by skipping members not having a 'name' property (@stefanoettl). [#368]

View File

@@ -68,6 +68,12 @@ class Pools:
# Fetch pool details and collect member names
pool_details = proxmox_api.pools(pool['poolid']).get()
for member in pool_details.get("members", []):
# We might also have objects without the key "name", e.g. storage pools
if "name" not in member:
logger.debug(f"Skipping member without name in pool: {pool['poolid']}")
continue
logger.debug(f"Got member: {member['name']} for pool: {pool['poolid']}")
pools['pools'][pool['poolid']]['members'].append(member["name"])