diff --git a/deploy/charts/litellm-helm/templates/deployment.yaml b/deploy/charts/litellm-helm/templates/deployment.yaml index 97123e5df6..c25c62cc40 100644 --- a/deploy/charts/litellm-helm/templates/deployment.yaml +++ b/deploy/charts/litellm-helm/templates/deployment.yaml @@ -116,6 +116,13 @@ spec: name: {{ include "redis.secretName" .Subcharts.redis }} key: {{include "redis.secretPasswordKey" .Subcharts.redis }} {{- end }} + {{- /* + Inject LITELLM_LOG only when envVars does not already define it. + */}} + {{- if and .Values.logLevel (not (hasKey (default dict .Values.envVars) "LITELLM_LOG")) }} + - name: LITELLM_LOG + value: {{ .Values.logLevel | quote }} + {{- end }} {{- if .Values.envVars }} {{- range $key, $val := .Values.envVars }} - name: {{ $key }} diff --git a/deploy/charts/litellm-helm/tests/deployment_tests.yaml b/deploy/charts/litellm-helm/tests/deployment_tests.yaml index b1cbafaf40..df6d134564 100644 --- a/deploy/charts/litellm-helm/tests/deployment_tests.yaml +++ b/deploy/charts/litellm-helm/tests/deployment_tests.yaml @@ -257,16 +257,16 @@ tests: value: 0 - equal: path: spec.template.spec.containers[0].livenessProbe.periodSeconds - value: 10 + value: 15 - equal: path: spec.template.spec.containers[0].livenessProbe.timeoutSeconds - value: 1 + value: 5 - equal: path: spec.template.spec.containers[0].livenessProbe.successThreshold value: 1 - equal: path: spec.template.spec.containers[0].livenessProbe.failureThreshold - value: 3 + value: 5 - equal: path: spec.template.spec.containers[0].readinessProbe.httpGet.path value: /health/readiness @@ -278,7 +278,7 @@ tests: value: 10 - equal: path: spec.template.spec.containers[0].readinessProbe.timeoutSeconds - value: 1 + value: 5 - equal: path: spec.template.spec.containers[0].readinessProbe.successThreshold value: 1 @@ -296,7 +296,7 @@ tests: value: 10 - equal: path: spec.template.spec.containers[0].startupProbe.timeoutSeconds - value: 1 + value: 5 - equal: path: spec.template.spec.containers[0].startupProbe.successThreshold value: 1 diff --git a/deploy/charts/litellm-helm/values.yaml b/deploy/charts/litellm-helm/values.yaml index 690ca69e73..3dd1112bea 100644 --- a/deploy/charts/litellm-helm/values.yaml +++ b/deploy/charts/litellm-helm/values.yaml @@ -94,20 +94,20 @@ service: separateHealthApp: false separateHealthPort: 8081 -# Probe tuning for proxy container +# Probes for LiteLLM gateway container livenessProbe: path: /health/liveliness initialDelaySeconds: 0 - periodSeconds: 10 - timeoutSeconds: 1 + periodSeconds: 15 + timeoutSeconds: 5 successThreshold: 1 - failureThreshold: 3 + failureThreshold: 5 readinessProbe: path: /health/readiness initialDelaySeconds: 0 periodSeconds: 10 - timeoutSeconds: 1 + timeoutSeconds: 5 successThreshold: 1 failureThreshold: 3 @@ -115,7 +115,7 @@ startupProbe: path: /health/readiness initialDelaySeconds: 0 periodSeconds: 10 - timeoutSeconds: 1 + timeoutSeconds: 5 successThreshold: 1 failureThreshold: 30 @@ -329,6 +329,17 @@ migrationJob: helm: enabled: false +# Log level for the litellm proxy (sets LITELLM_LOG in the deployment env). +# Rendered as a direct `env:` entry, which in Kubernetes takes precedence over +# any `envFrom:` source. If you currently source LITELLM_LOG from an +# environmentSecret or environmentConfigMap, set `logLevel: ""` here to +# disable injection — otherwise this value silently overrides your secret / +# configmap entry. +# +# Setting LITELLM_LOG inside `envVars:` below also wins: the template skips +# this injection entirely when envVars already defines LITELLM_LOG. +logLevel: INFO + # Additional environment variables to be added to the deployment as a map of key-value pairs envVars: {}