Correct what a docs walkthrough measured, and show the fabric's state - #43
Merged
Conversation
Five things found by following the pages step by step rather than by reading them. The cEOS image ID is per-import: `docker import` builds a fresh layer every time, so the ID in `lab.md` could never match a reader's. It is a placeholder now; the repository, tag and size are what should match. The scenarios overview applied `router` and removed `jsonrpc`, so following both blocks in order deleted resources that were never applied. Both use one scenario now, and the page says outright that the directory name is the only thing that changes. `router.md` opened with "the scenarios below this one", which the navigation puts above it. The contrast is about `Router` and does not need them. `fabric.md` said the whole scenario is one patch. The patch is what this repository adds; the scenario is also the render and the push, and the page never showed either. It now walks the `Fabric` and the eight `Device`s, the deviceCount and pyavd's verdict, and `status.push` on a device that was configured -- and gives the command whose output the running-config excerpt came from, which was quoted without one. 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.
Five things found by following
docs/step by step on a live lab rather thanby reading the pages.
lab.mdprinted an image ID nobody can match.docker importbuilds afresh layer every time, so the same cEOS tarball produces a different ID on
every machine and on every re-import. It is a placeholder now; the
repository, tag and size are what a reader should compare.
The scenarios overview applied one scenario and removed another. The
apply block used
router, the teardown block usedjsonrpc, so followingboth in order deletes resources that were never created:
Both blocks now use one scenario, and the page says outright that the
directory name is the only thing that changes.
router.mdsaid "the scenarios below this one", which the navigationputs above it. The contrast is about what
Routerdoes and does not needthem named.
fabric.mdsaid the whole scenario is one patch. The patch is what thisrepository adds; the scenario is also the render and the push, and the page
showed neither. It now walks the
Fabricand the eightDevices, thedeviceCountand pyavd's verdict, andstatus.pushon a device that wasactually configured.
fabric.mdquoted a device without a command. Every other page gives thecommand whose output it prints. This one showed the leaf's configuration with
no way to reproduce it;
show running-configis where those lines come from,scattered through it rather than contiguous.
Everything added here was measured against a running lab today, not carried
over from an earlier session.
mkdocs build --strictpasses.Not addressed here, because it is an upstream change:
kubectl -n avd get netclablists nothing, since function-avd'sFabricandDeviceXRDs carry nocategories. The explicit kinds used above stay correct either way.🤖 Generated with Claude Code