Skip to content
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

libmultipath: fix 'show paths format' failure #49

Conversation

pkoetsier
Copy link
Contributor

Prevent 'multipathd show paths format "%c"' from failing on orphan paths. For orphan paths the checker class isn't set, which caused snprint_path_checker() to fail which in turn caused 'show paths format' to fail when the format string contained "%c".

Prevent 'multipathd show paths format "%c"' from failing on orphan paths.
For orphan paths the checker class isn't set, which caused
snprint_path_checker() to fail which in turn caused 'show paths format' to fail
when the format string contained "%c".
@pkoetsier
Copy link
Contributor Author

On our Kubernetes nodes, the HPE CSI driver sometimes failed to attach persistent volumes with an "unable to get scsi slaves for device..." message.
It turned out this was because the CSI driver runs 'multipathd show paths format "%w %d %t %i %o %T %C %c %s %m"' to list the paths and this command was failing when there were orphan paths.
This patch fixed the issue.

@mwilck
Copy link
Contributor

mwilck commented Oct 26, 2022

Thanks, this looks ok. But although there've been a few exceptions lately, we still use a mailing-list based workflow. Please post your patch to dm-devel AT redhat.com, cc mwilck AT suse.com and bmarzins AT redhat.com.

@pkoetsier
Copy link
Contributor Author

Thanks, I've posted the patch to dm-devel and am closing this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants