From 4dcafe2c0723ddc73931cdb12ea1c4ae8fa333ae Mon Sep 17 00:00:00 2001
From: Asaf Chen <109060156+asafchen-dig@users.noreply.github.com>
Date: Wed, 10 Sep 2025 23:39:06 -0400
Subject: [PATCH 1/2] feat: add cpu and memory limits to petclinic deployment

---
 petshop-chart/values.yaml | 23 ++++++++++++++---------
 1 file changed, 14 insertions(+), 9 deletions(-)

diff --git a/petshop-chart/values.yaml b/petshop-chart/values.yaml
index 26c7efa2812..e6e2b4d4690 100644
--- a/petshop-chart/values.yaml
+++ b/petshop-chart/values.yaml
@@ -56,17 +56,22 @@ ingress:
   #    hosts:
   #      - chart-example.local
 
-resources: {}
-  # We usually recommend not to specify default resources and to leave this as a conscious
-  # choice for the user. This also increases chances charts run on environments with little
+resources:
+  requests:
+    cpu: 200m
+    memory: 512Mi
+  limits:
+    cpu: 1000m
+    memory: 1Gi  # choice for the user. This also increases chances charts run on environments with little
   # resources, such as Minikube. If you do want to specify resources, uncomment the following
   # lines, adjust them as necessary, and remove the curly braces after 'resources:'.
-  # limits:
-  #   cpu: 100m
-  #   memory: 128Mi
-  # requests:
-  #   cpu: 100m
-  #   memory: 128Mi
+  resources:
+    requests:
+      cpu: 200m
+      memory: 512Mi
+    limits:
+      cpu: 1000m
+      memory: 1Gi
 
 autoscaling:
   enabled: false

From 124f6f5d61d8de70810787f547327d66e1d3dce4 Mon Sep 17 00:00:00 2001
From: Asaf Chen <109060156+asafchen-dig@users.noreply.github.com>
Date: Wed, 10 Sep 2025 23:39:21 -0400
Subject: [PATCH 2/2] chore: update probe configs with delay period timeout and
 failure thresholds

---
 petshop-chart/templates/deployment.yaml | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/petshop-chart/templates/deployment.yaml b/petshop-chart/templates/deployment.yaml
index e8bba2c5ca3..38e8de3084a 100644
--- a/petshop-chart/templates/deployment.yaml
+++ b/petshop-chart/templates/deployment.yaml
@@ -52,13 +52,20 @@ spec:
             httpGet:
               path: /
               port: http
+            initialDelaySeconds: 30
+            periodSeconds: 10
+            timeoutSeconds: 5
+            failureThreshold: 3
           readinessProbe:
             httpGet:
               path: /
               port: http
+            initialDelaySeconds: 30
+            periodSeconds: 10
+            timeoutSeconds: 5
+            failureThreshold: 3
           resources:
-            {{- toYaml .Values.resources | nindent 12 }}
-      {{- with .Values.nodeSelector }}
+            {{- toYaml .Values.resources | nindent 12 }}      {{- with .Values.nodeSelector }}
       nodeSelector:
         {{- toYaml . | nindent 8 }}
       {{- end }}