fix robusta-dev#1601 : enable cpu limit for "robusta-forwarder" servi…

…ce as current config cause cpu hogging Signed-off-by: Pratik Raj <[email protected]>
Rajpratik71 · Oct 22, 2024 · 3655cf8 · 3655cf8
1 parent 31ccccf
commit 3655cf8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/helm/robusta/templates/forwarder.yaml b/helm/robusta/templates/forwarder.yaml
@@ -68,8 +68,8 @@ spec:
             cpu: {{ .Values.kubewatch.resources.requests.cpu }}
             memory: {{ if .Values.isSmallCluster }}"64Mi"{{ else }}{{ .Values.kubewatch.resources.requests.memory | quote }}{{ end }}
           limits:
+            cpu: {{ .Values.kubewatch.resources.limits.cpu }}
             memory: {{ if .Values.isSmallCluster }}"64Mi"{{ else if .Values.kubewatch.resources.limits.memory }}{{ .Values.kubewatch.resources.limits.memory | quote }}{{ else }}{{ .Values.kubewatch.resources.requests.memory | quote }}{{ end }}
-            {{ if .Values.kubewatch.resources.limits.cpu }}cpu: {{ .Values.kubewatch.resources.limits.cpu | quote }}{{ end }}
       volumes:
         - name:  {{ include "robusta.fullname" . }}-kubewatch-config
           configMap:

diff --git a/helm/robusta/values.yaml b/helm/robusta/values.yaml
@@ -595,7 +595,7 @@ kubewatch:
       cpu: 10m
       memory: 512Mi
     limits:
-      cpu: ~
+      cpu: 50m
   additional_env_vars: []
   priorityClassName: ""
   tolerations: []