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

Update Grafana dashboard in Monitor node page #6161

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

Imod7
Copy link
Contributor

@Imod7 Imod7 commented Aug 20, 2024

Description

Updated the Grafana dashboard used in the "Installing Grafana" section of the "Monitor your node" wiki page.

Issue

The reason of this update is that I tried to use the My Polkadot Metrics dashboard but most of the metrics are outdated so I do not get any data on the dashboard.

Screenshot 2024-08-20 at 13 52 36

Some examples of outdated metrics are the following (while syncing the Polkadot chain with ./target/release/polkadot --chain polkadot --prometheus-external) :

  • all the metrics in the "My Polkadot Metrics" dashboard are prefixed with polkadot while the node exposes metrics with the prefix substrate, e.g.:
    • metric in dashboard polkadot_tasks_polling_started_total -> metric exposed from node substrate_tasks_polling_started_total
    • polkadot_sub_libp2p_peers_count -> substrate_sub_libp2p_peers_count
  • some metrics like polkadot_load_avg, polkadot_cpu_usage_percentage or polkadot_memory_usage_bytes are not found under the list of metrics that the node exposes (http://localhost:9615/metrics). Even when I change the prefix from polkadot to substrate.

Proposed Solution

Use the dashboard Substrate Node Metrics which is an updated version of the "My Polkadot Metrics" dashboard wtih some extra panels. When imported it will show data like in the screenshot below:

6-dashboard-metric

Copy link
Contributor

@CrackTheCode016 CrackTheCode016 left a comment

Choose a reason for hiding this comment

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

This LGTM, thank you for catching this!

Many of our Maintain docs probably need a rehaul..

@CrackTheCode016 CrackTheCode016 merged commit c714ec9 into w3f:master Aug 20, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants