-
Notifications
You must be signed in to change notification settings - Fork 113
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 for active transfers. #3792
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Noice 👍
This PR adds simple gauges for active uploads and downloads. These can then be picked up by the prometheus handler.
As there's multiple managers that can upload/download, I moved the metrics to their own package, and added them to all the managers.
Besides the POST, monitor PATCH in tus manager as the upload happens in chunks.
7b42674
to
3a38605
Compare
This PR adds simple gauges for active uploads and downloads. These
can then be picked up by the prometheus handler.