docs(runbooks): correct the reverse-DNS claim, having now run this - #180
Merged
Conversation
Said a host override creates no reverse entry. It does — pfSense writes a local-data-ptr alongside the forward record. What it does not do is give every name one: only the first override claiming a given address gets the PTR. That detail is worth more than the wrong version was. lemmiwinks and oracle are one machine on 10.0.99.30, so the order they were added decided that reverse lookups of that address answer "lemmiwinks" and never "oracle" — and the same for prometheus over grafana on .20. Anything logging by PTR inherits that choice, and changing it later means deleting both and re-adding in the other order, because editing the second does not promote it. Also records that morpheus answers with two addresses, 10.0.99.1 and 10.7.7.1, because pfSense registers the firewall's hostname on each interface. That looked like drift when a dig returned the unexpected one mid-verification; it is round-robin over a pre-existing pair, and saying so here saves the next person the same detour. Marks the four as applied on 2026-08-30 rather than outstanding.
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.
Recovers a commit that was written, committed and pushed, but never merged.
How it went missing
#169 merged at
03:01:01Z. Commit3e723d6was made on the same branch at03:10:23Z— nine minutes later — and pushed, but no second PR was ever opened for it. It has been sitting onorigin/docs/runbook-host-overridesever since, reachable from nothing else.Found while auditing local branches after enabling
delete_branch_on_merge. Cherry-picked onto currentmainrather than reopening the old branch, which is 20 commits behind and whose name is already tied to the merged #169.Why it matters
maincurrently tells you the opposite of what is true:pfSense does write a
local-data-ptralongside the forward record. What it does not do is give every name one — only the first override claiming a given address gets the PTR:oracleandlemmiwinksare one machine on10.0.99.30, so the order they were added silently decided that reverse lookups of that address answerlemmiwinksand neveroracle. Same forprometheusovergrafanaon.20. Anything logging by PTR inherits that choice, and changing it later means deleting both and re-adding in the other order — editing the second does not promote it.The stale version is worse than merely incomplete: someone reading it would conclude reverse lookups are unavailable and go build something else, when the real constraint is that they exist and are first-come.
Also in this commit
morpheusanswers with two addresses,10.0.99.1and10.7.7.1, because pfSense registers the firewall's hostname on each interface. That looked like drift when adigreturned the unexpected one mid-verification; it is round-robin over a pre-existing pair.Checks
Cherry-picked cleanly onto
main— no conflicts; nothing has touchedadd-a-host-override.mdsince #169.make lintandmake validateboth pass, includingcheck_docs.py.Once this merges,
docs/runbook-host-overridesholds nothing unique and can be deleted.🤖 Generated with Claude Code