From 5b61c6f4a1308de88d19591e12c059aff8be0bbe Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Wojciech=20Smo=C5=82kowski?= Date: Tue, 11 Mar 2025 06:33:01 +0100 Subject: [PATCH 1/3] add resources secrtion to migrationJob values.yaml migration job need to have option to setup resources --- deploy/charts/litellm-helm/values.yaml | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/deploy/charts/litellm-helm/values.yaml b/deploy/charts/litellm-helm/values.yaml index 9f21fc40ad..3f528f8e25 100644 --- a/deploy/charts/litellm-helm/values.yaml +++ b/deploy/charts/litellm-helm/values.yaml @@ -188,6 +188,17 @@ migrationJob: disableSchemaUpdate: false # Skip schema migrations for specific environments. When True, the job will exit with code 0. annotations: {} ttlSecondsAfterFinished: 120 + 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, 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 # Additional environment variables to be added to the deployment envVars: { From 0595bbce95931c1adb3b0bb27a59b0a31af430e5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Wojciech=20Smo=C5=82kowski?= Date: Tue, 11 Mar 2025 06:37:29 +0100 Subject: [PATCH 2/3] Update migrations-job.yaml to add resources migration job need to have resources. --- deploy/charts/litellm-helm/templates/migrations-job.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/deploy/charts/litellm-helm/templates/migrations-job.yaml b/deploy/charts/litellm-helm/templates/migrations-job.yaml index e994c45548..07ae94997b 100644 --- a/deploy/charts/litellm-helm/templates/migrations-job.yaml +++ b/deploy/charts/litellm-helm/templates/migrations-job.yaml @@ -51,6 +51,8 @@ spec: {{- with .Values.volumeMounts }} volumeMounts: {{- toYaml . | nindent 12 }} + resources: + {{- toYaml .Values.migrationJob.resources | nindent 12 }} {{- end }} {{- with .Values.volumes }} volumes: From 9b8ae827e756dc576a034299ed445a023e62472e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Wojciech=20Smo=C5=82kowski?= Date: Tue, 11 Mar 2025 07:11:21 +0100 Subject: [PATCH 3/3] fix resource tag --- deploy/charts/litellm-helm/templates/migrations-job.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/deploy/charts/litellm-helm/templates/migrations-job.yaml b/deploy/charts/litellm-helm/templates/migrations-job.yaml index 07ae94997b..1b463e2b1e 100644 --- a/deploy/charts/litellm-helm/templates/migrations-job.yaml +++ b/deploy/charts/litellm-helm/templates/migrations-job.yaml @@ -51,9 +51,9 @@ spec: {{- with .Values.volumeMounts }} volumeMounts: {{- toYaml . | nindent 12 }} + {{- end }} resources: {{- toYaml .Values.migrationJob.resources | nindent 12 }} - {{- end }} {{- with .Values.volumes }} volumes: {{- toYaml . | nindent 8 }}