-
Notifications
You must be signed in to change notification settings - Fork 55
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
Add prometheus metrics #83
Comments
I love this. |
What about also exposing time spent processing requests? |
Might also be use to count socket and file system reads. |
I wonder if it’s appropriate to use Python audit hooks for that kind of performance tracing. |
We needed to start capturing metrics urgently to make sense of some user reports, so I rushed something into |
The socket and file usage can be observed through a generic node_exporter. I don’t think we need the Tiled application to do anything special there. Still needs docs. |
Old gist that is relevant: https://gist.github.com/danielballan/1bfb0576781c5671e3b97af2ebafcfa1 |
Evidently |
Draw inspiration from https://docs.dask.org/en/latest/setup/prometheus.html. Exposing
gc
metrics in particular seems a good idea.The text was updated successfully, but these errors were encountered: