Add option to disable TTY (#243)

itzg · Dec 14, 2024 · 62869b5 · 62869b5
1 parent 5eb6182
commit 62869b5
Show file tree

Hide file tree

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

diff --git a/charts/minecraft/templates/deployment.yaml b/charts/minecraft/templates/deployment.yaml
@@ -72,7 +72,7 @@ spec:
       - name: {{ template "minecraft.fullname" . }}
         image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}"
         imagePullPolicy: {{ .Values.image.pullPolicy }}
-        tty: true
+        tty: {{ default true .Values.minecraftServer.tty }}
         stdin: true
         {{- if or (.Values.lifecycle.postStart) (.Values.lifecycle.preStop)}}
         lifecycle:

diff --git a/charts/minecraft/values.yaml b/charts/minecraft/values.yaml
@@ -365,6 +365,9 @@ minecraftServer:
     ## Set the externalTrafficPolicy in the Service to either Cluster or Local
     # externalTrafficPolicy: Cluster
 
+  ## set this to false to not have colorized logs
+  tty: true
+
   extraPorts:
     []