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

Export metrics to do with the Ethereum event oracle #506

Open
Tracked by #2023
james-chf opened this issue Sep 23, 2022 · 3 comments
Open
Tracked by #2023

Export metrics to do with the Ethereum event oracle #506

james-chf opened this issue Sep 23, 2022 · 3 comments
Assignees

Comments

@james-chf
Copy link
Contributor

james-chf commented Sep 23, 2022

We should export metrics via the Prometheus endpoint that could be used for Grafana dashboards and also for alerts. For the ledger oracle component specifically, it's important validator operators know if this is not working properly for whatever reason (e.g. their ledger is unable to reach the Ethereum RPC endpoint), as then they will stop voting on Ethereum events. We should expose at the very least

  • a current_block_height (of the Ethereum RPC endpoint) timeseries
  • rate, error, duration metrics
@Fraccaman
Copy link
Member

We can do this by directly listening to smart contract events/functions. I think Namada shouldn't have anything to do with monitoring but it would be nice to have some external tool to have some metrics

@james-chf
Copy link
Contributor Author

Have updated the issue desc, this is specifically metrics for the oracle component of namadan ledger i.e. the client inside the ledger that calls the Ethereum endpoint. If this component starts erroring for whatever reason, we may want to expose something that validators can alert on.

We do emit WARN logs that validators could grep on, though this would be less robust.

@sug0 sug0 added the post-mainnet Don't worry about this yet. label Aug 8, 2023
@sug0
Copy link
Collaborator

sug0 commented Oct 17, 2023

this might be useful https://lib.rs/crates/opentelemetry

phy-chain pushed a commit to phy-chain/namada that referenced this issue Mar 1, 2024
* feat(airdrop): updating terms of service

* feat: add updated terms of services checkbox

* fix: validation and missing space

* fix: remove one of the same variables

* fix: remove unused hint

* feat: add refresh info to the hints

---------

Co-authored-by: Mateusz Jasiuk <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: Todo
Development

No branches or pull requests

4 participants