docs: refresh the guides against the code - #65
Merged
Merged
Conversation
- 41 phases (not 39) everywhere; PHASES.md gains the up flags, the down order with metrics-server, the phases without a down step, and correct preflight, forge-register, ebs-csi, k8s-foundation and activation-poll descriptions (9-minute cap) - ARCHITECTURE.md: SDK-only commitment covers client-go and the Helm SDK, cluster-side repairs section, honest state-recovery claim, full package table, manifestVersion in the example - README/COMMANDS/AGENTS/CLAUDE: bnk heal and doctor rows, 10-repair registry, awsbnkctl-test Job namespace, missing flags and subcommands, VERSION=2.3.0, terse wording - RELEASE.md rewritten: staging→main flow, squash titles, workflow_dispatch rebuild, make release gates, the tools images that are actually published - FORGE_INTEGRATION, UPGRADE-2.4, SCENARIOS, BGP: corrected env precedence, purge caveat, link lifecycle, JSON fields, assertions as the code makes them, no dated lab provenance - ALIGNMENT.md deleted; Local Zones report reduced to a results table; upstream issue gets a status line - code help text: activation poll is 9 min, test throughput is implemented, k_aliases comment no longer claims a host-side exec command Co-Authored-By: Claude Fable 5.1 <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.
What
Three audits of the documentation against the code, applied.
upflags, the down order includingmetrics-server, the phases with no down step, and describespreflight,forge-register,ebs-csi-hugepages,k8s-foundationandactivation-pollas the code behaves (9-minute poll cap, no EULA check, no gp3 StorageClass).bnk heal(10 repairs after feat(heal): test-namespace repair; k8s backend runs Jobs without the ops pod #64), thedoctor --backend k8srows it feeds, and theawsbnkctl-testnamespacetest --backend k8sJobs run in. CONTRIBUTING gains "Adding a Heal Repair" and the missing CI jobs andgosecgate.bnk heal/upgrade/migrate-2.4/mcp-sessionflags,forge scan/telemetry,benchmark ingest,targets scan,test dns/throughput/trafficflags).AWSBNKCTL_K8S_LONG_LIVEDremoved.workflow_dispatchrebuild,make releasegates, and the tools images that are actually published.AWSBNKCTL_FORGE_PROJECTpurge caveat, link lifecycle commands, the full-o jsonfield list, scenario assertions stated as the code makes them, no dated lab provenance.docs/ALIGNMENT.mddeleted (unreferenced cross-repo comparison). The Local Zones report is now a results table. The upstream issue has a status line.--skip-activation-pollsays 9 min,test throughputno longer says "not yet implemented",k_aliases.gocomment no longer claims a host-sideexeccommand.Depends on #64 for the tenth repair (
test-namespace); merge #64 first.🤖 Generated with Claude Code