Files
jklein 28a0230ef0
Validate Helm Charts / Detect changed charts (pull_request) Successful in 3s
Validate Helm Charts / Resource quantities (pull_request) Successful in 23s
Validate Helm Charts / Bump level (pull_request) Successful in 4s
Validate Helm Charts / Selector match (pull_request) Successful in 12s
Validate Helm Charts / HTTPRoute match (pull_request) Successful in 10s
Validate Helm Charts / YAML scalars (pull_request) Successful in 9s
Validate Helm Charts / NetworkPolicy (pull_request) Failing after 10s
Validate Helm Charts / Trivy config apache-tika (pull_request) Successful in 23s
Validate Helm Charts / Build apache-tika (pull_request) Successful in 14s
Validate Helm Charts / Build bentopdf (pull_request) Successful in 6s
Validate Helm Charts / Build bulwark (pull_request) Successful in 6s
Validate Helm Charts / Build calibre-web-automated (pull_request) Successful in 5s
Validate Helm Charts / Build cyberchef (pull_request) Successful in 5s
Validate Helm Charts / Build homeassistant (pull_request) Successful in 10s
Validate Helm Charts / Build garage (pull_request) Successful in 6s
Validate Helm Charts / Build gotenberg (pull_request) Successful in 11s
Validate Helm Charts / Build keycloak (pull_request) Successful in 5s
Validate Helm Charts / Build minecraft-router (pull_request) Successful in 8s
Validate Helm Charts / Build patchmon (pull_request) Successful in 8s
Validate Helm Charts / Build rallly (pull_request) Successful in 5s
Validate Helm Charts / Build satisfactory (pull_request) Successful in 5s
Validate Helm Charts / Build minecraft-server (pull_request) Successful in 7s
Validate Helm Charts / Build outline (pull_request) Successful in 5s
Validate Helm Charts / Build rss-bridge (pull_request) Successful in 5s
Validate Helm Charts / Build samba (pull_request) Successful in 5s
Validate Helm Charts / Build searxng (pull_request) Successful in 5s
Validate Helm Charts / Build sure (pull_request) Successful in 11s
Validate Helm Charts / Build teamspeak-server (pull_request) Successful in 18s
Validate Helm Charts / Build template-chart (pull_request) Successful in 12s
Validate Helm Charts / Build urlaubsverwaltung (pull_request) Successful in 4s
Validate Helm Charts / Build ytdlp-web-player (pull_request) Successful in 5s
Validate Helm Charts / Build voucher-vault (pull_request) Successful in 5s
Validate Helm Charts / Build zipline (pull_request) Successful in 4s
Validate Helm Charts / Trivy config bentopdf (pull_request) Successful in 13s
Validate Helm Charts / Trivy config bulwark (pull_request) Successful in 15s
Validate Helm Charts / Trivy config keycloak (pull_request) Successful in 12s
Validate Helm Charts / Trivy config minecraft-router (pull_request) Successful in 17s
Validate Helm Charts / Trivy config calibre-web-automated (pull_request) Successful in 11s
Validate Helm Charts / Trivy config cyberchef (pull_request) Successful in 13s
Validate Helm Charts / Trivy config garage (pull_request) Successful in 20s
Validate Helm Charts / Trivy config gotenberg (pull_request) Successful in 14s
Validate Helm Charts / Trivy config homeassistant (pull_request) Successful in 11s
Validate Helm Charts / Trivy config outline (pull_request) Successful in 14s
Validate Helm Charts / Trivy config rallly (pull_request) Successful in 15s
Validate Helm Charts / Trivy config minecraft-server (pull_request) Successful in 27s
Validate Helm Charts / Trivy config patchmon (pull_request) Successful in 12s
Validate Helm Charts / Trivy config rss-bridge (pull_request) Successful in 13s
Validate Helm Charts / Trivy config teamspeak-server (pull_request) Successful in 11s
Validate Helm Charts / Trivy config samba (pull_request) Successful in 16s
Validate Helm Charts / Trivy config satisfactory (pull_request) Successful in 22s
Validate Helm Charts / Trivy config searxng (pull_request) Successful in 22s
Validate Helm Charts / Trivy config sure (pull_request) Successful in 17s
Validate Helm Charts / Trivy config template-chart (pull_request) Successful in 14s
Validate Helm Charts / Trivy config urlaubsverwaltung (pull_request) Successful in 11s
Validate Helm Charts / Trivy config voucher-vault (pull_request) Successful in 12s
Validate Helm Charts / Trivy config ytdlp-web-player (pull_request) Successful in 17s
Validate Helm Charts / Trivy config zipline (pull_request) Successful in 18s
Install-test / Install-test bulwark (pull_request) Successful in 1m18s
Install-test / Install-test homeassistant (pull_request) Successful in 2m3s
Install-test / Install-test outline (pull_request) Successful in 2m9s
Install-test / Install-test rss-bridge (pull_request) Successful in 51s
Install-test / Install-test samba (pull_request) Successful in 1m14s
Install-test / Install-test apache-tika (pull_request) Successful in 1m7s
Install-test / Install-test bentopdf (pull_request) Successful in 1m5s
Install-test / Install-test calibre-web-automated (pull_request) Successful in 1m47s
Install-test / Install-test cyberchef (pull_request) Successful in 49s
Install-test / Install-test garage (pull_request) Successful in 1m10s
Install-test / Install-test gotenberg (pull_request) Successful in 1m35s
Install-test / Install-test keycloak (pull_request) Successful in 2m23s
Install-test / Install-test minecraft-router (pull_request) Successful in 37s
Install-test / Install-test minecraft-server (pull_request) Successful in 1m34s
Install-test / Install-test patchmon (pull_request) Successful in 1m17s
Install-test / Install-test rallly (pull_request) Successful in 1m30s
Install-test / Install-test searxng (pull_request) Successful in 1m9s
Install-test / Install-test sure (pull_request) Successful in 2m10s
Install-test / Install-test voucher-vault (pull_request) Successful in 1m48s
Install-test / Install-test ytdlp-web-player (pull_request) Successful in 59s
Install-test / Install-test zipline (pull_request) Successful in 1m59s
Validate Helm Charts / Install-test (pull_request) Successful in 33m44s
Install-test / Install-test teamspeak-server (pull_request) Successful in 1m15s
Install-test / Install-test urlaubsverwaltung (pull_request) Successful in 1m51s
feat!: NetworkPolicy in the NodePort and game-server charts, on by default
garage, minecraft-router, minecraft-server, samba, teamspeak-server,
satisfactory. One major each. These are the charts the plan said to leave for
last, and they earned it - three of them changed the convention rather than
just consuming it.

GARAGE is the clearest illustration after keycloak. Its four ports split three
ways: s3 and web are RESTRICTED by source (no probe touches them), admin is the
probe path and stays open inside the cluster - which is uncomfortable, because
admin is also the admin API, and that is said out loud instead of glossed - and
rpc gets the narrowest rule the object can express, a podSelector with no
namespaceSelector. All four numbers come from garage.settings.

MINECRAFT-SERVER is the tightest policy in the repository, and the reason is
worth knowing: all three of its probes are EXEC probes, which put no packet on
the network, so no port has to stay open for the kubelet and no probeSources
value is needed at all. RCON and metrics are restricted outright. The game port
has two branches rendered from the same value the Service reads -
server.service.type NodePort means players arrive from the internet and the
rule must be open; anything else means the only path is in-cluster, so the rule
is restricted and the router's namespace has to be in allowedNamespaces.

MINECRAFT-ROUTER derives its egress from the routing table instead of guessing:
server.backendPort plus the "port" override of every mapping that sets one. Add
a backend on a non-standard port and the policy follows. What it cannot express
is the destination - mappings name hosts, and this object matches no hostname -
so the rule is wider than the routing table, and that is written down.

SATISFACTORY gets policyTypes: [Ingress] and no egress section, and the second
half of the reason is the deciding one. Steam's content delivery is not a
pinnable port set (SteamCMD over TCP 80/443/27015-27050, master servers over
UDP 27000-27100 and 4380, changed by Valve, re-downloaded on every update) -
and this is the ONE chart with no install-test at all, because the image is
multiple GB and the first start pulls over 10GB. An egress cap here would be
the only one in the sweep with no empirical check behind it, on the chart whose
correct port set is least knowable.

SAMBA and TEAMSPEAK-SERVER carry the NodePort lesson explicitly: the rules name
445/139 and the pod's voice and file-transfer ports, never the NodePorts, and
teamspeak's file is explicit that its pod port and NodePort happening to share a
number is a coincidence of its values rather than a pattern to copy. Teamspeak's
voice rule is UDP, and the file says why that matters: a TCP rule on the same
number is accepted, looks complete, and drops every voice packet.

THREE THINGS THE CHECKS CAUGHT WHILE THIS BATCH WAS WRITTEN, all of them real:

  - minecraft-server's RCON rule named a port the pod does not declare. The
    containerPort is gated on server.rcon.enabled and the rule was not, so
    assertion 2 rejected it. The rule now carries the same gate.
  - samba and satisfactory would have shipped allowedNamespaces and
    probeSources values that reach NO rule - every one of their inbound rules
    is open to any source. A value that silently does nothing is worse than one
    that is absent, because it reads as a control, so both are gone from those
    two charts and from their schemas, with the reason in values.yaml.
    teamspeak-server loses probeSources for the same reason and keeps
    allowedNamespaces, which its web-query rule uses.
  - the policy templates no longer declare template variables they do not use.
    samba keeps one, satisfactory none.

Part of #5
2026-10-14 14:37:51 +00:00

429 lines
23 KiB
Plaintext

{{- $replicas := include "minecraft-server.replicas" (dict "scaleDown" .Values.scaleDown "replicas" .Values.server.replicas) -}}
{{- $res := fromYaml (include "minecraft-server.serverResources" .) -}}
{{- $req := $res.requests | default dict -}}
{{- $lim := $res.limits | default dict -}}
{{- $reqMem := include "minecraft-server.formatNumber" (index $req "memory") -}}
{{- $limMem := include "minecraft-server.formatNumber" (index $lim "memory") -}}
{{- $startup := fromYaml (include "minecraft-server.startupProbe" .Values.startupProbe) -}}
{{- $budget := "" -}}
{{- if and (kindIs "float64" $startup.periodSeconds) (kindIs "float64" $startup.failureThreshold) -}}
{{- $budget = printf "%.0f seconds" (mulf $startup.periodSeconds $startup.failureThreshold) -}}
{{- end -}}
{{- $tag := include "minecraft-server.imageTag" . -}}
{{- $line := include "minecraft-server.javaLine" . -}}
{{- $tagOverridden := ne (include "minecraft-server.imageTagSource" .) ".Chart.AppVersion" -}}
{{- $svc := printf "%s-%s-service" .Release.Name .Chart.Name -}}
{{- $sfs := printf "%s-%s-sfs" .Release.Name .Chart.Name -}}
{{- $pod := printf "%s-%s-sfs-0" .Release.Name .Chart.Name -}}
{{- $pvc := printf "%s-mc-volume-claim" .Release.Name -}}
{{- $secret := printf "%s-%s-secret" .Release.Name .Chart.Name -}}
{{- $nodePort := .Values.server.service.type | eq "NodePort" -}}
{{- /* Whether the operator set server.resources.requests.memory themselves;
an unset one is derived from the JVM heap by serverResources. */ -}}
{{- $givenReq := dict -}}
{{- $sr := .Values.server.resources -}}
{{- if kindIs "map" $sr -}}
{{- if kindIs "map" $sr.requests -}}
{{- $givenReq = $sr.requests -}}
{{- end -}}
{{- end -}}
{{- $memPinned := not (kindIs "invalid" (index $givenReq "memory")) -}}
minecraft-server {{ $tag }} — release {{ .Release.Name }} in namespace {{ .Release.Namespace }}.
ACCESS — A MINECRAFT SERVER, NOT A WEB SERVICE
This release has no Ingress, no certificate and no URL. Clients speak the
Minecraft protocol over raw TCP.
{{- if $nodePort }}
server.service.type is NodePort, so players dial a node of this cluster
directly:
<any-node-ip>:{{ .Values.server.service.outwardPort }}
server.service.outwardPort ({{ .Values.server.service.outwardPort }}) is the port on the node and the one that
goes into the client. server.service.port ({{ .Values.server.service.port }}) is what the Service publishes
inside the cluster and server.port ({{ .Values.server.port }}) is what the server listens on in the
container; neither of those two is what a player types.
{{- if or (lt (int .Values.server.service.outwardPort) 30000) (gt (int .Values.server.service.outwardPort) 32767) }}
{{ .Values.server.service.outwardPort }} IS OUTSIDE THE DEFAULT NODE-PORT RANGE (30000-32767)
Unless the API server runs with an extended --service-node-port-range it
rejects this Service, and the release fails on that one object while every
other object applies cleanly. Confirm it exists:
kubectl -n {{ .Release.Namespace }} get svc {{ $svc }}
{{- end }}
{{- else }}
server.service.type is ClusterIP, so nothing outside the cluster reaches this
server on its own. Inside the cluster it answers at
{{ $svc }}.{{ .Release.Namespace }}.svc.cluster.local:{{ .Values.server.service.port }}
The way in for players is the separate minecraft-router release, and that
router does NOT discover backends by itself: it routes on the hostname out of
the Minecraft handshake, using the static mappings table in its own values
file. Until an entry pointing at the Service name above is added THERE, this
server is unreachable from outside no matter how healthy it is here — and no
annotation on this release changes that.
mc.example.com → {{ $svc }}.{{ .Release.Namespace }}.svc.cluster.local:{{ .Values.server.service.port }}
Set server.service.type to NodePort instead to publish it on a node port
without the router; then server.service.outwardPort ({{ .Values.server.service.outwardPort }}) applies.
{{- end }}
Either way you can reach it from your own machine without changing the
Service — forward it and connect the client to localhost:{{ .Values.server.service.port }}:
kubectl -n {{ .Release.Namespace }} port-forward svc/{{ $svc }} {{ .Values.server.service.port }}:{{ .Values.server.service.port }}
THE JAVA LINE OF THIS RELEASE IS {{ $line | default "NOT RECOGNIZABLE FROM ITS TAG" | upper }}
The JVM is fixed by the image tag, not by a value of the game. This release
runs
itzg/minecraft-server:{{ $tag }}
with minecraft.version {{ .Values.minecraft.version }}.
{{- if $tagOverridden }}
That tag comes from image.tagOverride, NOT from the chart's appVersion
({{ .Chart.AppVersion }}). So the chart version no longer says what this release runs:
"helm list" reports {{ .Chart.Version }} while the pod is on the tag
above. That is the price of the override, and the reason it is not called
image.tag.
{{- else }}
That tag comes from the chart's appVersion; image.tagOverride is empty, so
the chart version tells the whole truth about the image.
{{- end }}
Since chart 5.0.0 THAT PAIRING IS CHECKED: if the line of the tag above does
not fit minecraft.version, the render fails and names both values. Two cases
are let through on purpose — a tag without a recognizable "java<N>" part, and
a minecraft.version that is not a plain numeric x[.y[.z]] (a snapshot, a
pre-release, "LATEST"). In those two the old failure mode still applies: the
release installs, and only the container fails afterwards — CrashLoopBackOff
once the startup budget{{ if $budget }} ({{ $budget }}){{ end }} is spent, with the reason in the log and
nowhere else:
kubectl -n {{ .Release.Namespace }} logs {{ $pod }}
The boundaries are Mojang's (javaVersion.majorVersion in the piston-meta
version manifest, per release):
up to 1.16.5 Java 8
1.17 - 1.17.1 Java 16
1.18 - 1.20.4 Java 17
1.20.5 - 1.21.11 Java 21
26.1 and later Java 25
ONLY THE NEWEST LINE IS MAINTAINED. This chart directory carries one line and
gets chart changes only for that one; java8 and java17 stay frozen at chart
4.0.0, java21 at 4.1.0. So moving the server across a line boundary is no
longer a change of chart version: set image.tagOverride to a tag of the target
line AND minecraft.version to a release of it, in the same upgrade. Changing
only one of the two now fails the render instead of leaving a broken pod
behind.
{{- if eq $replicas "0" }}
SCALED TO ZERO
{{ if .Values.scaleDown }}scaleDown: true{{ else }}server.replicas: 0{{ end }} — the StatefulSet runs with replicas: 0, so no server answers
until you set it back and upgrade again. The world PVC, the Service and the
OnePasswordItem stay in place, so nothing is lost.
{{- end }}
WHAT A GREEN ROLLOUT DOES NOT PROVE
All three probes run the image's mc-health, a real Minecraft server-list ping
against SERVER_PORT. Ready therefore means more here than "the process is
alive": the server is up and answering pings. It does not mean anyone can
play. A server-list ping is answered before any login is attempted, so every
reason a player is refused at login is invisible to all three probes — the
whitelist below is the usual one, and the rollout stays green throughout.
Two more blind spots worth knowing before you need them:
- helm returns before any of this. Without --wait, success is reported the
moment the objects are applied. On a fresh volume the image is then still
downloading the server jar{{ if or .Values.curseforge.enabled .Values.ftb.enabled .Values.curseforgeServerpack.enabled }} and the whole modpack{{ end }} before the world begins
to load, which is what the {{ if $budget }}{{ $budget }} {{ end }}startup budget is for. Watch the
rollout rather than trusting the exit code:
kubectl -n {{ .Release.Namespace }} rollout status statefulset/{{ $sfs }}
- the probes run INSIDE the pod. They say nothing about whether players can
get to it{{ if not $nodePort }} — with ClusterIP, a missing router mapping is a perfectly
healthy server that nobody can join{{ end }}.
{{- if .Values.minecraft.whitelist.enabled }}
{{- if not .Values.minecraft.whitelist.players }}
WHITELIST IS ON AND EMPTY — NOBODY CAN JOIN THIS SERVER
minecraft.whitelist.enabled is true while minecraft.whitelist.players is
unset. The chart emits ENABLE_WHITELIST=true and no WHITELIST, which is
white-list=true against an empty whitelist.json: every login is refused,
including the operators in minecraft.management.opPlayers. This is the image's
own behaviour, and it is exactly the state the probes cannot see — the server
answers pings and the rollout is green.
Set minecraft.whitelist.players, or set minecraft.whitelist.enabled to false.
{{- else }}
WHITELIST IS ON
Only these players can log in: {{ .Values.minecraft.whitelist.players }}
minecraft.whitelist.overridePolicy is {{ .Values.minecraft.whitelist.overridePolicy }}, which decides what happens to a
whitelist.json ALREADY on the volume: SKIP leaves it untouched (players then
changes nothing at all), MERGE adds these names to it, SYNCHRONIZE makes the
file match this list exactly and removes everyone else. Anyone added in-game
with /whitelist add is not in this value and does not survive a SYNCHRONIZE.
{{- end }}
{{- else }}
WHITELIST IS OFF, AND IT IS ACTIVELY OFF
minecraft.whitelist.enabled is false, so the chart emits ENABLE_WHITELIST=false
and the image rewrites white-list=false into the server.properties on the
volume. That matters because server.properties is persistent state: before
chart 2.0.0 a release that had once written white-list=true kept it forever,
since the chart had no way to say "off". It now does.
{{- end }}
{{- if .Values.minecraft.management.opPlayers }}
OPERATORS ARE SYNCHRONIZED FROM VALUES, NOT FROM THE GAME
minecraft.management.opPlayers is {{ .Values.minecraft.management.opPlayers }}, and the chart pins
EXISTING_OPS_FILE=SYNCHRONIZE: the image makes ops.json on the volume match
that list exactly on every start. A player promoted in-game with /op is not in
the value, so the next restart removes them again. Promote people in the
values file.
These names are also resolved against the external Playerdb API during
startup, and a name that does not resolve aborts the start. A typo here is
therefore not a missing operator but a server that never comes up — and an
outage of that third-party lookup delays every pod start.
{{- end }}
SECRETS — WHICH FIELDS THE 1PASSWORD ITEM MUST CARRY
The OnePasswordItem CR {{ $secret }} is rendered unconditionally, so
secrets.minecraftServerRef has to be set even when nothing consumes a key from
it; it has no default and would otherwise render an itemPath of null. The
template guards it with required(), so a release without it does not render at
all — this note tells you why the guard is there, not that you got past it.
secrets.minecraftServerRef → {{ .Values.secrets.minecraftServerRef }}
The operator copies the item's FIELDS into the Kubernetes Secret, one key per
field name. This release consumes:
{{- if .Values.server.rcon.enabled }}
rcon-password → RCON_PASSWORD (server.rcon.enabled is true)
{{- end }}
{{- if .Values.curseforge.enabled }}
curseforge-api-key → CF_API_KEY (curseforge.enabled is true)
{{- end }}
{{- if not (or .Values.server.rcon.enabled .Values.curseforge.enabled) }}
no key at all — RCON and the CurseForge downloader are both off, so the
item may be empty. It only needs "rcon-password" once server.rcon.enabled
is true and "curseforge-api-key" once curseforge.enabled is true.
{{- end }}
A field spelled differently yields a Secret without that key. The pod then
sticks in CreateContainerConfigError while the Secret itself exists, so it
looks as though the operator had done its job. What it actually delivered:
kubectl -n {{ .Release.Namespace }} get secret {{ $secret }} -o jsonpath='{.data}'
Pointing secrets.minecraftServerRef at a DIFFERENT item rolls the pod (the
StatefulSet hashes the rendered CR into checksum/secret-ref). Changing the
value INSIDE the same item does not: Helm never sees that content, the
rendered spec is unchanged, and the keys are consumed as environment
variables, which Kubernetes never refreshes in a running container. After
rotating a password in the vault, restart by hand:
kubectl -n {{ .Release.Namespace }} rollout restart statefulset/{{ $sfs }}
Budget for that restart: a full server start, measured at ~124 s for
ftb-skies on Longhorn.
THE WORLD LIVES ON ONE PVC THAT helm uninstall DELETES
/data and /modpacks are two subPaths of a single ReadWriteOnce claim,
{{ $pvc }} ({{ .Values.minecraft.resources.storage }} from storage class {{ .Values.minecraft.resources.storageClassName }}). Note the
name: it is deliberately NOT {{ .Release.Name }}-{{ .Chart.Name }}-..., because renaming a PVC
would make Kubernetes provision a new, empty volume and orphan the world.
It is an ordinary PVC in this release, not a StatefulSet volumeClaimTemplate.
"helm uninstall {{ .Release.Name }}" therefore deletes it, and with a Delete reclaim policy
the world goes with it. Back it up before uninstalling — a resource-policy
annotation added to the live PVC afterwards does not help, because Helm reads
that annotation from the release manifest.
Two more properties of that claim: minecraft.resources.storageClassName is
immutable once the volume is bound, so changing it does nothing to an existing
release; and growing minecraft.resources.storage only works if the storage
class allows expansion, while shrinking it is refused by Kubernetes and makes
the next helm upgrade fail on the PVC instead of touching the data.
server.properties lives on this volume too. That is why settings the chart
does not send as an environment variable persist across upgrades unchanged.
TWO MEMORY NUMBERS THAT MUST NOT DRIFT APART
minecraft.resources.memory ({{ .Values.minecraft.resources.memory }}) is the JVM heap, passed to the image as
MEMORY={{ .Values.minecraft.resources.memory }}G. This release runs with
memory request {{ $reqMem }} limit {{ $limMem | default "(none)" }} (limitFactor {{ .Values.server.resources.limitFactor | default 3 }})
{{- if not $memPinned }}
server.resources.requests.memory is unset, so the chart derives the request
from the heap and the two cannot drift. Keep it that way unless you have a
reason not to.
{{- else }}
server.resources.requests.memory is set explicitly, which DECOUPLES it from
the heap: the chart no longer derives one from the other. Keep the limit above
the heap plus the JVM's own overhead. A heap that does not fit the limit is
not a Java OutOfMemoryError — the kernel kills the container, and the pod shows
OOMKilled with nothing in the server log explaining it.
{{- end }}
{{- if .Values.server.rcon.enabled }}
RCON IS ENABLED BUT THE SERVICE DOES NOT PUBLISH IT
server.rcon.enabled is true, so the container declares port {{ .Values.server.rcon.port }} and the
chart wires RCON_PASSWORD from the Secret. The Service publishes only the
Minecraft port, so RCON is reachable from inside the pod's network and through
a forward — not through {{ $svc }}:
kubectl -n {{ .Release.Namespace }} port-forward {{ $pod }} {{ .Values.server.rcon.port }}:{{ .Values.server.rcon.port }}
Exposing RCON more widely is a decision to make deliberately: it is a remote
console with full server rights, protected by that one password.
{{- end }}
THE METRICS PORT IS DECLARED, NOT SERVED
The container declares port 19565 named "metrics". Declaring a containerPort
opens nothing: the Service does not publish it, this chart ships no
ServiceMonitor, and it sets no variable that would switch the image's metrics
on. Treat that entry as documentation, not as a scrape target.
{{- if .Values.curseforgeServerpack.enabled }}
MODPACK SOURCE: curseforgeServerpack
An init container downloads {{ .Values.curseforgeServerpack.downloadUrl }}
to /modpacks/serverpack.zip, and the server unpacks it below
{{ .Values.curseforgeServerpack.workingDirectory }}.
The script comes from the ConfigMap and is mounted with subPath, which the
kubelet never refreshes, so it is read only at pod start — the StatefulSet
carries checksum/config, which is what makes a changed URL take effect at all.
The restart is not free: expect a full server start (~124 s measured for
ftb-skies on Longhorn).
Its migration branch runs only when the working directory ALREADY exists; on
a first install the script just downloads, and the server does the unpacking.
When it does run, it copies banned-ips.json, banned-players.json, ops.json and
whitelist.json out of the backup unconditionally, under "set -e" — so an
existing serverpack directory that is missing any one of those four files
fails the init container, and the pod never starts. Check there first when a
serverpack upgrade hangs before the server logs anything:
kubectl -n {{ .Release.Namespace }} logs {{ $pod }} -c download-serverpack
{{- end }}
{{- if .Values.curseforge.enabled }}
MODPACK SOURCE: curseforge (AUTO_CURSEFORGE)
The image resolves slug "{{ .Values.curseforge.modpack.slug }}" / file id {{ .Values.curseforge.modpack.versionId }} against the CurseForge
API at every start, using the curseforge-api-key field of the item above. That
makes each start depend on a third-party API and on that key still being
valid; an expired key is a server that stops coming up, not a warning.
{{ if .Values.curseforge.forceReload }}curseforge.forceReload is true, so CF_FORCE_SYNCHRONIZE is set and the pack is
re-synchronized on every single start. Turn it off again once the reload has
happened — it costs that time at each restart.{{ else }}curseforge.forceReload is false, so an already installed pack is reused.{{ end }}
{{- end }}
{{- if .Values.ftb.enabled }}
MODPACK SOURCE: ftb (FTBA)
The image fetches modpack {{ .Values.ftb.modpackId }} version {{ .Values.ftb.versionId }} from the Feed-the-Beast API at
start. No API key is involved, but the start does depend on that service being
reachable, and the first one downloads the whole pack before the world loads.
{{- end }}
{{- if gt (atoi $replicas) 1 }}
MORE THAN ONE REPLICA WILL NOT WORK
server.replicas is {{ $replicas }}, but every pod of this StatefulSet mounts the same
ReadWriteOnce claim {{ $pvc }} — there is no volumeClaimTemplate and no
per-pod volume. Further pods stay stuck before start on a node that does not
hold the volume, and two Minecraft servers on one world directory would
corrupt it in any case. This chart is built for exactly one replica.
{{- end }}
{{- if .Values.env }}
server env IS APPENDED, NOT MERGED
The entries in "env" land after everything the chart sets itself and are
rendered through tpl, so a value may contain template expressions. Because the
chart's own variables come first, an entry repeating one of them is what the
image sees last — that is the escape hatch for anything the chart does not
model, and equally the way to break a setting the chart does model.
A plain "value" is stored in the pod spec in clear text and readable by
everyone who can read pods in {{ .Release.Namespace }}; anything sensitive belongs in
valueFrom.secretKeyRef.
{{- end }}
{{- if .Values.networkPolicy.enabled }}
NETWORKPOLICY IS ACTIVE — WHAT IT CAPS, AND WHAT IT DOES NOT
{{ .Release.Name }}-{{ .Chart.Name }}-networkpolicy selects this workload's pods.
Inbound: the game port (TCP {{ .Values.server.port }}){{ if eq .Values.server.service.type "NodePort" }} — OPEN, because the Service is a
NodePort and the senders are players on the internet{{ else }} — RESTRICTED to this namespace
and networkPolicy.allowedNamespaces, because the Service is {{ .Values.server.service.type }} and the only
path to it is in-cluster{{ end }}; RCON (TCP {{ .Values.server.rcon.port }}) and metrics (TCP 19565), both always
restricted by source. Outbound: DNS and TCP 443{{ with .Values.networkPolicy.egress }}{{ with .extraPorts }}, plus TCP {{ join ", " . }}{{ end }}{{ end }}.
THIS POLICY IS UNUSUALLY TIGHT, and the reason is that all three probes are
exec probes: they run inside the container and put no packet on the network,
so no port has to stay open for the kubelet and networkPolicy.probeSources is
not needed at all.
{{- if not (eq .Values.server.service.type "NodePort") }}
SO CHECK ONE THING: players reach this server through minecraft-router, which
runs in ITS OWN namespace. That namespace must be in
networkPolicy.allowedNamespaces, or the server is unreachable and the symptom
is a connection timeout in a game client with a perfectly healthy pod behind
it. The same goes for a Prometheus scraping port 19565 from elsewhere.
{{- end }}
Three limits belong in the same breath, because none of them is visible in
the object itself:
- EGRESS IS BY PORT, NEVER BY DESTINATION. A NetworkPolicy matches CIDRs and
selectors and cannot express a hostname, so every egress rule permits its
port to ANY address. That is a limit of the object, not a shortcut — this
cluster's CNI (Canal: Flannel plus Calico) has no FQDN rule either. An
outbound port the chart does not know about is BLOCKED: add it to
networkPolicy.egress.extraPorts, and expect to find out through an
application-level connection error rather than through this object.
{{- if .Values.networkPolicy.probeSources }}
- THE PROBE PATH IS RESTRICTED in this release, because
networkPolicy.probeSources is set to
{{ join ", " .Values.networkPolicy.probeSources }}
Probes come from the IP of the WORKER NODE running the pod. If none of those
CIDRs covers the node this pod lands on, the kubelet's probes are dropped
and THE POD NEVER BECOMES READY — the rollout then times out and the symptom
looks like a broken application. Compare
kubectl -n {{ .Release.Namespace }} describe pod -l app={{ .Release.Name }}-{{ .Chart.Name }}-sfs
against `kubectl get node -o wide`.
{{- else }}
- THE PROBED PORT IS OPEN INSIDE THE CLUSTER. networkPolicy.probeSources is
empty, so that rule carries no source selector at all and any pod the CNI
routes to can reach it. Not from the internet — but this is NOT pod-level
isolation. Narrowing it means listing the worker node IPs in
networkPolicy.probeSources, because the kubelet probes from the node's own
address and the chart cannot know it; a wrong guess there leaves the pod
permanently un-Ready.
{{- end }}
- IT IS ONLY A BOUNDARY WHERE THE CNI ENFORCES POLICIES. Where it does not,
this object is accepted and filters nothing, so everything reads as
configured while nothing is filtered. `kubectl -n {{ .Release.Namespace }} get networkpolicy`
shows the object exists; only a connection attempt from an unrelated
namespace shows that it bites.
Switch it off with networkPolicy.enabled: false.
{{- end }}