louwrentius / fio-plot

Create charts from FIO storage benchmark tool output
BSD 3-Clause "New" or "Revised" License
370 stars 87 forks source link

-t bw option produces an error on fio_plot #14

Closed nwilson666 closed 4 years ago

nwilson666 commented 4 years ago

when I use the -t bw to get bandwidth, it give me an error - lat and iops work options work fine.

Traceback (most recent call last): File "./fio_plot", line 135, in main() File "./fio_plot", line 123, in main uses_json_files[item](settings, parsed_data) File "/spice/data/users/itnw/fio-plot-master/fio_plot/fiolib/bar3d.py", line 28, in plot_3d rw, metric) File "/spice/data/users/itnw/fio-plot-master/fio_plot/fiolib/shared_chart.py", line 45, in get_record_set_3d row.append(record[metric]) KeyError: 'bw'

louwrentius commented 4 years ago

Thanks, the -t option for 3dGraphs only supports iops and lat no bw. For now, you could just use IOPS as IOPS x block size = bandwidth.

I should either implement it and in the mean time maybe gracefully handle this error.