feat: Add container lifecycle to the main Minecraft container (#158)

itzg · Jun 23, 2023 · 155c89a · 155c89a
1 parent c9f601b
commit 155c89a
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 1 deletion.
diff --git a/charts/minecraft/Chart.yaml b/charts/minecraft/Chart.yaml
@@ -1,6 +1,6 @@
 apiVersion: v1
 name: minecraft
-version: 4.9.0
+version: 4.9.1
 appVersion: SeeValues
 home: https://minecraft.net/
 description: Minecraft server

diff --git a/charts/minecraft/templates/deployment.yaml b/charts/minecraft/templates/deployment.yaml
@@ -142,6 +142,23 @@ spec:
         imagePullPolicy: {{ .Values.image.pullPolicy }}
         tty: true
         stdin: true
+        {{- if or (.Values.lifecycle.postStart) (.Values.lifecycle.preStop)}}
+        lifecycle:
+          {{- if .Values.lifecycle.postStart }}
+          postStart:
+            exec:
+              command: {{- range .Values.lifecycle.postStart }}
+                - {{ . }}
+              {{- end }}
+          {{- end }}
+          {{- if .Values.lifecycle.preStop }}
+          preStop:
+            exec:
+              command: {{- range .Values.lifecycle.postStart }}
+                - {{ . }}
+              {{- end }}
+          {{- end }}
+        {{- end }}
         resources:
 {{ toYaml .Values.resources | indent 10 }}
         {{- if .Values.startupProbe.enabled }}

diff --git a/charts/minecraft/values.yaml b/charts/minecraft/values.yaml
@@ -18,6 +18,10 @@ resources:
     memory: 512Mi
     cpu: 500m
 
+lifecycle:
+  postStart: []
+  preStop: []
+
 # upgrade strategy type (e.g. Recreate or RollingUpdate)
 strategyType: Recreate