docs(images): publish the four screenshots from a clean window - #117
Merged
Conversation
The captures were blocked on the window, not on the tooling. #62 fixed three collection faults on the afternoon of 2026-08-21, and `make screenshots` shoots `now-24h`, so every image taken that day pictured its own outage: a 14-hour hole in the host dashboard, 22 of 24 hours empty on the Docker one. A one-shot timer re-ran the capture at 2026-08-22 16:30, past the point where a full 24 hours of clean collection sat behind it. All four render with no gaps and pass the checklist in docs/images/README.md — RFC1918 addresses only, no MACs, no WAN IP, and a Container inventory panel showing only the three allowlisted fields after d1965f1. network-snmp.png and ups-power.png are replaced rather than left at their #115 versions. All four now come from one run, so they are comparable to each other and a gap in one would be visible against the rest. Two things visible in them are real and deliberately kept: IloBatteryCondition firing on shiva (#76), and the `renderer` container in two Docker legends, which exists only while a capture is running. The root README's screenshot block goes live, and the Makefile comment claiming five PNGs catches up with #108 dropping the Logs dashboard. Closes #12 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This was referenced Aug 24, 2026
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
The four dashboard screenshots go into
docs/images/, and the rootREADME.md'scommented screenshot block becomes a live
## Dashboardssection. No change tothe running lab — this is documentation only.
network-snmp.pngandups-power.pngare replaced rather than left at their#115 versions, so all four now come from one capture run.
Why
Closes #12. The last two boxes on that issue were "capture dashboard
screenshots" and "add them to the README", and the blocker was never the tooling
— it was the window.
make screenshotsshootsnow-24h, and #62 fixed threecollection faults on the afternoon of 2026-08-21, so every image taken that day
pictured its own outage: a 14-hour hole across the host dashboard, 22 of 24
hours empty on the Docker one.
A one-shot timer re-ran the capture at 2026-08-22 16:30, past the point where
a full clean day sat behind it.
Blast radius
secrets/*.sops.yamlDocumentation and one Makefile comment. Nothing deployed changes.
Verification
make validate— all checks pass, one skip (gitleaksnot installed locally;CI runs it).
Every image was read and checked against the pre-publish checklist in
docs/images/README.md:10.0.30.10,10.7.7.2,10.0.99.10)prometheus,shiva,mjolnir,neo), all already published indocs/hardware.mdcompose.yamlpath leak fixed in d1965f1 stays fixedTwo things visible in the images are real and deliberately kept rather than
re-shot around:
network-snmp.pngshowsIloBatteryConditionfiring onshiva. That is agenuine hardware fault tracked in IloBatteryCondition has been firing on shiva with no silence and no owner #76, and a picture of a stack with nothing
wrong would be the less honest one.
docker-containers.pngcarries arendererseries in two legends. That isthe
captureprofile container taking the screenshot, which is why theinventory below it lists six containers and not seven.
make validatepassesDeployed to the lab and confirmed working — the images are the lab
Docs updated (
docs/images/README.mdrewritten; the "why these two areabsent" table is gone, the window warning and the Logs-dashboard exclusion
stay)
Also fixes a one-word drift:
Makefile:260still said the capture shoots fivePNGs, which stopped being true when #108 dropped the Logs dashboard.
🤖 Generated with Claude Code