[Feat] Added support for modifying PVC through values.yaml

Signed-off-by: hanchenli <[email protected]>

helm/Chart.yaml

@@ -15,7 +15,7 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 0.0.5
+version: 0.0.6
 
 maintainers:
   - name: apostac

helm/templates/pvc.yaml

@@ -5,16 +5,32 @@ kind: PersistentVolumeClaim
 metadata:
   name: "{{ .Release.Name }}-{{$modelSpec.name}}-storage-claim"
   namespace: {{ .Release.Namespace }}
+  labels:
+    app.kubernetes.io/managed-by: Helm
+  annotations:
+    meta.helm.sh/release-name: "{{ .Release.Name }}"
+    meta.helm.sh/release-namespace: "{{ .Release.Namespace }}"
+{{- if $modelSpec.pvcAnnotations }}
+    {{- toYaml $modelSpec.pvcAnnotations | nindent 4 }}
+{{- end }}
 spec:
-  accessModes: {{ toYaml $modelSpec.pvcAccessMode | nindent 4 }}
+  accessModes:
+    {{- if $modelSpec.pvcAccessMode }}
+    {{- toYaml $modelSpec.pvcAccessMode | nindent 4 }}
+    {{- else }}
+    - ReadWriteOnce  # Default to EBS-compatible mode if not specified
+    {{- end }}
   resources:
     requests:
-      storage: {{ $modelSpec.pvcStorage }}
+      storage: {{ $modelSpec.pvcStorage | default "10Gi" }}  # Default to 10Gi if not set
+  {{- if $modelSpec.storageClass }}
+  storageClassName: "{{ $modelSpec.storageClass }}"  # Allow user-defined storageClass
+  {{- end }}
   {{- if not (empty $modelSpec.pvcMatchLabels) }}
   selector:
     matchLabels:
       {{- toYaml $modelSpec.pvcMatchLabels | nindent 8 }}
   {{- end }}
 {{- end }}
 ---
-{{- end }}
+{{- end }}