-
-
Notifications
You must be signed in to change notification settings - Fork 85
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
Stats export #64
Comments
Instrumenting for prometheus should be fairly easy (https://prometheus.io/docs/guides/go-application/). I think most common monitoring systems can ingest the data by scraping the /metrics page. |
Instrumenting the Go application is indeed fairly easy, since Prometheus itself provides the tools, but the issue here comes from the application not running continuously. This can be connected to #268, which would also require the application to take control and be the entrypoint of the containers. Regarding how Prometheus operates, the microservices expose a metrics endpoint, most commonly I would personally try to avoid having a state for a backup application, so the metrics should be limited to information available since last startup. The metrics I would be interested in: sum of backups, percentage of successful backups, how long were the containers stopped, how long it took to complete the backup. |
I looked into this in https://github.com/offen/docker-volume-backup/tree/serve-metrics once. All it would need is some logic that serializes the already available metrics (plus maybe some others) into the correct format and writes them to a file.
That'd also be an option in case busybox's |
#60 introduced the collection of stats about a backup run (e.g. size, duration, number of pruned items). These stats can currently be consumed in post-backup notifications, however it should also be possible to send these to arbitrary monitoring systems (e.g. Prometheus) so that users can monitor their backups on such systems.
I don't know too much about the monitoring landscape out there, but in case someone would like to use this feature, I'd be happy to hear from you about how this could work in your setup.
The text was updated successfully, but these errors were encountered: