[lvm-devel] [PATCH 0 of 12] LVM add 'mirrored' log type

[malahal at us.ibm.com]

Takahiro Yasui [tyasui at redhat.com] wrote:
> On 02/05/10 15:34, Jonathan Brassow wrote:
> > The following set of patches adds the 'mirrored' log type.  It differs
> > from the last batch by changing the name of the log type
> > (s/redundant/mirrored/), fixing all the issues uncovered by the
> > testsuite, adding new test cases, and including log monitoring (provided
> > by Malahal Naineni).
> 
> I'm testing the patch set, but a log monitoring doesn't work properly.
> When dmeventd detects an error, lvconvert is executed for its repair.
> For example, in case that the mirror VG/LV name is vg00/lv00, lvconvert
> was called with the following options:
> 
> lvconvert --config devices{ignore_suspended_devices=1} --repair \
>           --use-policies vg00/lv00_mlog
> 
> However, lvconvert can't accept vg00/lv00_mlog because "_mlog" is
> an reserved name. When lvconvert is executed by itself, the following
> message is displayed, and no repair is done.
> 
> # lvconvert --repair vg00/lv00_mlog
>   Names including "_mlog" are reserved. Please choose a different LV name.
>   Run `lvconvert --help' for more information.
> 
> We need to escape from this restriction when "mirrored" log is used.

I used an older LVM where it failed to convert a normal mirror. I used
"vgreduce" and it worked fine. So I used "vgreduce" to test the 'mlog'
conversion too. The actual command I used is constructed from the
following code piece:

snprintf(cmd_str, CMD_SIZE, "vgreduce --removemissing --force %s", vg)

I will test with the latest LVM and fix it. Thank you.

--Malahal.