4 Commits

Include renames

Author	SHA1	Message	Date
gregandClaude Opus 5.5	72a4280d44	fix(builder): boot drafts with the settings a new server really gets ... The builder's boot test filled every blank password-like setting with a throwaway value. A real deploy only fills x-generate fields, so a draft whose server refuses to run with an empty join password passed the builder and CI, and then shut down on its first real deploy. The builder now boots drafts with boottest-payload -as-deployed: only x-generate fields are filled, everything else exactly as the template's defaults. A server that won't run with its own defaults fails the build, and the model is told that empty settings stay empty, so the value must come from a default or x-generate. CI's boot test is unchanged. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>	2026-10-25 21:14:17 -04:00
gregandClaude Opus 5.5	2ad445ffdd	fix(ark): run as the image's own user; honour custom RCON ports ... ARK's image is built around user gameserver (uid 25000) and locks /home/gameserver to it, so under the node's default 1000:1000 the startup script got "Permission denied" and the server restart-looped, on real nodes as well as in boot tests. Set containerUser 25000:25000 (needs node-agent and panel support for containerUser). The startup command read ${RCON_PORT}, but the node passes the RCONPort setting as RCONPORT, so custom RCON ports were ignored for 27020. Boot-tested in the Linux agent image: up 30 minutes with no restart. The test now waits 10 minutes, covering the first ~13 GB download. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>	2026-10-24 22:17:03 -04:00
gregandClaude Opus 5.5	944b5fdd3e	fix(boot-test): never overwrite non-password settings ... The payload builder filled anything matching the validator's broad credential pattern that wasn't a non-empty string, so RCON_PORT, ENABLE_RCON and ARK's RCONEnabled/RCONPort became 'boot-123456' in the first runner run. Only fill keys named like passwords (or x-generate), and only when missing or an empty string. Adds a test. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>	2026-10-24 21:33:29 -04:00
gregandClaude Opus 5.5	e9ff989246	ci: boot-test changed games before publishing ... Only games whose folder changed are boot-tested (against the previous main commit, or against main on branches; manual runs pick games). Each one is deployed by the node-agent's real deploy code on a dedicated game-test runner, in a network that can reach the internet but not the LAN. Publishing to the panel runs only if they all booted. - boottest-payload builds the DEPLOY_SERVER message the panel would send - readyPattern/failPatterns in game.yaml; readyExample in test.yaml is checked against the pattern during validation - secrets in test.yaml come from CI; a game missing one is skipped - Minecraft gets a ready pattern; Dragonwilds needs DRAGONWILDS_OWNER_ID - RUNNER.md + scripts/runner: runner, isolated network, systemd unit Also fixes newline values ("\n" separators/footers) that yaml.v3 wrote as "" on import, which would have broken ARK and Dragonwilds config files once synced, and adds a round-trip test for the importer. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>	2026-10-24 17:49:19 -04:00