From b632cd9ffb81d6a8be876765dc9b89687d6f964c Mon Sep 17 00:00:00 2001
From: Erich Blume <blume.erich@gmail.com>
Date: Fri, 27 Mar 2026 22:36:32 -0700
Subject: [PATCH] Fix Immich resource limits and probe timeouts

Resources were under wrong Helm value keys (server.resources,
machine-learning.resources) and never applied to pods. Move to correct
bjw-s chart paths (*.controllers.main.containers.main.resources).

Increase liveness/readiness probe timeouts from 1s to 5s to prevent
kubelet from killing healthy-but-busy pods during ML inference load.
Remove CPU limits (keep requests only) to avoid throttling.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 argocd/manifests/immich/values.yaml | 50 ++++++++++++++++++++---------
 1 file changed, 35 insertions(+), 15 deletions(-)

diff --git a/argocd/manifests/immich/values.yaml b/argocd/manifests/immich/values.yaml
index 493d9b1..7d23cb8 100644
--- a/argocd/manifests/immich/values.yaml
+++ b/argocd/manifests/immich/values.yaml
@@ -37,19 +37,29 @@ immich:
 # Machine Learning service
 machine-learning:
   enabled: true
+  controllers:
+    main:
+      containers:
+        main:
+          resources:
+            requests:
+              memory: "512Mi"
+              cpu: "100m"
+            limits:
+              memory: "4Gi"
+          probes:
+            liveness:
+              spec:
+                timeoutSeconds: 5
+            readiness:
+              spec:
+                timeoutSeconds: 5
   persistence:
     cache:
       enabled: true
       type: persistentVolumeClaim
       accessMode: ReadWriteOnce
       size: 10Gi
-  resources:
-    requests:
-      memory: "512Mi"
-      cpu: "100m"
-    limits:
-      memory: "4Gi"
-      cpu: "2000m"
 
 # Valkey (Redis fork) - included in chart
 valkey:
@@ -60,12 +70,22 @@ valkey:
       type: emptyDir
       size: 1Gi
 
-# Server resources for minikube
+# Server configuration
 server:
-  resources:
-    requests:
-      memory: "256Mi"
-      cpu: "100m"
-    limits:
-      memory: "2Gi"
-      cpu: "2000m"
+  controllers:
+    main:
+      containers:
+        main:
+          resources:
+            requests:
+              memory: "256Mi"
+              cpu: "100m"
+            limits:
+              memory: "2Gi"
+          probes:
+            liveness:
+              spec:
+                timeoutSeconds: 5
+            readiness:
+              spec:
+                timeoutSeconds: 5