net: add summary-only node status wire contract - #750
Merged
Patrick W. Healy (phealy) merged 0 commit intoSep 17, 2026
Conversation
This was referenced Sep 16, 2026
This was referenced Sep 16, 2026
Patrick W. Healy (phealy)
force-pushed
the
phealy/lightweight-status-summary-contract
branch
from
September 17, 2026 14:46
4032b99 to
c9d1e2f
Compare
Patrick W. Healy (phealy)
merged commit Sep 17, 2026
c9d1e2f
into
phealy/lightweight-status-remove-matrix
Patrick W. Healy (phealy)
force-pushed
the
phealy/lightweight-status-remove-matrix
branch
from
September 17, 2026 14:46
bd2c6e1 to
0c18ea6
Compare
Patrick W. Healy (phealy)
deleted the
phealy/lightweight-status-summary-contract
branch
September 17, 2026 14:46
Patrick W. Healy (phealy)
restored the
phealy/lightweight-status-summary-contract
branch
September 17, 2026 14:48
Contributor
Author
|
This PR was part of the mistaken churn from unnecessarily splitting the lightweight-status work. Its foundation changes are preserved and now reviewed in #812 within the approved 12-PR draft stack. During consolidation, GitHub automatically marked this PR merged when its rewritten head became an ancestor of its previous feature-branch base, before I retargeted the PR. That ordering was my mistake. I did not perform a PR merge, and nothing was merged or pushed into main. GitHub does not allow reopening a merged PR, so only this review slot needed a replacement PR; the final stack still contains twelve PRs. The automatically deleted head branch has been restored. |
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.
Scope
Stacked on #748. Adds shared JSON/protobuf NodeStatusOverview with explicit peer health and observed route counts/mismatch, separate from heavy peer/route/BPF detail arrays. Protobuf envelope gains additive summary field. Existing publication behavior is unchanged.
366 changed LOC, including generated bindings and contract coverage.
Validation
Shared status tests passed after stacking. Pinned protobuf generation, scoped formatting/lint, race tests and binary builds also passed during implementation. Summary defaults will activate only after transport and consumer migrations.
Lightweight status draft stack
Base: #745. Each layer targets the preceding branch and remains draft.
#746 -> #747 -> #748 -> #750 -> #751 -> #752 -> #753 -> #754 -> #755 -> #756 -> #757 -> #758 -> #759 -> #760 -> #761 -> #762 -> #763 -> #764 -> #765 -> #766 -> #767 -> #768 -> #769 -> #770 -> #771 -> #772 -> #773 -> #774 -> #775 -> #776 -> #777 -> #778 -> #779 -> #780 -> #781 -> #782 -> #783 -> #784 -> #785 -> #786 -> #787 -> #788 -> #789 -> #790 -> #791 -> #792 -> #793 -> #794 -> #795 -> #796 -> #797 -> #798 -> #799