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

[andrewwhitecdw]

Closing this sweep-generated PR: sole commit is missing a valid Signed-off-by trailer. It does not meet the sweep requirements (single signed-off commit).

[andrewwhitecdw]

Closing this upstream PR — it was opened by automation without the required sweep-approved review and therefore does not comply with the sweep policy. A replacement fork-local PR is open at andrewwhitecdw#11 for review.