danvics/Pulse
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

perf: Cache lowercase RAID state in alerts processing

Browse source 
Compute strings.ToLower(array.State) once and reuse stateLower instead
of calling ToLower three times when checking RAID array status.
This commit is contained in:
rcourtman 2025-12-02 15:35:32 +00:00
parent 128dccf270
commit 9fc567dc2e
1 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
internal/alerts/alerts.go
View file

@ -2475,9 +2475,10 @@ func (m *Manager) CheckHost(host models.Host) {
} }
// Check for degraded or failed arrays // Check for degraded or failed arrays
isDegraded := strings.Contains(strings.ToLower(array.State), "degraded") || array.FailedDevices > 0 stateLower := strings.ToLower(array.State)
isRebuilding := strings.Contains(strings.ToLower(array.State), "recover") || isDegraded := strings.Contains(stateLower, "degraded") || array.FailedDevices > 0
strings.Contains(strings.ToLower(array.State), "resync") || isRebuilding := strings.Contains(stateLower, "recover") ||
strings.Contains(stateLower, "resync") ||
array.RebuildPercent > 0 array.RebuildPercent > 0
alertID := fmt.Sprintf("host-%s-raid-%s", host.ID, sanitizeRAIDDevice(array.Device)) alertID := fmt.Sprintf("host-%s-raid-%s", host.ID, sanitizeRAIDDevice(array.Device))