From cc4dee77dfffa2582e5972ca7997d9dfeacb1513 Mon Sep 17 00:00:00 2001
From: Nikolai Emil Damm <ned@devantler.com>
Date: Fri, 16 Aug 2024 00:05:14 +0200
Subject: [PATCH] Update values.yaml with better defaults

Currently the Helm Chart does not work, as the expected container port is 8080 and not 80. Furthermore the chart uses the "hostPath" as a storage class, which I do not think is preffered, as setting this value to and empty string would instead use the default storage class, which makes things easier for folks.

Signed-off-by: Nikolai Emil Damm <ned@devantler.com>
---
 charts/local-ai/values.yaml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/charts/local-ai/values.yaml b/charts/local-ai/values.yaml
index 2f488aa..3be5d2f 100644
--- a/charts/local-ai/values.yaml
+++ b/charts/local-ai/values.yaml
@@ -112,14 +112,14 @@ persistence:
   models: 
     enabled: true
     annotations: {}
-    storageClass: hostPath
+    storageClass: "" # Use default storage class
     accessModes: ReadWriteMany
     size: 10Gi
     globalMount: /models
   output:
     enabled: true
     annotations: {}
-    storageClass: hostPath
+    storageClass: "" # Use default storage class
     accessModes: ReadWriteMany
     size: 5Gi
     globalMount: /tmp/generated
@@ -128,7 +128,7 @@ service:
   type: ClusterIP
   # If deferring to an internal only load balancer
   # externalTrafficPolicy: Local
-  port: 80
+  port: 8080
   annotations: {}
   # If using an AWS load balancer, you'll need to override the default 60s load balancer idle timeout
   # service.beta.kubernetes.io/aws-load-balancer-connection-idle-timeout: "1200"