Skip to content

Commit c7d0b9b

Browse files
committed
Bump operator deployment memory request/limit defaults
During start of the operators it is seen in some environments that the current defaults get reached during start/initialization. Checking the usage, most of them peaks at the startup, using between 350-380 MiB, other around 300MiB, for a short time and then it get stabilized at 200-210 MiB. Jira: OSPRH-17785 Signed-off-by: Martin Schuppert <[email protected]>
1 parent 6d676c6 commit c7d0b9b

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

apis/operator/v1beta1/openstack_types.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,9 +60,9 @@ var (
6060
// DefaultManagerCPURequests - Default controller manager container CPU requests
6161
DefaultManagerCPURequests resource.Quantity = resource.MustParse("10m")
6262
// DefaultManagerMemoryLimit - Default controller manager container memory limit
63-
DefaultManagerMemoryLimit resource.Quantity = resource.MustParse("256Mi")
63+
DefaultManagerMemoryLimit resource.Quantity = resource.MustParse("512Mi")
6464
// DefaultManagerMemoryRequests - Default controller manager container memory requests
65-
DefaultManagerMemoryRequests resource.Quantity = resource.MustParse("128Mi")
65+
DefaultManagerMemoryRequests resource.Quantity = resource.MustParse("256Mi")
6666
// DefaultRbacProxyCPULimit - Default kube rbac proxy container CPU limit
6767
DefaultRbacProxyCPULimit resource.Quantity = resource.MustParse("500m")
6868
// DefaultRbacProxyCPURequests - Default kube rbac proxy container CPU requests

pkg/operator/override_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,7 @@ func TestApplyOperatorOverrides(t *testing.T) {
210210
},
211211
expectedRequests: &ResourceList{
212212
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
214214
},
215215
},
216216
{

0 commit comments

Comments
 (0)