From 18e3d17c29e828255200d76aa0934ea983f613f6 Mon Sep 17 00:00:00 2001
From: pipo02mix <fernando@giantswarm.io>
Date: Tue, 3 Dec 2024 14:42:14 +0100
Subject: [PATCH] Improve chart default values

---
 charts/descheduler/templates/cronjob.yaml    |  2 ++
 charts/descheduler/templates/deployment.yaml |  3 +--
 charts/descheduler/values.yaml               | 10 +++++++---
 3 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/charts/descheduler/templates/cronjob.yaml b/charts/descheduler/templates/cronjob.yaml
index 097047345..d2c9123c9 100644
--- a/charts/descheduler/templates/cronjob.yaml
+++ b/charts/descheduler/templates/cronjob.yaml
@@ -89,6 +89,8 @@ spec:
                 {{- end }}
               livenessProbe:
                 {{- toYaml .Values.livenessProbe | nindent 16 }}
+              ports:
+                {{- toYaml .Values.ports | nindent 16 }}
               resources:
                 {{- toYaml .Values.resources | nindent 16 }}
               {{- if .Values.securityContext }}
diff --git a/charts/descheduler/templates/deployment.yaml b/charts/descheduler/templates/deployment.yaml
index 01609fe7f..97fc6331d 100644
--- a/charts/descheduler/templates/deployment.yaml
+++ b/charts/descheduler/templates/deployment.yaml
@@ -61,8 +61,7 @@ spec:
             {{- end }}
             {{- include "descheduler.leaderElection" . | nindent 12 }}
           ports:
-            - containerPort: 10258
-              protocol: TCP
+            {{- toYaml .Values.ports | nindent 12 }}
           livenessProbe:
             {{- toYaml .Values.livenessProbe | nindent 12 }}
           resources:
diff --git a/charts/descheduler/values.yaml b/charts/descheduler/values.yaml
index bef366866..b064127bb 100644
--- a/charts/descheduler/values.yaml
+++ b/charts/descheduler/values.yaml
@@ -18,9 +18,13 @@ resources:
   requests:
     cpu: 500m
     memory: 256Mi
-  # limits:
-  #   cpu: 100m
-  #   memory: 128Mi
+  limits:
+    cpu: 500m
+    memory: 256Mi
+
+ports:
+  - containerPort: 10258
+    protocol: TCP
 
 securityContext:
   allowPrivilegeEscalation: false