New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Collate fio installations at one place #1506
Conversation
Currently the commands for installing fio is duplicated at two different places. This change merges them and puts it into a common script at perfmetrics/scripts/fio/install_fio.sh and invokes it from both the places.
3b65e0d
to
c6c3722
Compare
@raj-prince please review this first and then #1503 . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few minor comments..
334e5b6
to
00a6876
Compare
As fio-3.35 apparently doesn't support job_start config.
the perf presubmit test has passed: log. showing results... |
@raj-prince please review. |
* Collate fio installations at one place Currently the commands for installing fio is duplicated at two different places. This change merges them and puts it into a common script at perfmetrics/scripts/fio/install_fio.sh and invokes it from both the places. * addressed comments * Use fio-3.36 instead of fio-3.35 As fio-3.35 apparently doesn't support job_start config.
* Collate fio installations at one place Currently the commands for installing fio is duplicated at two different places. This change merges them and puts it into a common script at perfmetrics/scripts/fio/install_fio.sh and invokes it from both the places. * addressed comments * Use fio-3.36 instead of fio-3.35 As fio-3.35 apparently doesn't support job_start config.
* Collate fio installations at one place Currently the commands for installing fio is duplicated at two different places. This change merges them and puts it into a common script at perfmetrics/scripts/fio/install_fio.sh and invokes it from both the places. * addressed comments * Use fio-3.36 instead of fio-3.35 As fio-3.35 apparently doesn't support job_start config.
Description
Currently the commands for installing fio
is duplicated at two different places. This change merges them and puts it into a common script at
perfmetrics/scripts/fio/install_fio.sh and invokes it from both the places.
fio version 3.36 has been released, so we could directly installed that from the fio-3.36 package, but we need to build fio from source for the fix in clat latency calculations .
This is precursor to #1503.
Link to the issue in case of a bug fix.
NA
Testing details