Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a basic mempool storage Grafana dashboard #2866

Merged
merged 2 commits into from
Oct 13, 2021
Merged

Conversation

teor2345
Copy link
Contributor

@teor2345 teor2345 commented Oct 13, 2021

Motivation

I needed to see if the mempool was active to test PR #2862.

This implements part of #2627.

Solution

Create a mempool dashboard that shows:

  • storage transaction count
  • storage serialized transaction bytes
  • syncer downloaded blocks (as a proxy for recent sync lengths, which don't have a usable metric)
  • state committed blocks

Review

@conradoplg is working on more metrics.

Reviewer Checklist

  • Dashboard makes sense

@teor2345 teor2345 added C-enhancement Category: This is an improvement P-Low labels Oct 13, 2021
@teor2345 teor2345 requested a review from conradoplg October 13, 2021 00:15
@teor2345 teor2345 self-assigned this Oct 13, 2021
@teor2345
Copy link
Contributor Author

Here's what it looks like:
Screen Shot 2021-10-13 at 10 15 54

Copy link
Collaborator

@conradoplg conradoplg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@conradoplg conradoplg enabled auto-merge (squash) October 13, 2021 13:58
@conradoplg conradoplg merged commit be18531 into main Oct 13, 2021
@conradoplg conradoplg deleted the mempool-grafana branch October 13, 2021 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: This is an improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants