Skip to content

Commit

Permalink
Bump memory limits for the operator controller-manager pod
Browse files Browse the repository at this point in the history
Starting OCP 4.16, the current set limits need to be bumped
to prevent them getting OOMKilled, e.g.:

NAME                                                              CPU(cores)   MEMORY(bytes)
swift-operator-controller-manager-6764c568ff-r4mzl                2m           468Mi
telemetry-operator-controller-manager-7c4fd577b4-9nnvq            2m           482Mi

The reason for this is probably due to the move to cgroups v2.
OCP 4.16 release notes:
~~~
Beginning with OpenShift Container Platform 4.16, Control Groups
version 2 (cgroup v2), also known as cgroup2 or cgroupsv2, is enabled
by default for all new deployments, even when performance profiles
are present.Since OpenShift Container Platform 4.14, cgroups v2 has
been the default, but the performance profile feature required the
use of cgroups v1. This issue has been resolved.
~~~

Upstream memory increase discussion:
kubernetes/kubernetes#118916

While im mentiones its only node memory stats thats are wrong and
pod stats are correct, its probably related.

Resolves: https://issues.redhat.com/browse/OSPRH-8379

Signed-off-by: Martin Schuppert <[email protected]>
  • Loading branch information
stuggi committed Jul 8, 2024
1 parent 8bd8066 commit 7f41b36
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions config/manager/manager.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -65,9 +65,9 @@ spec:
resources:
limits:
cpu: 500m
memory: 256Mi
memory: 768Mi
requests:
cpu: 10m
memory: 128Mi
memory: 512Mi
serviceAccountName: controller-manager
terminationGracePeriodSeconds: 10

0 comments on commit 7f41b36

Please sign in to comment.