Merge pull request kubernetes-sigs#11004 from vincepri/use-memory-allocatable

k8s-ci-robot · web-flow · commit 085480ab201b · 2024-08-02T10:15:51.000-07:00
🌱 test/framework: scale up should use allocatable memory
diff --git a/test/framework/autoscaler_helpers.go b/test/framework/autoscaler_helpers.go
@@ -158,7 +158,7 @@ func AddScaleUpDeploymentAndWait(ctx context.Context, input AddScaleUpDeployment
 		if _, ok := n.Labels[nodeRoleOldControlPlane]; ok {
 			continue
 		}
-		memory = n.Status.Capacity.Memory() // Assume that all nodes have the same memory.
+		memory = n.Status.Allocatable.Memory() // Assume that all nodes have the same memory.
 		workers++
 	}
 	Expect(memory).ToNot(BeNil(), "failed to get memory for the worker node")

Original file line number	Diff line number	Diff line change
`@@ -158,7 +158,7 @@ func AddScaleUpDeploymentAndWait(ctx context.Context, input AddScaleUpDeployment`
`158`	`158`	`if _, ok := n.Labels[nodeRoleOldControlPlane]; ok {`
`159`	`159`	`continue`
`160`	`160`	`}`
`161`		`- memory = n.Status.Capacity.Memory() // Assume that all nodes have the same memory.`
	`161`	`+ memory = n.Status.Allocatable.Memory() // Assume that all nodes have the same memory.`
`162`	`162`	`workers++`
`163`	`163`	`}`
`164`	`164`	`Expect(memory).ToNot(BeNil(), "failed to get memory for the worker node")`