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

[OBSERVABILITY][SENTINEL] Setup Sentinel balance alerts based on Prometheus metrics #287

Open
2 tasks
philipandersson opened this issue Oct 13, 2023 · 3 comments
Assignees
Labels
Project: SENTINEL Team: Protocol Protocol Core, Sentinel, Peripherals, Protocol Infrastructure Tools & DevOps Type: DevOps Improvements to the processes part of DevOps Type: Enhancement New feature or request

Comments

@philipandersson
Copy link

Acceptance criteria

  • Establish balance thresholds for each network
  • Use a alert-mananger to route alets to #notif-solvency-mainnets when balance falls under threshold
@philipandersson philipandersson added Project: SENTINEL Team: Protocol Protocol Core, Sentinel, Peripherals, Protocol Infrastructure Tools & DevOps Type: DevOps Improvements to the processes part of DevOps Type: Enhancement New feature or request labels Oct 13, 2023
@philipandersson
Copy link
Author

@d10r we would need your input on balance threshold here, thanks

@d10r
Copy link
Collaborator

d10r commented Dec 18, 2023

I think we should avoid having to provide thresholds denominated in native coins.
If we instead denominate them in "gas amount" or "nr of simple txs" (a simple tx being 21000 gas), with a bit of added logic the native coin amount can be auto-derived. Example implementation of such a logic: https://github.com/d10r/fund-account/blob/master/fund-accounts.js

The mechanism for adding/changing thresholds should be straightforward and not require assistance of an ops guy. E.g. a web UI, or change a json file in a git repo, or interact with a chatbot in slack.

@philipandersson
Copy link
Author

@mmd-afegbua Is it doable to denominate in gas token?

This would be configurable in the alertsmanager config.

@hellwolf hellwolf transferred this issue from another repository Mar 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Project: SENTINEL Team: Protocol Protocol Core, Sentinel, Peripherals, Protocol Infrastructure Tools & DevOps Type: DevOps Improvements to the processes part of DevOps Type: Enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants