Skip to content

Commit

Permalink
Update values.yaml with better defaults
Browse files Browse the repository at this point in the history
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 <[email protected]>
  • Loading branch information
devantler committed Aug 15, 2024
1 parent f6d8172 commit cc4dee7
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions charts/local-ai/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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"
Expand Down

0 comments on commit cc4dee7

Please sign in to comment.