Repository navigation
Conversation
The audit found rpcbind on *:111 and nfs-blkmap on a node with no NFS storage, plus spiceproxy, postfix and unattended-upgrades the repo never named. build-the-playground.md §4 now lists each with its reason, disables and masks the NFS client units, and gives the ss read-back. §7 adds a power-on check for Proxmox packages, which unattended-upgrades does not cover. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
There was a problem hiding this comment.
🟡 Changes recommended
The runbook can leave kernel security updates inactive without a required reboot and repeat network checks, and its listener inventory omits owners and keep/remove rationale for two expected ports.
2 open issues
What changed in this PR
This docs-only PR documents service hardening and patching for gandalf, the Proxmox host supporting the range. The procedures have not yet been applied to the host.
Changes:
- Adds a service inventory, commands to disable unused NFS plumbing and mask
rpcbind, and expected listener checks. - Adds a power-on check and manual upgrade step for Proxmox packages outside unattended updates.
- Records the audit findings and new procedures in the changelog.
| File | Description |
|---|---|
docs/runbooks/build-the-playground.md |
Documents service decisions, NFS shutdown commands, and pre-session updates. |
docs/changelog.md |
Adds a dated record of the audit findings and runbook changes. |
Review effort: Balanced
Keeps both sides: §4 now has the SSH keys-only step from #1498, then the services subsection; the changelog keeps both entries. Review fixes folded in: - §4's service table adds pvedaemon (:85) and chronyd (:323), the two loopback listeners the ss read-back expects. - §7 reboots into a newly installed kernel (or on /run/reboot-required) and restarts the power-on checks from the top. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
docs/runbooks/build-the-playground.md§4 has a new subsection, Services ongandalf: only what the range uses. It lists every service ongandalfthat listens or patches, and why each one stays or goes. It also gives the commands that disable and maskrpcbind,nfs-blkmapandnfs-client.target, and thessread-back to expect.apt list --upgradable, thenapt full-upgradebefore the session starts).unattended-upgradescovers Debian's packages only.docs/changelog.mdhas a 2026-11-13 entry.Why
Finding 6 of the read-only audit of
gandalf:rpcbindlistens on*:111(tcp and udp), andnfs-blkmapruns, butstorage.cfghas no NFS storage. The range is therangeZFS pool.spiceproxy(*:3128),postfix(loopback) andunattended-upgradesalso run, and the repository named none of them.unattended-upgradesonly uses the Debian and Debian-Security origins, so Proxmox packages were patched only when someone ran apt by hand.Blast radius
This PR changes docs only, and nothing has been run on
gandalfyet:rpcbindmeans a future NFS storage would fail to mount until it is unmasked, and the runbook says so.spiceproxyandpostfixare part of Proxmox and stay. The firewall already drops 3128 from everywhere.This PR and #1498 both add a subsection just before §5, so whichever merges second needs a small merge.
Verification
make validate: all checks passed, with 46 skipped because there is no Docker here../scripts/lint.sh: passed, with 14 skipped (shellcheck, markdownlint and editorconfig, which CI runs).gandalfbefore writing:systemctl is-enabled/is-activefor each service named above.ss -Hltungave the port list in the table.storage.cfgandpvesm statusshow no NFS storage.apt-cache policyand50unattended-upgradesshow Proxmox is not among the unattended origins.🤖 Generated with Claude Code