Skip to content

net (11/12): Unify controller reads and expire viewer diagnostics - #788

Draft
Patrick W. Healy (phealy) wants to merge 1 commit into
phealy/lightweight-status-cli-transportfrom
phealy/lightweight-status-thin-cluster
Draft

Patrick W. Healy (phealy) wants to merge 1 commit into
phealy/lightweight-status-cli-transportfrom
phealy/lightweight-status-thin-cluster

Conversation

@phealy

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

Copy link
Copy Markdown
Contributor

Consolidated lightweight-status stack: 11/12

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

Scope

Cluster overview retention and summary-only pulls, unified legacy per-node reads, expiring viewer detail writes and lifecycle observers. Obsolete per-node broadcast subscription calls are removed with their owner; routine full broadcast history remains until layer 12.

Original work represented: #788, #789, #791. Cross-cutting originals may also be represented by adjacent functional layers.

Size

+1,202 / -283 = 1,485 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 with serialized shared linter lock (tmp/layer11-fmt.log)
  • go test ./cmd/unbounded-net-controller (tmp/layer11-tests.log)
  • go test ./cmd/unbounded-net-controller ./cmd/kubectl-unbounded/app/net after each CLI restack (tmp/layer11-cli-fix-tests.log, tmp/layer11-info-fix-tests.log)
  • Controller and CLI tests after final decoder restack (tmp/layer11-allocation-fix-tests.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

This was referenced Sep 16, 2026
This was referenced Sep 16, 2026
@phealy
Patrick W. Healy (phealy) force-pushed the phealy/lightweight-status-thin-cluster branch from 12c96b0 to 1ba7ee0 Compare September 17, 2026 14:46
@phealy Patrick W. Healy (phealy) changed the title net controller: retain cluster overviews and pull summaries only net (11/12): Unify controller reads and expire viewer diagnostics Sep 17, 2026
@phealy
Patrick W. Healy (phealy) changed the base branch from phealy/lightweight-status-thin-node-cache to phealy/lightweight-status-cli-transport September 17, 2026 14:49
@phealy
Patrick W. Healy (phealy) added this pull request to stack #813 September 17, 2026 14:59
@phealy
Patrick W. Healy (phealy) force-pushed the phealy/lightweight-status-thin-cluster branch from 1ba7ee0 to 4e7b6be Compare September 17, 2026 21:01
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-thin-cluster branch from 4e7b6be to ce8d264 Compare September 18, 2026 14:31
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