From c3c3a8d34321e18bfa4164bffff38d1d1fa2c200 Mon Sep 17 00:00:00 2001
From: Alputer <alp.tuna.453@gmail.com>
Date: Mon, 9 Sep 2024 15:10:43 +0200
Subject: [PATCH] feat(dask): introduce dask variable in the helm chart (#821)

---
 helm/reana/Chart.yaml           | 7 +++++++
 helm/reana/README.md            | 1 +
 helm/reana/templates/roles.yaml | 4 +++-
 helm/reana/values.yaml          | 3 +++
 4 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/helm/reana/Chart.yaml b/helm/reana/Chart.yaml
index ed58383f..b7336ff0 100644
--- a/helm/reana/Chart.yaml
+++ b/helm/reana/Chart.yaml
@@ -35,3 +35,10 @@ dependencies:
     condition: traefik.enabled
     tags:
       - ingress
+  - name: dask-kubernetes-operator
+    version: 2024.8.0  
+    repository: https://helm.dask.org
+    condition: dask.enabled
+    tags:
+      - dask
+
diff --git a/helm/reana/README.md b/helm/reana/README.md
index fdfee412..77275201 100644
--- a/helm/reana/README.md
+++ b/helm/reana/README.md
@@ -60,6 +60,7 @@ This Helm automatically prefixes all names using the release name to avoid colli
 | `components.reana_workflow_engine_snakemake.environment` | [REANA-Workflow-Engine-Snakemake](https://github.com/reanahub/reana-workflow-engine-snakemake) environment variables | `{}`            |
 | `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` | Install dask-kubernetes-operator custom resources in the cluster for dask workflows | true
 | `db_env_config.REANA_DB_HOST`                            | Environment variable to connect to external databases                                | `<chart-release-name>-db`                       |
 | `db_env_config.REANA_DB_NAME`                            | Environment variable to connect to external databases                                | reana                                           |
 | `db_env_config.REANA_DB_PORT`                            | Environment variable to connect to external databases                                | "5432"                                          |
diff --git a/helm/reana/templates/roles.yaml b/helm/reana/templates/roles.yaml
index e235d9e6..e7c5c1bb 100644
--- a/helm/reana/templates/roles.yaml
+++ b/helm/reana/templates/roles.yaml
@@ -23,11 +23,13 @@ rules:
   resources: ["pods", "nodes"]
   verbs: ["get", "list", "watch"]
 # Custom dask kubernetes resources
+{{- if .Values.dask.enabled }}
 - apiGroups: ["kubernetes.dask.org"]
   resources: ["daskclusters", "daskautoscalers"]
   verbs: ["create", "get", "list", "watch", "update", "patch", "delete"]
+{{- end }}
 # Traefik Middleware permissions
-{{- if .Values.traefik.enabled }}
+{{- if and .Values.traefik.enabled .Values.dask.enabled }}
 - apiGroups: ["traefik.containo.us"]
   resources: ["middlewares"]
   verbs: ["get", "list", "watch", "create", "update", "patch", "delete"]
diff --git a/helm/reana/values.yaml b/helm/reana/values.yaml
index 4189eea5..a11e174f 100644
--- a/helm/reana/values.yaml
+++ b/helm/reana/values.yaml
@@ -173,6 +173,9 @@ traefik:
     enabled: true
     isDefaultClass: true
 
+dask:
+  enabled: true
+
 pgbouncer:
   enabled: false
   image: docker.io/bitnami/pgbouncer:1.23.1