Skip to content

Commit

Permalink
feat(helm): make Dask autoscaler configurable (#835)
Browse files Browse the repository at this point in the history
Closes #834
  • Loading branch information
Alputer committed Nov 11, 2024
1 parent c0d2e92 commit aeebee1
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 0 deletions.
1 change: 1 addition & 0 deletions helm/reana/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@ This Helm automatically prefixes all names using the release name to avoid colli
| `components.reana_workflow_engine_snakemake.image` | [REANA-Workflow-Engine-Snakemake image](https://hub.docker.com/r/reanahub/reana-workflow-engine-snakemake) to use | `docker.io/reanahub/reana-workflow-engine-snakemake:<chart-release-version>` |
| `compute_backends` | List of supported compute backends (kubernetes, htcondorcern, slurmcern) | "kubernetes" |
| `dask.enabled` | Enable support for running Dask workflows | false |
| `dask.autoscaler_enabled` | Enable Dask autoscaler | true |
| `dask.cluster_max_memory_limit` | The maximum memory limit for Dask clusters created by users | "16Gi" |
| `dask.cluster_default_number_of_workers` | The number of Dask workers created by default | 2 |
| `dask.cluster_default_single_worker_memory` | The amount of memory used by default by a single Dask worker | "2Gi" |
Expand Down
2 changes: 2 additions & 0 deletions helm/reana/templates/reana-server.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,8 @@ spec:
- name: DASK_ENABLED
value: {{ .Values.dask.enabled | quote }}
{{- if .Values.dask.enabled }}
- name: DASK_AUTOSCALER_ENABLED
value: {{ .Values.dask.autoscaler_enabled | quote }}
- name: REANA_DASK_CLUSTER_MAX_MEMORY_LIMIT
value: {{ .Values.dask.cluster_max_memory_limit | default "16Gi" }}
- name: REANA_DASK_CLUSTER_DEFAULT_NUMBER_OF_WORKERS
Expand Down
6 changes: 6 additions & 0 deletions helm/reana/templates/reana-workflow-controller.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -182,6 +182,8 @@ spec:
- name: DASK_ENABLED
value: {{ .Values.dask.enabled | quote }}
{{- if .Values.dask.enabled }}
- name: DASK_AUTOSCALER_ENABLED
value: {{ .Values.dask.autoscaler_enabled | quote }}
- name: REANA_DASK_CLUSTER_MAX_MEMORY_LIMIT
value: {{ .Values.dask.cluster_max_memory_limit | default "16Gi" }}
- name: REANA_DASK_CLUSTER_DEFAULT_NUMBER_OF_WORKERS
Expand Down Expand Up @@ -289,6 +291,10 @@ spec:
- name: GIT_SSL_NO_VERIFY
value: "true"
{{- end }}
{{- if .Values.dask.enabled }}
- name: DASK_AUTOSCALER_ENABLED
value: {{ .Values.dask.autoscaler_enabled | quote }}
{{- end }}
- name: REANA_DB_USERNAME
valueFrom:
secretKeyRef:
Expand Down
1 change: 1 addition & 0 deletions helm/reana/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -180,6 +180,7 @@ traefik:

dask:
enabled: false
autoscaler_enabled: true
cluster_max_memory_limit: "16Gi"
cluster_default_number_of_workers: 2
cluster_default_single_worker_memory: "2Gi"
Expand Down

0 comments on commit aeebee1

Please sign in to comment.