refactor(diagnostics): emit system.ntp_synchronized + top-level pi_throttle#102
Merged
d4rken merged 2 commits intoMay 19, 2026
Merged
Conversation
…rottle Splits the misnamed pi_health wire block. NTP-sync is a universal host signal and moves into system.ntp_synchronized; the eight vcgencmd throttle bits become a flat top-level pi_throttle block, absent on non-Pi feeders. collect_pi_ntp_sync renames to collect_ntp_sync.
…=false Stub vcgencmd alongside timedatectl in the no-probes test so a Pi-based dev box can't leak vcgencmd output through the stub gap. Add an exact-key-set assertion on pi_throttle (the server sanitizer rejects partial blocks). Add a 'no' NTP case so 'false' is pinned through the prune pass. Tone down the collect_ntp_sync comment to match runtime behaviour in containers / stripped images.
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
1 participant
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.
Renames the misnamed
pi_healthblock on the diagnostics wire. NTP-sync is a universal host signal — every Linux feeder has it viatimedatectl, not just Raspberry Pis. It moves intosystem.ntp_synchronizedalongsideuptime_seconds/ CPU / memory / disk. The eightvcgencmd get_throttledbits become a top-levelpi_throttleblock, flat and absent on every non-Pi feeder rather than present-with-null.A non-Pi Linux feeder now sends nothing Pi-shaped at all. On a Pi,
pi_throttleis a top-level sibling ofsystem, nothrottle:nesting.Pre-launch, breaking-by-design: no compat shim, no schema-version bump. The server-side accept landed in airplanes-live/website#72 first.