schema+wire: SSH get_config.running from show running-config script (#324) - #388
Draft
AdamRickards wants to merge 1 commit into
Draft
AdamRickards wants to merge 1 commit into
AdamRickards wants to merge 1 commit into
Conversation
…324) running: new base wire/config.yaml (synthetic hm2configrunning; no MIB object backs a config-text dump) + wire/ssh/config.yaml overlay sourced from "show running-config script". parser: none is explicit and required: SSH.yaml's protocol-level default parser is dot_keys, which returns {} against the raw text. startup: deliberately unwired. "show startup-config" is rejected on the tested firmware, and the saved config can only be read by a mutating copy into running, which a read-only getter must not do. MOPS/SNMP unchanged (running stays empty). Offline (tested): SSH sends the command and returns the text; MOPS offline unchanged. Live sidecar proof: untested (pending). Open items: receipt redaction, parity/gold handling for an SSH-only field. Schema + wire YAML only. Draft for sidecar live proof; do not merge. Related #324
Owner
Author
|
Sidecar live proof —
Result: fail (ssh untested for content: call did not finish inside the call budget on the tested device; output was still streaming, no prompt returned). Receipt safety: on timeout, the receipt's Draft; do not merge. |
This was referenced Sep 29, 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
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.
Draft for sidecar live proof; do not merge.
Wires
get_config.runningto SSHshow running-config scriptvia newwire/config.yaml(synthetichm2configrunning) +wire/ssh/config.yamlwith explicitparser: none(SSH default parser isdot_keys).startupstays unwired: no read-only CLI exists on the tested firmware. MOPS/SNMP unchanged (empty).Offline (tested): SSH sends the command and returns the text; MOPS offline
runningstays empty; schema shape unchanged.Live (untested, pending): sidecar
get_config.readon this head vs main.Open before any merge:
Receipt redaction: the raw running config would appear in sidecar
raw/cliand in parity_diffs prefixes; it may contain credential material.Parity/gold: SSH-only non-empty
runningvs empty MOPS/SNMP and gold floor.pytest tests/: pass
scripts/ci_offline.sh: pass
generate_floors_board.py --check: pass
Related #324