From 13e3c2e297c7a6f44d726009b991172c34a53942 Mon Sep 17 00:00:00 2001
From: Bart Jeukendrup <bart@jeukendrup.nl>
Date: Fri, 4 Dec 2020 12:14:06 +0100
Subject: [PATCH] Add settings for datawarehouse

---
 backend/README.md                             |  6 ++++++
 backend/templates/deployment.yaml             |  8 ++++----
 .../templates/persistent-volume-claim.yaml    | 20 +++++++++++++++++--
 backend/values.yaml                           | 11 +++++++---
 4 files changed, 36 insertions(+), 9 deletions(-)

diff --git a/backend/README.md b/backend/README.md
index 75d1e8d..536109c 100644
--- a/backend/README.md
+++ b/backend/README.md
@@ -17,6 +17,12 @@ The backend Helm chart installs the Signalen API and the by default the followin
 | `image.tag` | The tag of the Docker image | `latest` |
 | `replicaCount` | The number of API replicas | `1` |
 | `workerReplicaCount` | The number of background worker replicas | `1` |
+| `persistence.media.enabled` | Enable persistence of media | `true` |
+| `persistence.media.size` | Specify the size of the media PVC | `1Gi` |
+| `persistence.media.existingClaim` | Name of an existing PVC to use | `null` |
+| `persistence.datawarehouse.enabled` | Enable persistence of datawarehouse | `false` |
+| `persistence.datawarehouse.size` | Specify the size of the datawarehouse PVC | `1Gi` |
+| `persistence.datawarehouse.existingClaim` | Name of an existing PVC to use for datawarehouse | `null` |
 | `settings.allowedHosts` | Restrict the allowed hosts of the API | `*` |
 | `settings.defaultPdokMunicipalities` | A (comma-seperated) list of [PDOK municipalities](https://www.pdok.nl/introductie/-/article/cbs-wijken-en-buurten) the API allows complaints for (e.g. `"Amsterdam,'s-Hertogenbosch"`) | `""` |
 | `settings.organizationName` | The name of the organization | `"Signalen"` |
diff --git a/backend/templates/deployment.yaml b/backend/templates/deployment.yaml
index 8110a19..aac2596 100644
--- a/backend/templates/deployment.yaml
+++ b/backend/templates/deployment.yaml
@@ -128,16 +128,16 @@ spec:
               containerPort: 8000
       volumes:
         - name: media
-        {{- if .Values.persistence.enabled }}
+        {{- if .Values.persistence.media.enabled }}
           persistentVolumeClaim:
-            claimName: {{ if .Values.persistence.existingClaim }}{{ .Values.persistence.existingClaim }}{{- else }}{{ template "signals-backend.fullname" . }}{{- end }}
+            claimName: {{ if .Values.persistence.media.existingClaim }}{{ .Values.persistence.media.existingClaim }}{{- else }}{{ template "signals-backend.fullname" . }}{{- end }}
         {{- else }}
           emptyDir: {}
         {{- end }}
         - name: dwh-media
-        {{- if .Values.persistence.enabled }}
+        {{- if .Values.persistence.datawarehouse.enabled }}
           persistentVolumeClaim:
-            claimName: {{ if .Values.persistence.existingClaim }}{{ .Values.persistence.existingClaim }}{{- else }}{{ template "signals-backend.fullname" . }}{{- end }}
+            claimName: {{ if .Values.persistence.datawarehouse.existingClaim }}{{ .Values.persistence.datawarehouse.existingClaim }}{{- else }}{{ template "signals-backend.fullname" . }}-datawarehouse{{- end }}
         {{- else }}
           emptyDir: {}
         {{- end }}
diff --git a/backend/templates/persistent-volume-claim.yaml b/backend/templates/persistent-volume-claim.yaml
index cafa26b..1f443c1 100644
--- a/backend/templates/persistent-volume-claim.yaml
+++ b/backend/templates/persistent-volume-claim.yaml
@@ -1,4 +1,5 @@
-{{- if and .Values.persistence.enabled (not .Values.persistence.existingClaim) }}
+{{- if and .Values.persistence.media.enabled (not .Values.persistence.media.existingClaim) }}
+---
 apiVersion: v1
 kind: PersistentVolumeClaim
 metadata:
@@ -10,5 +11,20 @@ spec:
     - ReadWriteOnce
   resources:
     requests:
-      storage: {{ .Values.persistence.size }}
+      storage: {{ .Values.persistence.media.size }}
+{{- end }}
+{{- if and .Values.persistence.datawarehouse.enabled (not .Values.persistence.datawarehouse.existingClaim) }}
+---
+apiVersion: v1
+kind: PersistentVolumeClaim
+metadata:
+  name: {{ template "signals-backend.fullname" . }}-datawarehouse
+  labels:
+    {{- include "signals-backend.labels" . | nindent 4 }}
+spec:
+  accessModes:
+    - ReadWriteMany
+  resources:
+    requests:
+      storage: {{ .Values.persistence.datawarehouse.size }}
 {{- end }}
diff --git a/backend/values.yaml b/backend/values.yaml
index a1b73a8..e4075a2 100644
--- a/backend/values.yaml
+++ b/backend/values.yaml
@@ -32,9 +32,14 @@ ingress:
         - api.signals.local
 
 persistence:
-  enabled: true
-  size: 1Gi
-  existingClaim: null
+  media:
+    enabled: true
+    size: 1Gi
+    existingClaim: null
+  datawarehouse:
+    enabled: false
+    size: 1Gi
+    existingClaim: null
 
 settings:
   allowedHosts: "*"