#20763 closed enhancement (fixed)

Add shared random to consensus health

Reported by: teor Owned by: tom
Priority: Medium Milestone:
Component: Metrics/Consensus Health Version:
Severity: Normal Keywords:
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Consensus method 23 introduced the shared random protocol, which has lines like these in the votes:

shared-rand-participate
shared-rand-commit 1 sha3-256 188AB2EB56A3E6E097386E02DE799B14FC515C82 AAAAAFg2L
YBcpntRXkieDarbOn7IXXJuzNNjSyjPvUGWGiWQuiHI7w== AAAAAFg2LYBO0rQDcdSHsKm74LNZmyZk
4leZENT8iVV1dsIsrOwKdQ==
shared-rand-commit 1 sha3-256 4CAEC248004A0DC6CE86EBD5F608C9B05500C70C AAAAAFg2L
YDLqk+EH1tqita97dnlxpfBWq6MFqFR5RA+SRXIrjoiWA== AAAAAFg2LYCc0lvA9/n4lf5ZPhM9JiRk
j1l+/P9b7sAtfvbE3Zjbuw==
(one commit with an optional reveal per authority)
shared-rand-previous-value 8 SaljvGkIUuVY5e4XRiXH85mcS71/7FMbHvaZxdx6CYU=
shared-rand-current-value 8 Nb3qyVtVOkeUG3o2c0jYZN59m+w45O0VzwGd33rp0rg=

And lines like these in the consensus:

shared-rand-previous-value 8 SaljvGkIUuVY5e4XRiXH85mcS71/7FMbHvaZxdx6CYU=
shared-rand-current-value 8 Nb3qyVtVOkeUG3o2c0jYZN59m+w45O0VzwGd33rp0rg=

It might be useful to have these in Consensus Health, particularly when the authorities on the public network are running 0.2.9 and later.

It would be useful to know, for each authority's commit and reveal in its own vote, how many other authorities have a vote with that authority's commit and reveal (and if any have mismatching values, that's a red flag).

It would be useful to know, for the voted shared random current and previous values, how many other authorities have a the same, different, or missing voted current and previous value.

Child Tickets

Change History (6)

comment:1 Changed 11 months ago by tom

Opened #20789 which will block this.

comment:2 Changed 10 months ago by teor

stem has shared random support now, so #20789 is no longer a blocker.

comment:3 Changed 10 months ago by atagar

s/now/five months ago

This isn't a new change. I added shared randomness parsing soon as it was added to the spec.

comment:4 Changed 10 months ago by tom

My plan is to implement #20917, #20763, and #20762 (and trial #20403) in January, but I won't be able to do it before then.

comment:5 Changed 10 months ago by tom

Opened #21102 which blocks this

comment:6 Changed 10 months ago by tom

Resolution: fixed
Status: newclosed

This is completed.

Note: See TracTickets for help on using tickets.