-
Notifications
You must be signed in to change notification settings - Fork 123
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
fixing multipath flush problem in module_load_unload.py #2761
fixing multipath flush problem in module_load_unload.py #2761
Conversation
Before fix
After FIx
|
b19cd8d
to
9daf848
Compare
io/driver/module_unload_load.py
Outdated
@@ -98,6 +98,7 @@ def module_load_unload(self, module_list): | |||
for mod in sub_mod.split(' '): | |||
if mod == 'multipath': | |||
if self.flush_mpath(mdl) is False: | |||
self.error("failed to flush the multipath") | |||
self.error_modules.append(mdl) |
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.
if the self.error is called,, it will not append in next line.. meaning the test exits
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.
i say use self.fail instead of self. error
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.
ahh, that was just a pointer I have added to debug exact issue.
I have removed it. As we are capturing the module in a list and validating at the end of the test.
Hence it is not required here.
just removed that line. please re-check
multipath flush was failing due to string not matching exactly as below, nvme 55964 4 nvme_core 183368 5 nvme we were expecting only : "nvme 55964 4" the first line only. But it is matching all the line that contain the module name. -w option resolve this as below. nvme 55964 4 multipath is added in config file for nvme module. Signed-off-by: Naresh Bannoth <[email protected]>
9daf848
to
1f8ac43
Compare
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.
@Naresh-ibm as praveen said, self.error is a config fail,, and if it is functional fail.. use self.fail, but here self.error_modules is a param.. so we are good
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
multipath flush was failing due to string not matching exactly as below,
nvme 55964 4
nvme_core 183368 5 nvme
we were expecting only : "nvme 55964 4"
the first line only. But it is matching all the line that contain the module name.
-w option resolve this as below.
nvme 55964 4