-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Add unapproved count #7179
base: master
Are you sure you want to change the base?
Add unapproved count #7179
Conversation
e618352
to
34fda02
Compare
The CI pipeline was cancelled due to failure one of the required jobs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @kshitjj ! I assume this is not yet ready for review but I just added a comment to clarify what needs to be done here.
s.push(' '); | ||
} | ||
let mut unapproved: i32 = 0; | ||
for (i, bit) in bits.iter().enumerate() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This currently counts the unapproved relay chain blocks. I would still keep this information but add the count of unapproved parachain blocks under the unfinalized relay chain blocks. Also please add a prometheus metric so it is easy to monitor how approval voting performs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @sandreim,
I am working on the adding the promotheus metric, but my workstation keeps crashing while building polkadot from source.
Is there a way to run polkadot from source(without building it)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately you need to build it. What would help is to identify why it is crashing ? OOM ? or you could try building on a separate machine like a cloud VM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's crashing because it's running out of RAM, I am trying to increase the swap. If that doesn't work I will try the virtual machine option.
add an unapproved parachain counter, instead of showing bunch of one's.
solves: #7124