Fix: deny allocation when memory cannot be reclaimed

grussorusso · grussorusso · commit 1399c85926f9 · 2025-10-30T16:37:03.000+01:00
diff --git a/internal/node/pool.go b/internal/node/pool.go
@@ -295,10 +295,11 @@ cleanup: // second phase, cleanup
 			}
 			LocalResources.warmPoolUsedMem -= item.memory
 		}
+		return true, nil
 	} else {
 		log.Printf("Not enough containers to free up at least %d MB (avail to dismiss: %d)", requiredMemoryMB, cleanedMB)
+		return false, nil
 	}
-	return true, nil
 }
 
 // DeleteExpiredContainer is called by the container cleaner

Original file line number	Diff line number	Diff line change
`@@ -295,10 +295,11 @@ cleanup: // second phase, cleanup`
`295`	`295`	`}`
`296`	`296`	`LocalResources.warmPoolUsedMem -= item.memory`
`297`	`297`	`}`
	`298`	`+ return true, nil`
`298`	`299`	`} else {`
`299`	`300`	`log.Printf("Not enough containers to free up at least %d MB (avail to dismiss: %d)", requiredMemoryMB, cleanedMB)`
	`301`	`+ return false, nil`
`300`	`302`	`}`
`301`		`- return true, nil`
`302`	`303`	`}`
`303`	`304`
`304`	`305`	`// DeleteExpiredContainer is called by the container cleaner`