mirror of
https://github.com/rcourtman/Pulse.git
synced 2026-09-11 14:00:29 +00:00
rcourtman
3f4ea6f640
Two hand-added standalone Proxmox connections whose machines are both natively named pve survived the state-layer identity fixes but still collapsed in the presentation host coalesce: a provider node row carries no machine identity of its own, so once the site agents connected, one site's node row absorbed an agent row on the shared short hostname and the other site's node row folded into it. Input ordering varies per snapshot, so the surviving row alternated between sites on every refresh, which is exactly the reported NODES 1 view with the VM count flipping 11/18 and the Agent badge hopping rows on v6.4.1. Add a provider-scope veto mirroring the state rule: node facets from different connection instances only merge with same-machine proof (same node identity, same non-empty cluster, or same endpoint host). In a bucket holding two distinct provider scopes, an agent row now attaches only to the node whose state-layer agent link names it, instead of the first compatible row. The merged row also keeps the Proxmox node row's display-name-aware name instead of the agent's bare reported hostname, which is the original 'Node Name field not observed' complaint. Validated: internal/unifiedresources, internal/monitoring, and internal/api green on linux/amd64 with the change. Refs #1753