Post Processing: Refactor of result modules#347
Merged
Conversation
Re-factor the run_result modules into a format that will support the adding of resource usage statirstics and plots to a report in the future Signed-off-by: Chris Harris <harriscr@uk.ibm.com>
harriscr
force-pushed
the
wip_ch_fio_cpu_report
branch
from
a3773b1 to
797d387
Compare
Re-factor the run_result modules into a format that will support the adding of resource usage statirstics and plots to a report in the future Signed-off-by: Chris Harris <harriscr@uk.ibm.com>
harriscr
force-pushed
the
wip_ch_fio_cpu_report
branch
from
797d387 to
c421fb3
Compare
Add an extra line to the graphs in a simple report that shows the CPU usage as reported by FIO. The default is not to plot the CPU usage, but this can be changed by an option to the script or in the CBT yaml file in the report section Signed-off-by: Chris Harris <harriscr@uk.ibm.com>
lee-j-sanders
approved these changes
Jan 14, 2026
Member
lee-j-sanders
left a comment
lee-j-sanders
left a comment
There was a problem hiding this comment.
I've been using the comparison tool for the past couple of days without any issues. The change itself is quite large to review, but given I've been using this for a few days without problems, it should be good to merge.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
In order to be able to include resource usage statistics in the reports produced as part of the post processing some re-factoring of the post_processing/run_result modules was necessary.
This is mainly the creating of a run_result directory and creation of some base classes that can be extended for particular run types.
This is the first step in adding resource monitor result file parsing and adding to the common format output file (which we use to generate the reports). Future PRs will deal with:
Testing
Teuthology perf-basic: https://pulpito.ceph.com/harriscr-2025-12-01_13:29:31-perf-basic-main-distro-default-smithi/
rados/perf: https://pulpito.ceph.com/harriscr-2025-12-02_08:34:46-rados:perf-main-distro-default-smithi/
All tests passed