Red Hat Bugzilla – Bug 676165
Typo in multipath output, "mulipath.conf line 17, invalid keyword:"
Last modified: 2011-08-16 08:59:34 EDT
Description of problem:
There is a typo in the new multipath package:
[root@cs-rh5-1 /]# multipath
mulipath.conf line 17, invalid keyword: polling_interval
I wrote a quick patch to fix it.
Version-Release number of selected component (if applicable):
[root@cs-rh5-1 /]# rpm -q device-mapper-multipath
Steps to Reproduce:
1. Run the command multiapth with an error in the config file
2. Look at console output and messages file
Multipath.conf is spelled wrong.
Multipath.conf is spelled correct.
Attached a patch that should fix it.
Created attachment 477713 [details]
Patch that fixes typo
Patch that fixes typo
Patch applied. Thanks.
how the heck do you patch this? what are you actually patching?
Any help would be most appreciated!
Commont #1 has the patch, I'm patching the device-mapper-multipath source
It's available for web-viewing here:
and can be checked out via cvs with
cvs -d :pserver:email@example.com:/cvs/dm login cvs
cvs -d :pserver:firstname.lastname@example.org:/cvs/dm checkout multipath-tools
The password is cvs.
This is a trivial fix for a spelling mistake.
kewl, thank you,
so do I just replace the directories? do I create a rpm package?
not sure where to go from here.
I'm confused as to why you want to build your own version of device-mapper-multipath with this fix. All this does is fix a typo in one error message.
Instead of printing
mulipath.conf line <LINE_NR>, invalid keyword:
it will now print
multipath.conf line <LINE_NR>, invalid keyword:
The easiest way to patch this is to get the src.rpm you want, apply the patch to that, and rebuild the rpm.
The other way is to run
# cvs -d :pserver:email@example.com:/cvs/dm login cvs
The password is "cvs"
# cvs -d :pserver:firstname.lastname@example.org:/cvs/dm checkout -r RHEL5_FC6 multipath-tools
This will checkout the latest multipath.conf source. Assuming you have all of the development libraries, you can build this, and it already includes this patch. If you want to continue using your existing multipath code with simply this package added, you need to look at the src.rpm for it, figure out the proper cvs tag, checkout that version, manually apply the patch and rebuild.
But, like I said, that's a lot of work to simply change "mulipath" to "multipath" in one error message.
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.
Just a small note for those who got here because of the "polling_interval" problem (as opposed to the typo problem which this ticket handles) -
Starting on RHEL 5.6 (due to #560892 fix or 75d588b92f6 git hash), multipath.conf processing started reporting conf file errors.
So in cases such as putting the polling_interval configuration directive in the wrong place (e.g. inside "devices" section, whereas it belongs to the "defaults" section), this error is reported.
Before RHEL5.6 the problem was silently skipped.