Merge pull request #572 from Yannig/set-exporter-ressources

pascaliske · Jun 13, 2024 · 97d608a · 97d608a
2 parents 06dae5d + fc743b2
commit 97d608a
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 12 deletions.
diff --git a/charts/redis/templates/controller.yaml b/charts/redis/templates/controller.yaml
@@ -94,19 +94,9 @@ spec:
               containerPort: 9121
               protocol: TCP
           securityContext:
-            runAsUser: 59000
-            runAsGroup: 59000
-            allowPrivilegeEscalation: false
-            capabilities:
-              drop:
-              - ALL
+            {{- toYaml .Values.redisExporter.securityContext | nindent 12 }}
           resources:
-            requests:
-              cpu: 10m
-              memory: 50Mi
-            limits:
-              cpu: 100m
-              memory: 100Mi
+            {{- toYaml .Values.redisExporter.resources | nindent 12 }}
         {{- end }}
       {{- if eq (include "base.persistence.enabled" . ) "true" }}
       {{- if eq (include "base.persistence.type" . ) "volumes" }}

diff --git a/charts/redis/values.yaml b/charts/redis/values.yaml
@@ -62,6 +62,22 @@ redisExporter:
     tag: 'latest'
     # -- The pull policy for the exporter.
     pullPolicy: IfNotPresent
+  # -- Pod-level security attributes. More info [here](https://kubernetes.io/docs/reference/kubernetes-api/workload-resources/pod-v1/#security-context).
+  securityContext:
+    runAsUser: 59000
+    runAsGroup: 59000
+    allowPrivilegeEscalation: false
+    capabilities:
+      drop:
+        - ALL
+  # -- Compute resources used by the container. More info [here](https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/).
+  resources:
+    requests:
+      cpu: 10m
+      memory: 50Mi
+    limits:
+      cpu: 100m
+      memory: 100Mi
 
 env:
   # -- Timezone for the container.