Replace hard-coded PVC name

Uses a template to calculate the PVC name, matching the pattern used for other objects like the Deployment.
PrefectHQ · Dec 5, 2024 · 51e0515 · 51e0515
1 parent 0af72ae
commit 51e0515
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/charts/prefect-server/templates/deployment.yaml b/charts/prefect-server/templates/deployment.yaml
@@ -165,7 +165,7 @@ spec:
         {{- if .Values.sqlite.enabled }}
         - name: sqlite-storage
           persistentVolumeClaim:
-            claimName: sqlite-pvc
+            claimName: {{ template "common.names.fullname" . }}-sqlite
         {{- end }}
         {{- if .Values.server.extraVolumes }}
         {{- include "common.tplvalues.render" (dict "value" .Values.server.extraVolumes "context" $) | nindent 8 }}

diff --git a/charts/prefect-server/templates/pvc.yaml b/charts/prefect-server/templates/pvc.yaml
@@ -2,7 +2,7 @@
 apiVersion: v1
 kind: PersistentVolumeClaim
 metadata:
-  name: sqlite-pvc
+  name: {{ template "common.names.fullname" . }}-sqlite
   namespace: {{.Release.Namespace}}
 spec:
   {{- with .Values.sqlite.persistence.storageClassName }}

diff --git a/charts/prefect-server/tests/database_test.yaml b/charts/prefect-server/tests/database_test.yaml
@@ -37,7 +37,7 @@ tests:
           content:
             name: sqlite-storage
             persistentVolumeClaim:
-              claimName: sqlite-pvc
+              claimName: prefect-server-sqlite
       - template: deployment.yaml
         contains:
           path: .spec.template.spec.containers[0].volumeMounts
@@ -48,7 +48,7 @@ tests:
         containsDocument:
           kind: PersistentVolumeClaim
           apiVersion: v1
-          name: sqlite-pvc
+          name: prefect-server-sqlite
           namespace: prefect
 
   - it: Should not allow an HPA