tronnet/ProxmoxAAS-Fabric
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix missing pool membership clearing logic in ResolvePoolMembership

Browse Source
This commit is contained in:
Arthur Lu
2026-07-01 18:08:21 +00:00
parent cbda49ff69
commit b506aae260
1 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/model.go
+8 -1
View File
@@ -90,6 +90,13 @@ func (cluster *Cluster) ResolvePoolMembership() error {
cluster.lock.Lock() cluster.lock.Lock()
defer cluster.lock.Unlock() defer cluster.lock.Unlock()
//clear existing pool memberships
for _, node := range cluster.Nodes {
for _, instance := range node.Instances {
instance.Pool = ""
}
}
//resolve pool membership //resolve pool membership
pools, err := cluster.pve.client.Pools(context.Background()) pools, err := cluster.pve.client.Pools(context.Background())
if err != nil { if err != nil {
@@ -110,7 +117,7 @@ func (cluster *Cluster) ResolvePoolMembership() error {
if !ok { if !ok {
return fmt.Errorf("Instance %d claimed to be in node %s but was not", member.VMID, node.Name) return fmt.Errorf("Instance %d claimed to be in node %s but was not", member.VMID, node.Name)
} }
if instance.Pool != "" { if instance.Pool != "" { // enforces that an instance cannot be in two different pools
return fmt.Errorf("Instance %d is in pools %s and %s which is not supported", member.VMID, instance.Pool, pool.PoolID) return fmt.Errorf("Instance %d is in pools %s and %s which is not supported", member.VMID, instance.Pool, pool.PoolID)
} }
instance.Pool = pool.PoolID instance.Pool = pool.PoolID