Don't log collector errors (#1050)

SuperQ · web-flow · commit 56914a1d8b28 · 2025-10-28T07:08:08.000-04:00
Avoid logging collector failures at Error level to avoid log spam.

Signed-off-by: SuperQ &lt;superq@gmail.com&gt;
diff --git a/collector/collector.go b/collector/collector.go
@@ -185,7 +185,7 @@ func execute(ctx context.Context, name string, c Collector, ch chan<- prometheus
 		if IsNoDataError(err) {
 			logger.Debug("collector returned no data", "name", name, "duration_seconds", duration.Seconds(), "err", err)
 		} else {
-			logger.Error("collector failed", "name", name, "duration_seconds", duration.Seconds(), "err", err)
+			logger.Warn("collector failed", "name", name, "duration_seconds", duration.Seconds(), "err", err)
 		}
 		success = 0
 	} else {

Original file line number	Diff line number	Diff line change
`@@ -185,7 +185,7 @@ func execute(ctx context.Context, name string, c Collector, ch chan<- prometheus`
`185`	`185`	`if IsNoDataError(err) {`
`186`	`186`	`logger.Debug("collector returned no data", "name", name, "duration_seconds", duration.Seconds(), "err", err)`
`187`	`187`	`} else {`
`188`		`- logger.Error("collector failed", "name", name, "duration_seconds", duration.Seconds(), "err", err)`
	`188`	`+ logger.Warn("collector failed", "name", name, "duration_seconds", duration.Seconds(), "err", err)`
`189`	`189`	`}`
`190`	`190`	`success = 0`
`191`	`191`	`} else {`