Merge pull request #10 from andyverberne/issue/#6A

Fixes #6 -- Removes media volume from open notificaties
open-zaak · Jun 30, 2021 · af70657 · af70657
2 parents cf54e62 + d65433a
commit af70657
Show file tree

Hide file tree

Showing 5 changed files with 1 addition and 34 deletions.
diff --git a/charts/open-notificaties/Chart.yaml b/charts/open-notificaties/Chart.yaml
@@ -3,7 +3,7 @@ name: open-notificaties
 description: API voor het routeren van notificaties
 
 type: application
-version: 0.3.0
+version: 0.3.1
 appVersion: 1.1.0
 
 dependencies:

diff --git a/charts/open-notificaties/README.md b/charts/open-notificaties/README.md
@@ -40,9 +40,6 @@ helm install open-notificaties open-zaak/open-notificaties \
 | `ingress.annotations` | Additional annotations on the API ingress | `{}` |
 | `ingress.hosts` | Ingress hosts | `"{open-notificaties.gemeente.nl}"` |
 | `ingress.tls` | Ingress TLS settings | `"[]"` |
-| `persistence.enabled` | Enable persistency for application media | `false` |
-| `persistence.size` | The size of the application media persistent volume | `"1Gi"` |
-| `persistence.existingClaim` | Use an existing claim for application media | `null` |
 | `settings.allowedHosts` | A comma-separated list of hosts allowed by the application | `"open-notificaties.gemeente.nl"` |
 | `settings.secretKey` | The secret key of the application | `"SOME-RANDOM-SECRET"` |
 | `settings.database.host` | The hostname of PostgreSQL | `"open-notificaties-postgresql"` |

diff --git a/charts/open-notificaties/templates/deployment.yaml b/charts/open-notificaties/templates/deployment.yaml
@@ -62,17 +62,6 @@ spec:
             {{- toYaml .Values.readinessProbe | nindent 12 }}
           resources:
             {{- toYaml .Values.resources | nindent 12 }}
-          volumeMounts:
-            - name: media
-              mountPath: /media
-      volumes:
-        - name: media
-        {{- if .Values.persistence.enabled }}
-          persistentVolumeClaim:
-            claimName: {{ if .Values.persistence.existingClaim }}{{ .Values.persistence.existingClaim }}{{- else }}{{ include "open-notificaties.fullname" . }}{{- end }}
-        {{- else }}
-          emptyDir: {}
-        {{- end }}
       {{- with .Values.nodeSelector }}
       nodeSelector:
         {{- toYaml . | nindent 8 }}

diff --git a/charts/open-notificaties/templates/pvc.yaml b/charts/open-notificaties/templates/pvc.yaml
diff --git a/charts/open-notificaties/values.yaml b/charts/open-notificaties/values.yaml
@@ -87,11 +87,6 @@ tolerations: []
 
 affinity: {}
 
-persistence:
-  enabled: false
-  size: 1Gi
-  existingClaim: null
-
 worker:
   replicaCount: 1
   resources: {}