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

Add unapproved count #7179

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kshitjj
Copy link
Contributor

@kshitjj kshitjj commented May 4, 2023

add an unapproved parachain counter, instead of showing bunch of one's.

solves: #7124

@kshitjj kshitjj force-pushed the add-unapproved-count branch from e618352 to 34fda02 Compare May 5, 2023 00:35
@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/2784384

Copy link
Contributor

@sandreim sandreim left a 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() {
Copy link
Contributor

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.

Copy link
Contributor Author

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)?

Copy link
Contributor

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.

Copy link
Contributor Author

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.

@sandreim sandreim added the A3-in_progress Pull request is in progress. No review needed at this stage. label May 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A3-in_progress Pull request is in progress. No review needed at this stage.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants