Skip to content
This repository has been archived by the owner on Apr 18, 2024. It is now read-only.

Metrics / logging #2

Closed
willscott opened this issue Jan 31, 2023 · 3 comments
Closed

Metrics / logging #2

willscott opened this issue Jan 31, 2023 · 3 comments

Comments

@willscott
Copy link
Contributor

there is not yet logging.

the pool should track latency and bandwidth of requests as metrics.
logs should be emitted for each successful & failed request

@lidel
Copy link
Contributor

lidel commented Feb 6, 2023

FYSA started mapping things we want to measure in bifrost-gateway, Saturn things are (2) and details TBD.

Some things need to be backend-independent, but we do want Saturn-specific metrics dded to that list and tracked/registered if caboose provides them 👍

willscott added a commit that referenced this issue Feb 6, 2023
@willscott
Copy link
Contributor Author

#7 should now capture the 3 metrics specified in (2).

let me know if the exposed CabooseMetrics prometheus.Registry is a reasonable interface for the package, or if it's easier for the metrics to be registered against the implicit global registry.

@willscott
Copy link
Contributor Author

(it was #8 that captured the 3 metrics specified)

fixed with #8

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants