-
Notifications
You must be signed in to change notification settings - Fork 125
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
fio: fix device path issue for vpmem device type #2674
fio: fix device path issue for vpmem device type #2674
Conversation
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.
Can you provide run logs for failure and pass case please
b7941a2
to
55bcbea
Compare
vpmem device test
no pmem and with no disk in yaml
if disk is given as input to yaml
|
@AYUSHJAIN951 Please review |
failure case
|
55bcbea
to
e650d2e
Compare
@AYUSHJAIN951 do you have any comments on this? |
with this #2695 added a todo to move pmem code out of fio, this can be targeted as next todo on fio |
The last changes has error in getting path of the device in case if the device type is nvdimm as the fio test do not expect disk input instead it creates its own vpmem device so this changes make sure test do not fail if disk input is empty Also cancel test if either disk not given or if there is no vpmem device exists in lpar Signed-off-by: Abdul Haleem <[email protected]>
e650d2e
to
d3599da
Compare
@AYUSHJAIN951 please ACK. |
with normal disk
with vpmem device
else part
|
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.
LGTM
The last changes has error in getting path of the device in case if the device type is nvdimm as the fio test do not expect disk input instead it creates its own vpmem device so this changes make sure test do not fail if disk input is empty