Skip to content

Commit

Permalink
fix(dvcx-worker): Disable dvcx-workers when udf is disabled
Browse files Browse the repository at this point in the history
  • Loading branch information
mjasion committed Aug 23, 2023
1 parent c8dd482 commit e772086
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
4 changes: 4 additions & 0 deletions charts/studio/templates/deployment-studio-dvcx-worker.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,11 @@ metadata:
{{- include "studio-dvcx-worker.labels" . | nindent 4 }}
spec:
{{- if not .Values.studioDvcxWorker.autoscaling.enabled }}
{{- if ((.Values.dvcx).udfEnabled) }}
replicas: {{ .Values.studioDvcxWorker.replicaCount }}
{{- else }}
replicas: 0
{{- end }}
{{- end }}
selector:
matchLabels:
Expand Down
5 changes: 5 additions & 0 deletions charts/studio/templates/hpa-studio-dvcx-worker.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,13 @@ spec:
apiVersion: apps/v1
kind: Deployment
name: studio-dvcx-worker
{{- if ((.Values.dvcx).udfEnabled) }}
minReplicas: {{ .Values.studioDvcxWorker.autoscaling.minReplicas }}
maxReplicas: {{ .Values.studioDvcxWorker.autoscaling.maxReplicas }}
{{- else }}
minReplicas: 0
maxReplicas: 0
{{- end}}
metrics:
{{- if .Values.studioDvcxWorker.autoscaling.targetCPUUtilizationPercentage }}
- type: Resource
Expand Down

0 comments on commit e772086

Please sign in to comment.