Bug 1589658

Summary: 'multipath -r' dosen't print the expected warning messages after modify the /etc/multipath.conf file
Product: [Fedora] Fedora Reporter: lnie <lnie>
Component: device-mapper-multipathAssignee: LVM and device-mapper development team <lvm-team>
Status: CLOSED NOTABUG QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: agk, bmarzins, cfeist, heinzm, kzak, lilin, lvm-team, mcsontos, msnitzer, prajnoha, prockai
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-06-12 03:06:12 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
reproducer testcase
none
output of sh -x main.sh none

Description lnie 2018-06-11 06:01:45 UTC
Created attachment 1449879 [details]
reproducer testcase

Description of problem:
running the attached test,see some failures in the output.
Not sure if this is a bug or just a consequence of code change.
Please feel free to close this if it's not a bug,and I will change the testcase accordingly.

Version-Release number of selected component (if applicable):
device-mapper-multipath-0.7.6-4.git1cb704b.fc29.x86_64

How reproducible:
always

Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 lnie 2018-06-11 06:03:22 UTC
Created attachment 1449880 [details]
output of sh -x main.sh

Comment 2 Ben Marzinski 2018-06-11 20:30:39 UTC
I'm not sure what messages you are worried about. The missing quotes test appears to be wrong. The multipath.conf file used for it doesn't have mismatched quotes (it has none, which is fine).

In the new code multipath will offload some commands to multipathd. This is the case with

multipath -r

That's why it doesn't print any output. If you run "multipath -l" afterwards, and look for the new name, it should be there.  Were there any other tests that gave you bad results, that you don't understand?

Comment 3 lnie 2018-06-12 03:05:56 UTC
(In reply to Ben Marzinski from comment #2)
> I'm not sure what messages you are worried about. The missing quotes test
> appears to be wrong. The multipath.conf file used for it doesn't have
> mismatched quotes (it has none, which is fine).
> 
> In the new code multipath will offload some commands to multipathd. This is
> the case with
> 
> multipath -r
> 
> That's why it doesn't print any output. If you run "multipath -l"
> afterwards, and look for the new name, it should be there.  Were there any
> other tests that gave you bad results, that you don't understand?

 nope atm,gonna to close this.Thanks for your quick response and detailed explanation:)