Repository navigation
docs(range): §4 says what changes on gandalf (#421) - #1329
Merged
Merged
Conversation
§4 records the Saruman run and said gandalf "gets the same treatment". Three steps don't carry over. The collector, its alerts and the telemetry check need an agent, and ADR-0017 gives gandalf none. host.fw needs phoenix's 8006 rule too, because ADR-0074 builds tpl-kali through gandalf's own API. There are no guest NICs to clear yet. A substitution table now covers these, and §7 reads the firewall back at every power-on in place of the alerts. Refs #421 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
There was a problem hiding this comment.
🟢 Approval recommended
No active review issues are reported for the runbook documentation update.
No open issues 🎉
What changed in this PR
Clarifies how to apply the playground runbook’s recorded Saruman firewall procedure to gandalf, preparing the next range-build step for #421 without changing the lab.
Changes:
- Adds
gandalf-specific console, node path, address, guest-NIC, and firewall-rule guidance. - Limits collector and alert instructions to
Saruman, sincegandalfhas no monitoring agent. - Requires firewall status and connectivity checks before each range session.
| File | Description |
|---|---|
| docs/runbooks/build-the-playground.md | Documents gandalf substitutions and adds per-session firewall verification. |
Review effort: Balanced
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
build-the-playground.md§4 gains an Ongandalftable listing every value and step that differs from the recordedSarumanrun. §7's power-on sequence now begins by reading the firewall back. Nothing on the lab changes. This prepares the next step of the range build.Why
#421's next step is §4 on
gandalf. §4 records the 2026-09-20Sarumanrun (#566) and saidgandalf"gets the same treatment". Followed as written, three things would have gone wrong:up{}check are useless ongandalf. ADR-0017 gives it no agent, so nothing would read the collector's file. The table also expected3host rules ongandalf, which assumed something would count them. Since nothing watches it, the check that the firewall stays on becomes a §7 step at every power-on:pve-firewall statusplus the probe frommorpheus.host.fwongandalfneedsphoenix's 8006 rule. ADR-0074 buildstpl-kali(Build tpl-kali (VMID 902) on ifrit once it exists #790) ongandalffromphoenix.build-the-lab-templates.md§5 step 2 already says to admit it, but §4 saidgandalftakes three rules. It takes four.Saruman's 140 and 170.gandalfhas no guests yet (ADR-0106). The flag is cleared as §5 and §6 create each NIC. Clones oftpl-kalialready carryfirewall=0from Packer, but a NIC added in the GUI, like the attack VM'snet1, does not.The table also gives the console path, KVM port 4. AMT is disabled, so there is no second way in. It also gives the alias (
10.0.30.30), the lower-case node path and the probe target.Refs #421.
Blast radius
Docs only. The one runbook will be followed on
gandalf(VLAN 30).docs/security.md,network.mdandarchitecture.mdstill describeSaruman's firewall alone, and that is still true until §4 runs ongandalf. They change in that PR.secrets/*.sops.yamlVerification
python3 scripts/check_docs.py(in a venv fromscripts/requirements.txt):docs OK — 159 Prometheus + 20 Loki rules, 8 dashboards, 156 panels, 12 assertionsmarkdownlint-cli2on the file: 0 issues./scripts/lint.sh: passed, but 14 checks were skipped (no docker daemon or pipx here), so CI's--require-allrun is the real lintmake validatewas not run in fullmake validatepassesDeployed to the lab and confirmed working
Docs updated (
docs/, service README, ordocs/roadmap.md)🤖 Generated with Claude Code