docs(roadmap): track #309 and #311, and record #292 as done - #313
Merged
Conversation
#292 shipped, so its entry moves out of Monitoring and into Done rather than sitting above the work it describes as though it were pending — which is the failure the #97 entry says this section exists to prevent. The Done entry records the part no plan predicted: deploying the check found that sysDescr is not morpheus-only, because the ilo module walks it too, and the runbook comparison was selecting by parse order rather than by device. It was correct only because the iLO firmware has two version components rather than three. In its place, the two things #292 did not cover, tracked rather than inherited: #309 shiva — parsing. It has both halves of a checkable claim already, and is unchecked only by a scoping line, but os_key misreads both sides and the iLO's "4" is a generation, not a firmware version. #311 Saruman — collection. Nothing on the wire carries the number the documents claim. The textfile collector is already there; the hard part is that the monitoring host cannot reach VLAN 30, so whatever writes it has to live on a host this repo reaches through a person. Co-Authored-By: Claude Opus 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.
Adds #309 and
#311 under Monitoring, and
moves #292 into Done.
Why #292 moves at the same time
It shipped — #302 and
#306 — and its entry was still
sitting in Monitoring describing the work as pending. Leaving it there while
adding its two follow-ups directly above would have been the failure the #97
entry says this section exists to prevent: "The entry outlived the work by a
day."
The Done entry records the part no plan predicted. Deploying the check found
that
sysDescris notmorpheus-only — theilomodule walks it too, so thequery returns two series with
shivafirst — and the runbook comparison wasselecting by parse order rather than by device. It was correct only because the
iLO firmware is
2.82, two components rather than three. An iLO numbered2.82.1would have reported the BMC's firmware under the firewall's name.The two that replace it
shiva— parsing. It already has both halves of a checkable claim:network.mdgives itiLO 2.82and itssysDescris already scraped. Theonly thing keeping it unchecked is a scoping line, but it is not a one-line
change, because
os_keymisreads both sides and the iLO's4is ageneration, not a firmware version.
Saruman— collection. Nothing on the wire carries the number thedocuments claim, so
Proxmox VE 9is currently unfalsifiable. The textfilecollector is already enabled and provisioned; the awkward half is that the
monitoring host cannot reach VLAN 30, so whatever writes the metric has to
live on a host this repository can only reach through a person.
Both entries say why they are not trivial, since both look trivial from the
outside — which is the specific way each would get picked up and abandoned.
Docs only.
make lintandmake check-docspass.🤖 Generated with Claude Code