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

Feat/nodes stats 1160 #1169

Merged
merged 20 commits into from
Mar 24, 2023
Merged

Feat/nodes stats 1160 #1169

merged 20 commits into from
Mar 24, 2023

Conversation

Torresmorah
Copy link
Collaborator

@Torresmorah Torresmorah commented Mar 23, 2023

Add statistics from endpoint health checks

What does this PR do?

  • Resolve Show stats about Nodes behavior #1160
  • Avoid to request the health check for the same url when several nodes share the endpoints
  • Tracking the number of health checks performed per day and how many times the endpoints have responded.
  • Save the response time from Costa Rica and show the top 5 fastest endpoints.
  • New page with statistics of endpoints per producer.

Steps to test

  1. Run the project locally
  2. Go the new page API Endpoints Stats /endpoints-stats
  3. Check the content
  4. Search a producer with the Select component
  5. Check that the search show the endpoints data
  6. Go to endpoints list page
  7. Click the icon next to the producer's name
  8. Check that link to the new page

Screenshot

Full page

imagen

Fastest endpoints

imagen

List of endpoints

imagen

Response time graph from costa rica

imagen
imagen

CheckList

  • Follow proper Markdown format
  • The content is adequate
  • The content is available in both english and spanish
  • I Ran a spell check

@Torresmorah Torresmorah self-assigned this Mar 23, 2023
@Torresmorah Torresmorah added the WIP Work In Progress label Mar 23, 2023
@Torresmorah Torresmorah removed the WIP Work In Progress label Mar 23, 2023
Copy link
Collaborator

@xavier506 xavier506 left a comment

Choose a reason for hiding this comment

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

LGTM

@xavier506 xavier506 merged commit 1683611 into dev Mar 24, 2023
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.

Show stats about Nodes behavior
3 participants