prometheus-community / windows_exporter

Prometheus exporter for Windows machines
MIT License
2.89k stars 694 forks source link

/ should serve some welcome page instead of redirecting #25

Closed carlpett closed 8 years ago

carlpett commented 8 years ago

Currently we do a redirect from / to /metrics. Drawback to this is that there is no request that can be done to check that everything is alright (http server is working, at least) without also incurring the cost of running the wmi queries.

Our use case is that we have / as a http healthcheck endpoint for exporters which we register in Consul. If we want to keep the redirect, we should have some endpoint which indicates that things are ok, at least.

martinlindhe commented 8 years ago

I've personally no real opinion on this one. The redirect / -> metrics came while copying boilerplate code (from node_exporter, or one of the other exporters. Don't remember).

If a simple health check page is useful, perhaps put it on /health, rather than on the root?