Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Check that the authority-discovery module discovers all (or almost all) validators #6615

Closed
Tracked by #989
tomaka opened this issue Jul 9, 2020 · 4 comments
Closed
Tracked by #989

Comments

@tomaka
Copy link
Contributor

tomaka commented Jul 9, 2020

#6614 adds a new metric that reports the number of validators known to the authority-discovery module.

This issue is a reminder that, after this PR has been merged and deployed on our nodes, we should check if it reports a correct value.

@tomaka
Copy link
Contributor Author

tomaka commented Jul 10, 2020

The metric is now on 8 nodes, and they seem to be somewhere between 14 and 16 known validators.

Screenshot from 2020-07-10 18-38-15

@mxinden
Copy link
Contributor

mxinden commented Jul 10, 2020

As of today the authority-discovery module is still disabled by default on Polkadot: https://github.com/paritytech/polkadot/blob/master/cli/src/cli.rs#L62-L72. I would still expect a node to find more validators as at least Parity and web3 are enabling this feature.

@tomaka
Copy link
Contributor Author

tomaka commented Jul 22, 2020

Everything seems to work fine on Kusama!

Screenshot from 2020-07-22 13-06-19

The number is now almost at 200, and nothing significant has happened.

Polkadot validators are still in the process of upgrading, but everything seems to work well too.

@tomaka
Copy link
Contributor Author

tomaka commented Jul 22, 2020

Closing this as done 🎉

@tomaka tomaka closed this as completed Jul 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants