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

Prometheus: remove id label on hostd_consensus_state_index #312

Closed
wants to merge 1 commit into from

Conversation

bustedware
Copy link
Contributor

The id label on hostd_consensus_state_index is breaking continuity with block height panels and was introduced during the refactor. Removing the label will restore continuity.

Screenshot 2024-02-21 173716

@n8maninger
Copy link
Member

id is an important piece of consensus information. You can adjust the query to fix it rather than removing the label.

Either

min (hostd_consensus_state_index{})

Screenshot 2024-02-21 at 2 54 57 PM

If you want to separate multiple instances:

min by (name) (hostd_consensus_state_index{})

Screenshot 2024-02-21 at 2 54 34 PM

@n8maninger n8maninger closed this Feb 21, 2024
@bustedware
Copy link
Contributor Author

@n8maninger if you're just going to remove it, why have it collected? personally, i don't think, I would ever look at that value. not dismissing its important though.

@n8maninger
Copy link
Member

@n8maninger if you're just going to remove it, why have it collected? personally, i don't think, I would ever look at that value. not dismissing its important though.

We use it on our dashboard

@bustedware
Copy link
Contributor Author

@n8maninger can I ask how do you use it in your dashboard? it will help if I understand so I can add to siagrafana. thank you

@bustedware
Copy link
Contributor Author

@n8maninger something else to consider:

https://last9.io/blog/how-to-manage-high-cardinality-metrics-in-prometheus/

High-label cardinality means that a metric has many unique label combinations. This can have implications for the performance and scalability of Prometheus, as each distinct label combination creates a separate time series in the system. A high cardinality can increase memory usage, CPU usage, longer query times, and higher storage requirements

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.

None yet

2 participants