fix(fio): IOPS parser drops decimal values with '\d+' regex (+1 more)

[andrewwhitecdw]

This PR addresses 2 issues in bobber/lib/analysis/fio.py.

Fix 1

fix: IOPS parser drops decimal values with '\d+' regex

Fix: Replace:

    number = float(re.findall(r'\d+', iops)[0])

with:

    number = float(re.findall(r'[-+]?\d+(?:\.\d+)?', iops)[0])

Fix 2

fix: missing IOPS logs are treated as zero instead of skipped

Fix: Replace:

        write_iops = fio_iops_results(log_contents, systems, 'write: IOPS=.*',
                                      log)
        read_iops = fio_iops_results(log_contents, systems, 'read: IOPS=.*',
                                     log)
        write_system_results[systems].append(sum(write_iops))
        read_system_results[systems].append(sum(read_iops))

with:

        write_iops = fio_iops_results(log_contents, systems, 'write: IOPS=.*',
                                      log)
        if write_iops == []:
            continue
        read_iops = fio_iops_results(log_contents, systems, 'read: IOPS=.*',
                                     log)
        if read_iops == []:
            continue
        write_system_results[systems].append(sum(write_iops))
        read_system_results[systems].append(sum(read_iops))

Files changed

• bobber/lib/analysis/fio.py