19986: Prevent division by zero when reporting max disk usage
authorBrett Smith <brett.smith@curii.com>
Thu, 2 Mar 2023 19:31:50 +0000 (14:31 -0500)
committerBrett Smith <brett.smith@curii.com>
Thu, 2 Mar 2023 19:31:50 +0000 (14:31 -0500)
Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>

lib/crunchstat/crunchstat.go

index 4241f5fb5b4994629a78d3791fc7a467b37c152e..ad1cc7a97a47eba4423bd3be704b0c21807fb62b 100644 (file)
@@ -177,7 +177,7 @@ func (r *Reporter) LogMaxima(logger logPrinter, memLimits map[string]int64) {
                logger.Printf("Total disk I/O on %s was %d bytes written and %d bytes read",
                        disk, sample.txBytes, sample.rxBytes)
        }
-       if r.maxDiskSpaceSample.hasData {
+       if r.maxDiskSpaceSample.total > 0 {
                percentage := 100 * r.maxDiskSpaceSample.used / r.maxDiskSpaceSample.total
                logger.Printf("Maximum disk usage was %d%%, %d/%d bytes",
                        percentage, r.maxDiskSpaceSample.used, r.maxDiskSpaceSample.total)