TeamSpeak chart: naming scheme, hardening, configurable node ports

Batch-3 major PR for teamspeak-server (issue #32):

- Rename service and StatefulSet to the repo naming scheme
  ({{ .Release.Name }}-{{ .Chart.Name }}-service / -sfs); the PVC keeps
  its historical name (documented naming exception, data preservation).
- NodePorts configurable via server.voicePort / server.fileTransferPort
  (defaults 30987/30033, satisfactory pattern); ports are also passed to
  the server via TSSERVER_DEFAULT_PORT / TSSERVER_FILE_TRANSFER_PORT.
- Hardened, values-configurable security contexts
  (securityContext.pod/container): non-root uid/gid 9987,
  readOnlyRootFilesystem, all capabilities dropped.
- Probes values-configurable (defaults unchanged).
- Additive env value rendered through tpl (value/secretKeyRef/
  configMapKeyRef forms).
- Chart 3.1.2 -> 4.0.0 (+up6.0.0.beta12.1).

teamspeak-server/templates/teamspeak-server.service.yaml

@@ -1,19 +1,21 @@
 apiVersion: v1
 kind: Service
 metadata:
-  name: {{ .Release.Name }}-service
+  name: {{ .Release.Name }}-{{ .Chart.Name }}-service
+  labels:
+    app: {{ .Release.Name }}-{{ .Chart.Name }}-service
 spec:
   selector:
-    app: {{ .Release.Name }}-server-sfs
+    app: {{ .Release.Name }}-{{ .Chart.Name }}-sfs
   ports:
-    - port: 30987
+    - port: {{ .Values.server.voicePort }}
       name: voice-port
       targetPort: voice-port
       protocol: UDP
-      nodePort: 30987
-    - port: 30033
+      nodePort: {{ .Values.server.voicePort }}
+    - port: {{ .Values.server.fileTransferPort }}
       name: file-transfer
       targetPort: file-transfer
       protocol: TCP
-      nodePort: 30033
+      nodePort: {{ .Values.server.fileTransferPort }}
   type: NodePort

teamspeak-server/templates/teamspeak-server.sfs.yaml

@@ -1,31 +1,33 @@
 apiVersion: apps/v1
 kind: StatefulSet
 metadata:
-  name: {{ .Release.Name }}-server-sfs
+  name: {{ .Release.Name }}-{{ .Chart.Name }}-sfs
   labels:
-    app: {{ .Release.Name }}
+    app: {{ .Release.Name }}-{{ .Chart.Name }}-sfs
 spec:
   replicas: {{ include "teamspeak-server.replicas" (dict "scaleDown" .Values.scaleDown "replicas" .Values.replicas) }}
   selector:
     matchLabels:
-      app: {{ .Release.Name }}-server-sfs
+      app: {{ .Release.Name }}-{{ .Chart.Name }}-sfs
   template:
     metadata:
-      name: {{ .Release.Name }}-server-sfs
+      name: {{ .Release.Name }}-{{ .Chart.Name }}-sfs
       labels:
-        app: {{ .Release.Name }}-server-sfs
+        app: {{ .Release.Name }}-{{ .Chart.Name }}-sfs
     spec:
       automountServiceAccountToken: false
+      securityContext:
+        {{- toYaml .Values.securityContext.pod | nindent 8 }}
       containers:
-        - name: {{ .Release.Name }}-ts-server-container
+        - name: {{ .Release.Name }}-{{ .Chart.Name }}-sfs
           image: "teamspeaksystems/teamspeak6-server:{{ .Chart.AppVersion }}"
           imagePullPolicy: IfNotPresent
           ports:
             - name: voice-port
-              containerPort: 30987
+              containerPort: {{ .Values.server.voicePort }}
               protocol: UDP
             - name: file-transfer
-              containerPort: 30033
+              containerPort: {{ .Values.server.fileTransferPort }}
               protocol: TCP
             - name: web-query
               containerPort: 10080
@@ -34,40 +36,47 @@ spec:
             - name: TSSERVER_LICENSE_ACCEPTED
               value: "accept"
             - name: TSSERVER_DEFAULT_PORT
-              value: "30987"
-          # The voice service is UDP and cannot be probed; probe the TCP
-          # file-transfer port instead.
+              value: {{ .Values.server.voicePort | quote }}
+            - name: TSSERVER_FILE_TRANSFER_PORT
+              value: {{ .Values.server.fileTransferPort | quote }}
+            {{- range $value := .Values.env }}
+            {{- if $value.value }}
+            - name: {{ $value.name }}
+              value: {{ tpl $value.value $ }}
+            {{- else if $value.valueFrom }}
+            {{- if $value.valueFrom.secretKeyRef }}
+            - name: {{ $value.name }}
+              valueFrom:
+                secretKeyRef:
+                  name: {{ tpl $value.valueFrom.secretKeyRef.name $ }}
+                  key: {{ tpl $value.valueFrom.secretKeyRef.key $ }}
+            {{- else if $value.valueFrom.configMapKeyRef }}
+            - name: {{ $value.name }}
+              valueFrom:
+                configMapKeyRef:
+                  name: {{ tpl $value.valueFrom.configMapKeyRef.name $ }}
+                  key: {{ tpl $value.valueFrom.configMapKeyRef.key $ }}
+            {{- end }}
+            {{- end }}
+            {{- end }}
           livenessProbe:
-            tcpSocket:
-              port: file-transfer
-            periodSeconds: 10
-            timeoutSeconds: 5
-            failureThreshold: 3
+            {{- toYaml .Values.livenessProbe | nindent 12 }}
           readinessProbe:
-            tcpSocket:
-              port: file-transfer
-            periodSeconds: 10
-            timeoutSeconds: 5
-            failureThreshold: 3
-          # The startup probe gates liveness/readiness. Startup budget = 5s * 30 = 150s.
+            {{- toYaml .Values.readinessProbe | nindent 12 }}
           startupProbe:
-            tcpSocket:
-              port: file-transfer
-            periodSeconds: 5
-            timeoutSeconds: 5
-            failureThreshold: 30
+            {{- toYaml .Values.startupProbe | nindent 12 }}
           resources:
             {{- include "teamspeak-server.resources" .Values.resources | nindent 12 }}
+          securityContext:
+            {{- toYaml .Values.securityContext.container | nindent 12 }}
           volumeMounts:
             - name: "teamspeak-volume"
               mountPath: /var/tsserver/
       restartPolicy: Always
-      securityContext:
-        fsGroup: 9987
       volumes:
         - name: "teamspeak-volume"
           persistentVolumeClaim:
+            # Deliberate exception from the naming scheme: renaming the PVC
+            # would detach existing data (see README, naming exceptions).
             claimName: {{ .Release.Name }}-teamspeak-volume-claim
-  serviceName: {{ .Release.Name }}-service
-
-
+  serviceName: {{ .Release.Name }}-{{ .Chart.Name }}-service