fix: skip temperature collection if SENSORS is empty string (#196)

This commit is contained in:
Henry Dollman
2024-10-24 15:10:20 -04:00
parent a5f9e2615c
commit 4094df3a61
2 changed files with 7 additions and 3 deletions

View File

@@ -62,9 +62,11 @@ func (a *Agent) Run(pubKey []byte, addr string) {
if sensors, exists := os.LookupEnv("SENSORS"); exists { if sensors, exists := os.LookupEnv("SENSORS"); exists {
a.sensorsWhitelist = make(map[string]struct{}) a.sensorsWhitelist = make(map[string]struct{})
for _, sensor := range strings.Split(sensors, ",") { for _, sensor := range strings.Split(sensors, ",") {
if sensor != "" {
a.sensorsWhitelist[sensor] = struct{}{} a.sensorsWhitelist[sensor] = struct{}{}
} }
} }
}
// initialize system info / docker manager // initialize system info / docker manager
a.initializeSystemInfo() a.initializeSystemInfo()

View File

@@ -172,10 +172,12 @@ func (a *Agent) getSystemStats() system.Stats {
} }
// temperatures (skip if sensors whitelist is set to empty string) // temperatures (skip if sensors whitelist is set to empty string)
if a.sensorsWhitelist == nil || len(a.sensorsWhitelist) > 0 { if a.sensorsWhitelist != nil && len(a.sensorsWhitelist) == 0 {
slog.Debug("Skipping temperature collection")
} else {
temps, err := sensors.TemperaturesWithContext(a.sensorsContext) temps, err := sensors.TemperaturesWithContext(a.sensorsContext)
if err != nil { if err != nil {
// err.(*sensors.Warnings).Verbose = true err.(*sensors.Warnings).Verbose = true
slog.Debug("Sensor error", "err", err) slog.Debug("Sensor error", "err", err)
} }
slog.Debug("Temperature", "sensors", temps) slog.Debug("Temperature", "sensors", temps)