Skip to content
This repository has been archived by the owner on May 28, 2022. It is now read-only.

feature: add more logging to watcher #153

Open
luketchang opened this issue Feb 3, 2022 · 3 comments
Open

feature: add more logging to watcher #153

luketchang opened this issue Feb 3, 2022 · 3 comments
Assignees
@luketchang
Labels
agents needs more info This issue is not yet actionable observability rust 🦀

Comments

@luketchang
Copy link
Collaborator

luketchang commented Feb 3, 2022 •
edited
Loading

Just watcher:

  • Gauge: number of updates sent through UpdateHandler (number of updates checked)
  • Gauge: 0 or 1 binary for double update detected
  • Histogram: latency between Update event emitted and update sent through UpdateHandler (see ContractSync for example)

All agents that check home failure:

  • Gauge: 0 or 1 for improper update observed (failed home; can be for all agents)
  • Gauge: number of home failed checks (can be for all agents)
@luketchang luketchang self-assigned this Feb 3, 2022
@yourbuddyconner
Copy link
Contributor

@ltchang2019 what kind of logs do you want to add here?

@yourbuddyconner yourbuddyconner added the needs more info This issue is not yet actionable label Feb 8, 2022
@luketchang
Copy link
Collaborator Author

@yourbuddyconner updated

@luketchang
Copy link
Collaborator Author

Will turn this into broader per-agent agent_metrics field with a proper decl_metrics! macro

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@luketchang luketchang

Labels
agents needs more info This issue is not yet actionable observability rust 🦀
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@yourbuddyconner @luketchang