Prometheus Metrics

[arbarlow]

Basic Prometheus metrics integration with the following metrics:

cert_reqs_total: Total number of request to ACME cert_errors_total: Total number of errors whilst requesting certs or keys from ACME cert_days_until_expiry: The number of days left per certificate

I've added a standard handler at /metrics also.

Note the tests pass but I'm unable to vendor the prometheus client dependency because I believe you're on an older version of dep? So I you might need to vendor before cutting your next release.

Happy to change the metric names, location or code etc!

[stibi]

Hello, how can I help to get this PR merged?

[simonswine]

@arbarlow Thanks for your contribution. It needs a few little nits but will help users of kube-lego.

• Make sure you run dep ensure, and add the libraries necessary for prometheus
• A rebase is need after the recent merge I have made
• May comment of out puting the expiry unixtime instead of the relative time left

@stibi feel free to look at these things as well

Ping me if I seem unresponsive :laughing:

[arbarlow]

Hey! OK, I'll take a look and come back soon!

[arbarlow]

@simonswine I've rebased so the current version should have no conflicts, the tests seem to fail on Travis but run locally, I would trigger a rebuild but don't have permissions

[simonswine]

Closing and reopening the pull request regularly triggers a new build

[arbarlow]

@simonswine ok, I think that's all right now?

[arbarlow]

@simonswine no problem, I'll sort these out and get back to you

[simonswine]

@arbarlow this PR against cert-manager has similar functionality: https://github.com/jetstack/cert-manager/pull/225