Metrics MVP - Githubissues

cartesi / rollups

Cartesi Rollups

30 stars 12 forks source link

Metrics MVP #17

Closed renan061 closed 1 year ago

renan061 commented 1 year ago

📚 Context

We need to improve the observability of the Cartesi Node by exposing domain-driven metrics.

✔️ Solution

We are going to use Prometheus to add, store, and query metrics in the rollups' services.

📈 Subtasks

[x] Metrics library wrapping over Prometheus (included in the dispatcher issue)
[x] cartesi/rollups#18
[x] cartesi/rollups-node#1
[x] cartesi/rollups-node#2
[x] cartesi/rollups-node#3
[x] cartesi/rollups-node#4
[x] cartesi/rollups#102

gligneul commented 1 year ago

Regarding the first item (Metrics library wrapping over Prometheus), I prefer to keep the wrapper slim as possible. It might be a good idea to implement the metrics in the dispatcher first without a wrapper and then create it when we realize what code can be reused.

gligneul commented 1 year ago

We reached the MVP with the dispatcher metrics. I will close the epic issue and leave the other issues open.