Skip to content

Spawn GRANDPA observer task for non-validator nodes - #73

Merged
motoZ-crypto merged 1 commit into
masterfrom
motoz/grandpa-observer
May 1, 2026
Merged

Spawn GRANDPA observer task for non-validator nodes#73
motoZ-crypto merged 1 commit into
masterfrom
motoz/grandpa-observer

Conversation

@motoZ-crypto

Copy link
Copy Markdown
Collaborator

The non-authority branch of new_full() was dropping the grandpa notification stream returned from the grandpa block import setup. That immediately closed the corresponding essential task and any follow-up grandpa peer dialing failed, leaving full nodes stuck at 0 peers and unable to follow finality.

Mirror the authority branch but pass keystore=None so non-authority nodes participate as observers only (no voting, no equivocation reporting).

The non-authority branch of new_full() was dropping the grandpa
notification stream returned from the grandpa block import setup.
That immediately closed the corresponding essential task and any
follow-up grandpa peer dialing failed, leaving full nodes stuck at
0 peers and unable to follow finality.

Mirror the authority branch but pass keystore=None so non-authority
nodes participate as observers only (no voting, no equivocation
reporting).
@motoZ-crypto motoZ-crypto added this to the Alpha-0.1 milestone May 1, 2026
@motoZ-crypto motoZ-crypto self-assigned this May 1, 2026
@motoZ-crypto motoZ-crypto added T-bug Something isn't working A-grandpa GRANDPA finality and fork-choice labels May 1, 2026
@motoZ-crypto
motoZ-crypto marked this pull request as ready for review May 1, 2026 08:10
@motoZ-crypto
motoZ-crypto merged commit e49563a into master May 1, 2026
1 check passed
@motoZ-crypto
motoZ-crypto deleted the motoz/grandpa-observer branch July 5, 2026 04:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-grandpa GRANDPA finality and fork-choice T-bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant