[DRACUT PATCH] lvm_scan: use '--ignoremonitoring' rather than '--monitor n'

[DRACUT PATCH] lvm_scan: use '--ignoremonitoring' rather than '--monitor n'

[Mike Snitzer]

lvchange and vgchange '--monitor n' will not prevent lvm from
attempting to dlopen the libdevmapper-event library.

dracut git commit 47ab3b6c5e introduced the use of '--monitor n' but
'--ignoremonitoring' is needed now that the libdevmapper-event library
isn't copied into the initramfs (ever since 0fae59d6eb3)

Signed-off-by: Mike Snitzer <snitzer-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
---
 modules.d/90lvm/lvm_scan.sh |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/modules.d/90lvm/lvm_scan.sh b/modules.d/90lvm/lvm_scan.sh
index 85cd78e..f4f7b4a 100755
--- a/modules.d/90lvm/lvm_scan.sh
+++ b/modules.d/90lvm/lvm_scan.sh
@@ -60,13 +60,13 @@ nopoll=$(
 if [ -n "$LVS" ] ; then
     info "Scanning devices $lvmdevs for LVM logical volumes $LVS"
     lvm lvscan --ignorelockingfailure 2>&1 | vinfo
-    lvm lvchange -ay --ignorelockingfailure $nopoll --monitor n $LVS 2>&1 | vinfo    
+    lvm lvchange -ay --ignorelockingfailure $nopoll --ignoremonitoring $LVS 2>&1 | vinfo
 fi
 
 if [ -z "$LVS" -o -n "$VGS" ]; then
     info "Scanning devices $lvmdevs for LVM volume groups $VGS"
     lvm vgscan --ignorelockingfailure 2>&1 | vinfo
-    lvm vgchange -ay --ignorelockingfailure $nopoll --monitor n $VGS 2>&1 | vinfo
+    lvm vgchange -ay --ignorelockingfailure $nopoll --ignoremonitoring $VGS 2>&1 | vinfo
 fi
 
 if [ "$lvmwritten" ]; then

Re: [DRACUT PATCH] lvm_scan: use '--ignoremonitoring' rather than '--monitor n'

[Mike Snitzer]

On Wed, Mar 10 2010 at  2:24pm -0500,
Mike Snitzer <snitzer-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> wrote:

> lvchange and vgchange '--monitor n' will not prevent lvm from
> attempting to dlopen the libdevmapper-event library.
> 
> dracut git commit 47ab3b6c5e introduced the use of '--monitor n' but
> '--ignoremonitoring' is needed now that the libdevmapper-event library
> isn't copied into the initramfs (ever since 0fae59d6eb3)

And just to be clear:
'--monitor n' is "stop monitoring this (VG or LV)"
'--ignoremonitoring' is "don't do any monitoring".

We'll be fixing up lvm's vgchange and lvchange man pages so that they
are more precise on what '--monitor n' means.

Mike

Re: [DRACUT PATCH] lvm_scan: use '--ignoremonitoring' rather than '--monitor n'

[Harald Hoyer]

On 03/10/2010 08:24 PM, Mike Snitzer wrote:
> lvchange and vgchange '--monitor n' will not prevent lvm from
> attempting to dlopen the libdevmapper-event library.
>
> dracut git commit 47ab3b6c5e introduced the use of '--monitor n' but
> '--ignoremonitoring' is needed now that the libdevmapper-event library
> isn't copied into the initramfs (ever since 0fae59d6eb3)
>
> Signed-off-by: Mike Snitzer<snitzer-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
> ---
>   modules.d/90lvm/lvm_scan.sh |    4 ++--
>   1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/modules.d/90lvm/lvm_scan.sh b/modules.d/90lvm/lvm_scan.sh
> index 85cd78e..f4f7b4a 100755
> --- a/modules.d/90lvm/lvm_scan.sh
> +++ b/modules.d/90lvm/lvm_scan.sh
> @@ -60,13 +60,13 @@ nopoll=$(
>   if [ -n "$LVS" ] ; then
>       info "Scanning devices $lvmdevs for LVM logical volumes $LVS"
>       lvm lvscan --ignorelockingfailure 2>&1 | vinfo
> -    lvm lvchange -ay --ignorelockingfailure $nopoll --monitor n $LVS 2>&1 | vinfo
> +    lvm lvchange -ay --ignorelockingfailure $nopoll --ignoremonitoring $LVS 2>&1 | vinfo
>   fi
>
>   if [ -z "$LVS" -o -n "$VGS" ]; then
>       info "Scanning devices $lvmdevs for LVM volume groups $VGS"
>       lvm vgscan --ignorelockingfailure 2>&1 | vinfo
> -    lvm vgchange -ay --ignorelockingfailure $nopoll --monitor n $VGS 2>&1 | vinfo
> +    lvm vgchange -ay --ignorelockingfailure $nopoll --ignoremonitoring $VGS 2>&1 | vinfo
>   fi
>
>   if [ "$lvmwritten" ]; then
> --
> To unsubscribe from this list: send the line "unsubscribe initramfs" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

pushed