balloons: fix the order of assigning containers into balloons

Assign containers into balloons in the order in which their pods and themselves have been created. This makes policy restart and reconfiguration behavior more predictable. Signed-off-by: Antti Kervinen <[email protected]>
containers · Mar 1, 2024 · ad41e5e · ad41e5e
1 parent f285001
commit ad41e5e
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/cmd/plugins/balloons/policy/balloons-policy.go b/cmd/plugins/balloons/policy/balloons-policy.go
@@ -196,6 +196,9 @@ func (p *balloons) Sync(add []cache.Container, del []cache.Container) error {
 			log.Warnf("releasing resources for Sync produced an error: %v", err)
 		}
 	}
+
+	cache.SortContainers(add, cache.ComparePodCtime, cache.CompareContainerCtime)
+
 	for _, c := range add {
 		if err := p.AllocateResources(c); err != nil {
 			log.Warnf("allocating resources for Sync produced an error: %v", err)