Skip to content

Commit

Permalink
[helm] Independent setting of memory resources for migration pods (#3281
Browse files Browse the repository at this point in the history
)

* add separate resources for migration pod

* default to global resources if not set
  • Loading branch information
wjayesh authored Jan 21, 2025
1 parent fec0956 commit dd91059
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 3 deletions.
8 changes: 5 additions & 3 deletions src/zenml/zen_server/deploy/helm/templates/server-db-job.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -104,10 +104,12 @@ spec:
envFrom:
- secretRef:
name: {{ include "zenml.fullname" . }}-db-migration
{{- with .Values.resources }}
resources:
{{- toYaml . | nindent 12 }}
{{- end }}
{{- if .Values.resourcesMigrationPod }}
{{- toYaml .Values.resourcesMigrationPod | nindent 12 }}
{{- else }}
{{- toYaml .Values.resources | nindent 12 }}
{{- end }}
{{- with .Values.tolerations }}
tolerations:
{{- toYaml . | nindent 8 }}
Expand Down
4 changes: 4 additions & 0 deletions src/zenml/zen_server/deploy/helm/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -990,6 +990,10 @@ resources: {}
# cpu: 100m
# memory: 128Mi

resourcesMigrationPod: {}
# Resource requests and limits for the database migration pod.
# If not set, falls back to the global resources configuration defined in .Values.resources

autoscaling:
enabled: false
minReplicas: 1
Expand Down

0 comments on commit dd91059

Please sign in to comment.