Merge pull request #60 from SQLJames/feature/pvc-labels

adding dynamic labels to the pvc

Author: SQLJames

CHANGELOG.md

@@ -1,5 +1,11 @@
 ## Changelog
 
+### V2.0.4
+
+#### Non-Breaking Changes
+
+- Added new PVC labels - Thank you ohnoitsyou 
+
 ### V2.0.3
 
 #### Non-Breaking Changes

charts/factorio-server-charts/Chart.yaml

@@ -20,7 +20,7 @@ sources:
 # 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: 2.0.3
+version: 2.0.4
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Versions are not expected to

charts/factorio-server-charts/README.md

@@ -236,6 +236,7 @@ If you do run into any issues with mods, I will try to work with you on finding
 | `persistence.dataDir.existingClaim` | The name of an existing PVC to use for persistence |        |
 | `persistence.storageClassName`      | Persistent Volume storage class                    | `""`   |
 | `persistence.annotations`           | Persistent Volume Claim annotations                | `{}`   |
+| `persistence.lables`                | Persistent Volume Claim labels                     | `{}`   |
 
 ### Factorio Parameters
 

charts/factorio-server-charts/templates/datadir-pvc.yaml

@@ -6,6 +6,9 @@ metadata:
   name: {{ template "factorio-server-charts.fullname" . }}-datadir
   labels:
     app: "{{ template "factorio-server-charts.fullname" . }}-{{ .Chart.Version }}"
+    {{- range $key, $val := .Values.persistence.labels }}
+    {{ $key }}: {{ $val | quote }}
+    {{- end}}
 {{- if .Values.persistence.annotations }}
   annotations:
 {{ toYaml .Values.persistence.annotations | indent 4 }}

charts/factorio-server-charts/values.yaml

@@ -106,6 +106,7 @@ affinity: {}
 ## @extra persistence.dataDir.existingClaim The name of an existing PVC to use for persistence
 ## @param persistence.storageClassName Persistent Volume storage class
 ## @param persistence.annotations Persistent Volume Claim annotations
+## @param persistence.labels Persistent Volume Claim labels
 ## Sets the data persistence volume configuration
 ## IMPORTANT: If you do not setup a PV all your savegames will be lost on pod recreation or helm upgrade
 persistence:
@@ -118,6 +119,8 @@ persistence:
   storageClassName: ""
   ## If you have annotations to spin up a PV i.e. the location. Remove the curly braces
   annotations: {}
+  ## If you have additional labels to apply to the volume
+  labels: {}
 
 
 #### Factorio application configuration ####