Re: [libvirt] storage pool that contains thin LVs

2013-09-20 Thread Dusty Mabe
On Fri, Sep 20, 2013 at 5:18 AM, Daniel P. Berrange wrote: > On Thu, Sep 19, 2013 at 11:34:15PM -0400, Dusty Mabe wrote: >> >> I'm sure the fix isn't this trivial but i can finish it up and submit >> an official patch (through git) if this is close. > > Do we really want to hide thin volumes. Unle

Re: [libvirt] storage pool that contains thin LVs

2013-09-20 Thread Daniel P. Berrange
On Thu, Sep 19, 2013 at 11:34:15PM -0400, Dusty Mabe wrote: > Hey guys, > > Just looking at https://bugzilla.redhat.com/show_bug.cgi?id=924672 and > looks like all we need to do is ignore thin pools and thin pool data > devices. With some trivial testing this seems to work fine (i.e. > failed befo

Re: [libvirt] storage pool that contains thin LVs

2013-09-19 Thread Doug Goldstein
On Thu, Sep 19, 2013 at 10:34 PM, Dusty Mabe wrote: > Hey guys, > > Just looking at https://bugzilla.redhat.com/show_bug.cgi?id=924672 and > looks like all we need to do is ignore thin pools and thin pool data > devices. With some trivial testing this seems to work fine (i.e. > failed before and w

[libvirt] storage pool that contains thin LVs

2013-09-19 Thread Dusty Mabe
Hey guys, Just looking at https://bugzilla.redhat.com/show_bug.cgi?id=924672 and looks like all we need to do is ignore thin pools and thin pool data devices. With some trivial testing this seems to work fine (i.e. failed before and works now). diff --git a/src/storage/storage_backend_logical.c