mirror of
https://github.com/prometheus/node_exporter.git
synced 2024-11-23 20:36:21 +01:00
Fix promhttp_metric_handler_errors_total metric not being disabled by flag
Signed-off-by: ToMe25 <ToMe25@gmx.de>
This commit is contained in:
parent
e8c5110ada
commit
db3a43783a
@ -123,7 +123,10 @@ func (h *handler) innerHandler(filters ...string) (http.Handler, error) {
|
|||||||
if err := r.Register(nc); err != nil {
|
if err := r.Register(nc); err != nil {
|
||||||
return nil, fmt.Errorf("couldn't register node collector: %s", err)
|
return nil, fmt.Errorf("couldn't register node collector: %s", err)
|
||||||
}
|
}
|
||||||
handler := promhttp.HandlerFor(
|
|
||||||
|
var handler http.Handler
|
||||||
|
if h.includeExporterMetrics {
|
||||||
|
handler = promhttp.HandlerFor(
|
||||||
prometheus.Gatherers{h.exporterMetricsRegistry, r},
|
prometheus.Gatherers{h.exporterMetricsRegistry, r},
|
||||||
promhttp.HandlerOpts{
|
promhttp.HandlerOpts{
|
||||||
ErrorLog: stdlog.New(log.NewStdlibAdapter(level.Error(h.logger)), "", 0),
|
ErrorLog: stdlog.New(log.NewStdlibAdapter(level.Error(h.logger)), "", 0),
|
||||||
@ -132,13 +135,22 @@ func (h *handler) innerHandler(filters ...string) (http.Handler, error) {
|
|||||||
Registry: h.exporterMetricsRegistry,
|
Registry: h.exporterMetricsRegistry,
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
if h.includeExporterMetrics {
|
|
||||||
// Note that we have to use h.exporterMetricsRegistry here to
|
// Note that we have to use h.exporterMetricsRegistry here to
|
||||||
// use the same promhttp metrics for all expositions.
|
// use the same promhttp metrics for all expositions.
|
||||||
handler = promhttp.InstrumentMetricHandler(
|
handler = promhttp.InstrumentMetricHandler(
|
||||||
h.exporterMetricsRegistry, handler,
|
h.exporterMetricsRegistry, handler,
|
||||||
)
|
)
|
||||||
|
} else {
|
||||||
|
handler = promhttp.HandlerFor(
|
||||||
|
r,
|
||||||
|
promhttp.HandlerOpts{
|
||||||
|
ErrorLog: stdlog.New(log.NewStdlibAdapter(level.Error(h.logger)), "", 0),
|
||||||
|
ErrorHandling: promhttp.ContinueOnError,
|
||||||
|
MaxRequestsInFlight: h.maxRequests,
|
||||||
|
},
|
||||||
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
return handler, nil
|
return handler, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user