Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix: Better monitoring experience #33

Merged
merged 8 commits into from
Jul 18, 2024
Merged

fix: Better monitoring experience #33

merged 8 commits into from
Jul 18, 2024

Conversation

akhercha
Copy link
Member

@akhercha akhercha commented Jul 17, 2024 •
edited
Loading

Resolves: #26

Introduced changes

Added

  • custom alerts for long-tail assets
  • verify_indexer_sync utility

Updated

  • text alerts are more understandable
  • template is detected
  • template is working..?

New metrics screen: (only long tail assets set are NSTR & ZEND here)
image

🤔 Follow-up: monitoring should compare future entries that have the same expiry only?

@akhercha akhercha changed the title fix(better_monitoring): Renamed some errors fix: Better monitoring experience Jul 17, 2024
@akhercha akhercha marked this pull request as ready for review July 17, 2024 20:28
@akhercha akhercha self-assigned this Jul 17, 2024
@EvolveArt EvolveArt self-requested a review July 18, 2024 12:18
@EvolveArt EvolveArt merged commit c7b78dd into main Jul 18, 2024
4 checks passed
@EvolveArt EvolveArt deleted the fix/better_monitoring branch July 18, 2024 15:38
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@EvolveArt EvolveArt EvolveArt approved these changes

Assignees

@akhercha akhercha

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

dev: long tail assets custom alert
2 participants
@akhercha @EvolveArt