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.

Sign up for GitHub

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

Jump to bottom

Unreliable backlog metrics #3723

Closed
5 tasks
ljoss17 opened this issue Dec 8, 2023 · 0 comments · Fixed by #3722
Closed
5 tasks

Unreliable backlog metrics #3723

ljoss17 opened this issue Dec 8, 2023 · 0 comments · Fixed by #3722
Assignees
@ljoss17
Labels
I: telemetry Internal: related to Telemetry & metrics

Comments

@ljoss17
Copy link
Contributor

ljoss17 commented Dec 8, 2023

Description

The backlog metrics are not always reliable as it requires the Hermes instance to observe the SendPacket and Acknowledgment in order to have precise values.

This could be improved by using the pending packets queried when clearing packets in order to update the backlog values.

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@ljoss17 ljoss17 added the I: telemetry Internal: related to Telemetry & metrics label Dec 8, 2023
@ljoss17 ljoss17 self-assigned this Dec 8, 2023
@github-project-automation github-project-automation bot moved this to 🩹 Triage in Hermes Dec 8, 2023
@ljoss17 ljoss17 mentioned this issue Dec 8, 2023
Merged
7 tasks
@github-project-automation github-project-automation bot moved this from 🩹 Triage to ✅ Done in Hermes Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ljoss17 ljoss17

Labels
I: telemetry Internal: related to Telemetry & metrics
Projects
Hermes
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Improve backlog metrics informalsystems/hermes
1 participant
@ljoss17