Skip to content

net (10/12): Migrate CLI summaries, diagnostics and current node info - #767

Draft
Patrick W. Healy (phealy) wants to merge 1 commit into
phealy/lightweight-status-dashboard-controlsfrom
phealy/lightweight-status-cli-transport
Draft

Patrick W. Healy (phealy) wants to merge 1 commit into
phealy/lightweight-status-dashboard-controlsfrom
phealy/lightweight-status-cli-transport

Conversation

@phealy

@phealy Patrick W. Healy (phealy) commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Consolidated lightweight-status stack: 10/12

Based on #766. This existing draft PR is reused for the user-approved consolidation, replacing unnecessarily fragmented PRs rather than creating another set.

Scope

Complete CLI summary list/watch, explicit named-node show/refresh, authenticated aggregated diagnostic transport, polling, correlation, and expiry. Folds code/tests from independent fixes 1643e7d49723173eedb84b89eea997a248ecb7d3 and 55c62ff8d60d9cdb01188524d59f1da767d39501: controller status-json preserves raw summary fields/unknown metadata and projects legacy nodes without diagnostic arrays; node show INFO uses current overview health/metadata without changing raw diagnostic content or TTL. Both fixes' documentation belongs to layer 12. Includes final decoder allocation fix 31878a7c5b08c151a26e119ec399467a7ed916c7, which prevents summary metadata from retaining the full legacy diagnostic object.

Original work represented: #767, #768, #769, #785, #795. Cross-cutting originals may also be represented by adjacent functional layers.

Size

+1,790 / -172 = 1,962 non-generated changed lines, measured against the immediate parent. Generated changes: 0 lines, excluded from the approved 2,000-line ceiling. About 1,000 is a guideline, not a requirement to split coherent functionality.

Validation

  • Scoped make fmt (tmp/layer10-fmt.log, tmp/layer10-cli-fix-fmt.log, tmp/layer10-info-fix-fmt.log)
  • go test ./cmd/kubectl-unbounded/app/net (tmp/layer10-net-tests.log)
  • go test -race ./cmd/kubectl-unbounded/app/net after each independent fix integration (tmp/layer10-cli-fix-race.log, tmp/layer10-info-fix-race.log)
  • go build -o bin/kubectl-unbounded ./cmd/kubectl-unbounded
  • Final decoder fix scoped make fmt and CLI race tests (tmp/layer10-allocation-fix-fmt.log, tmp/layer10-allocation-fix-race.log)

The final stack preserves the validated implementation and all three CLI corrections exactly; its only additional tracked difference is the requested AGENTS.md approval/size policy. No PR is merged into main.

Approved 12-PR draft stack

Base: #745. Each PR targets its preceding layer.

#746 -> #812 -> #760 -> #762 -> #755 -> #757 -> #774 -> #764 -> #766 -> #767 -> #788 -> #748

Validation caveat: broader CLI install tests require rendered operator manifests absent from the isolated worktree. The changed net CLI package tests/race suite and complete plugin build pass.

This was referenced Sep 16, 2026
This was referenced Sep 16, 2026
@phealy
Patrick W. Healy (phealy) force-pushed the phealy/lightweight-status-cli-transport branch from c1f9fbc to 00af906 Compare September 17, 2026 14:46
@phealy Patrick W. Healy (phealy) changed the title kubectl: prepare authenticated per-node diagnostic transport net (10/12): Migrate CLI summaries, diagnostics and current node info Sep 17, 2026
@phealy
Patrick W. Healy (phealy) added this pull request to stack #813 September 17, 2026 14:59
…stics

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: d2243398-6c36-4c3d-969e-7ed7bfb5b459
@phealy
Patrick W. Healy (phealy) force-pushed the phealy/lightweight-status-cli-transport branch from 00af906 to e714ec0 Compare September 17, 2026 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant