Cap overalPct at 100% to avoid wrong values (#21)

* Cap overalPct at 100% to avoid wrong values
DataDog · Jul 24, 2017 · 60b0047 · 60b0047
1 parent a1c15a9
commit 60b0047
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/checks/process.go b/checks/process.go
@@ -275,6 +275,11 @@ func calculatePct(deltaProc, deltaTime, numCPU float64) float32 {
 	// on a 2-CPU-core system would be reported as 50% instead of 100%.
 	overalPct := (deltaProc / deltaTime) * 100
 
+	// Sometimes we get values that don't make sense, so we clamp to 100%
+	if overalPct > 100 {
+		overalPct = 100
+	}
+
 	// In order to emulate top we multiply utilization by # of CPUs so a busy loop would be 100%.
 	return float32(overalPct * numCPU)
 }