Re: [libvirt] [PATCH] Fix lv scanning with encrypted volumes

[Cole Robinson <crobinso redhat com>]

Daniel P. Berrange wrote:
> On Wed, Nov 12, 2008 at 04:37:16PM -0500, Cole Robinson wrote:
>> See https://bugzilla.redhat.com/show_bug.cgi?id=470693
>>
>> Using ':' as a delimiter for the lvs command isn't reliable,
>> since it looks like encrypted volume groups have a colon in
>> the physical device name, which throws of the regex. The
>> attached patch changes the delimiter to ','.
>>
>> Maintains existing behavior for me, just waiting on
>> confirmation from the reporter that this indeed does the
>> job.
> 
> ACK if it fixes the bug.
> 
> Daniel

Okay, reporter confirmed the fix. I've just pushed this.

Thanks,
Cole