-
Notifications
You must be signed in to change notification settings - Fork 6k
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
qa: add cbt task for performance testing #17583
qa: add cbt task for performance testing #17583
Conversation
Signed-off-by: Neha Ojha <nojha@redhat.com>
Signed-off-by: Neha Ojha <nojha@redhat.com>
osd_ra: [4096] | ||
pool_profile: 'rbd' | ||
log_avg_msec: 100 | ||
cmd_path: '/usr/local/bin/fio' |
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.
the cbt task should probably set this automatically if there's a librbdfio section
qa/tasks/cbt.py
Outdated
'cd', os.path.join(testdir, 'fio'), run.Raw('&&'), | ||
'./configure', run.Raw('&&'), | ||
'make', run.Raw('&&'), | ||
'sudo', 'make', 'install' |
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.
should avoid 'make install' into a persistent dir, since the machines will get reused, and could get problems from different versions later
configure with --prefix=testdir/fio/build or something, so that it's cleaned up at the end?
rebuild_every_test=False, | ||
use_existing=True, | ||
iterations=self.config.get('cluster', {}).get('iterations', 1), | ||
tmp_dir='/tmp/cbt', |
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.
maybe make this in the archive dir, so we save anything that cbt doesn't summarize as well
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.
As discussed, this change doesn't seem to be required at the moment, because the cbt task archives all the information generated in the cbt results dir.
Signed-off-by: Neha Ojha <nojha@redhat.com>
Yay Neha! |
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.
looks good to me! let's get this merged so we can start seeing how variable test runs will be
The idea is to run cbt performance tests using teuthology.
We can provide a normal cbt configuration to the cbt task, telling it which benchmarks to run and it will setup performance tests on teuthology nodes.
Signed-off-by: Neha Ojha nojha@redhat.com