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

Issue-43 Add Node Health Prom Metrics per Chain #44

Merged

Conversation

Maxitosh
Copy link
Contributor

Github issue

#43

Description

This PR enhances the node health metrics by introducing detailed per-chain gauges.
It adds three new metrics:

  • Number of healthy nodes per chain.
  • Number of synced nodes per chain.
  • Number of nodes in timeout per chain.

These metrics will be exported in Prometheus format and available for scraping at the /metrics endpoint, providing improved visibility and monitoring capabilities.

Type of change

Please delete option that is not relevant.

  • New feature (non-breaking change which adds functionality)

@nodiesBlade nodiesBlade self-requested a review May 26, 2024 21:27
Copy link
Collaborator

@nodiesBlade nodiesBlade left a comment

Choose a reason for hiding this comment

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

Pretty clean and exactly how I would've done it. LGTM

@nodiesBlade nodiesBlade merged commit fd302f2 into pokt-network:main May 27, 2024
2 checks passed
@Maxitosh Maxitosh deleted the enhancement/43/node-health-metrics branch May 27, 2024 17:36
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