Merge pull request #1561 from stuggi/bump_defaults

openshift-merge-bot[bot] · web-flow · commit fc9c4c29c791 · 2025-08-12T13:07:48.000Z
Bump operator deployment memory request/limit defaults
diff --git a/apis/operator/v1beta1/openstack_types.go b/apis/operator/v1beta1/openstack_types.go
@@ -60,9 +60,9 @@ var (
 	// DefaultManagerCPURequests - Default controller manager container CPU requests
 	DefaultManagerCPURequests resource.Quantity = resource.MustParse("10m")
 	// DefaultManagerMemoryLimit - Default controller manager container memory limit
-	DefaultManagerMemoryLimit resource.Quantity = resource.MustParse("256Mi")
+	DefaultManagerMemoryLimit resource.Quantity = resource.MustParse("512Mi")
 	// DefaultManagerMemoryRequests - Default controller manager container memory requests
-	DefaultManagerMemoryRequests resource.Quantity = resource.MustParse("128Mi")
+	DefaultManagerMemoryRequests resource.Quantity = resource.MustParse("256Mi")
 	// DefaultRbacProxyCPULimit - Default kube rbac proxy container CPU limit
 	DefaultRbacProxyCPULimit resource.Quantity = resource.MustParse("500m")
 	// DefaultRbacProxyCPURequests - Default kube rbac proxy container CPU requests
diff --git a/pkg/operator/override_test.go b/pkg/operator/override_test.go
@@ -210,7 +210,7 @@ func TestApplyOperatorOverrides(t *testing.T) {
 			},
 			expectedRequests: &ResourceList{
 				CPU:    "1",
-				Memory: "128Mi", // the default must not be overridden if we just change the cpu
+				Memory: "256Mi", // the default must not be overridden if we just change the cpu
 			},
 		},
 		{

Original file line number	Diff line number	Diff line change
`@@ -210,7 +210,7 @@ func TestApplyOperatorOverrides(t *testing.T) {`
`210`	`210`	`},`
`211`	`211`	`expectedRequests: &ResourceList{`
`212`	`212`	`CPU: "1",`
`213`		`- Memory: "128Mi", // the default must not be overridden if we just change the cpu`
	`213`	`+ Memory: "256Mi", // the default must not be overridden if we just change the cpu`
`214`	`214`	`},`
`215`	`215`	`},`
`216`	`216`	`{`