Bug 1347486

Summary: cannot pool-define/create mpath pool
Product: [Fedora] Fedora Reporter: Han Han <hhan>
Component: libvirtAssignee: Ján Tomko <jtomko>
Status: CLOSED UPSTREAM QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: high Docs Contact:
Priority: high    
Version: rawhideCC: agedosier, berrange, clalancette, crobinso, dyuan, itamar, jtomko, laine, libvirt-maint, rbalakri, veillard, virt-bugs, virt-maint, xuzhang, yanyang, yisun
Target Milestone: ---Keywords: Regression
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1346724 Environment:
Last Closed: 2016-06-22 00:27:16 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:

Description Han Han 2016-06-17 04:30:07 UTC
It can also be reproduced on libvirt-1.3.5-2.fc25.x86_64.
+++ This bug was initially created as a clone of Bug #1346724 +++

Description:
cannot pool-define/create mpath pool

versions:
libvirt-1.3.5-1.el7.x86_64

reproduce rate:
100%

THIS IS A REGRESSION BUG, and it works on libvirt-1.3.4-1.el7.x86_64


Steps:
1. # cat pool.mpath
<pool type="mpath">
<name>mpath-pool-1</name>
<target>
<path>/dev/mapper</path>
</target>
</pool>


2. # virsh pool-define pool.mpath
error: Failed to define pool from pool.mpath
error: internal error: missing backend for pool type 7 (mpath)

Actual result:
mpath pool cannot be defined


Expected result:
mpath pool can be defined


Additional info:
This works fined with libvirt-1.3.4-1.el7.x86_64

--- Additional comment from Red Hat Bugzilla Rules Engine on 2016-06-15 04:54:39 EDT ---

This bug report has Keywords: Regression or TestBlocker.

Since no regressions or test blockers are allowed between releases, it is also being [proposed|marked] as a blocker for this release.

Please resolve ASAP.

Comment 1 Ján Tomko 2016-06-17 10:22:40 UTC
Upstream patch:
https://www.redhat.com/archives/libvir-list/2016-June/msg01147.html

Comment 2 Ján Tomko 2016-06-17 13:39:59 UTC
Fixed upstream by:
commit 7303e7079ccfec1bf7d2891c6e87a8646355f124
Author:     Ján Tomko <jtomko>
CommitDate: 2016-06-17 15:35:46 +0200

    configure: define preprocessor macros for SCSI and MPATH
    
    This fixes building these backends when explicitly enabled
    on the command line.
    
    Exposed by commit a659559 which started adding
    --with-storage-mpath in the spec file.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1346724

git describe: v1.3.5-306-g7303e70

Comment 3 Cole Robinson 2016-06-22 00:27:16 UTC
Just closing as upstream, I doubt anyone is really depending on mpath pools on rawhide, and this will be fixed in a couple weeks with the next rebase