Skip to content

feat: SubscribeNodeGroupStatus RPC for per-node-group chain status - #166

Open
l0gun0v wants to merge 1 commit into
masterfrom
separation-nodecore
Open

feat: SubscribeNodeGroupStatus RPC for per-node-group chain status#166
l0gun0v wants to merge 1 commit into
masterfrom
separation-nodecore

Conversation

@l0gun0v

@l0gun0v l0gun0v commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Design: see the design doc (Notion) — problem statement, the group key, the full stream contract and the phase-2 plan live there.

Adds a dedicated RPC for per-node-group chain status:

rpc SubscribeNodeGroupStatus (SubscribeNodeGroupStatusRequest) returns (stream SubscribeChainStatusResponse) {}

message SubscribeNodeGroupStatusRequest {
}

message ChainDescription {
  ChainRef chain = 1;                   // unchanged
  repeated ChainEvent chain_event = 2;  // unchanged
  string node_group_id = 3;             // set on node-group streams
}
  • The new RPC reuses SubscribeChainStatusResponse, so consumers parse exactly the events they already know — the difference is that each ChainDescription describes one node group instead of the merged per-network view.
  • node_group_id is an opaque "<client_type>:<methods_hash8>" string (e.g. erigon:3fa9c21b); clients must not parse it. It is always empty on SubscribeChainStatus.
  • A group's ChainStatus going AVAIL_UNAVAILABLE is its removal signal.
  • SubscribeChainStatus is untouched: existing consumers see no change, the two views never mix in one stream, and rollback is "stop calling the new RPC".

Consumers: drpcorg/nodecore#316 (server), drpcorg/dproxy#2804 (aggregator) — both pin their submodule to this branch and need this merged first.

A dedicated RPC streams ChainDescription events per node group (tagged
with node_group_id); SubscribeChainStatus keeps the merged view and is
untouched.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@l0gun0v
l0gun0v force-pushed the separation-nodecore branch from eb2594d to cd4025c Compare August 19, 2026 09:38
@l0gun0v l0gun0v changed the title feat: Separation mode fields for SubscribeChainStatus feat: SubscribeNodeGroupStatus RPC for per-node-group chain status Aug 19, 2026
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.

2 participants