[patch] proposed patch to fix issue

t (text/plain), 1.14 KB, created by Norman Back on 2012-10-09 14:00:25 UTC

--- lvm-monitoring.initd-2.02.67-r2	2010-08-20 20:18:29.000000000 +0100
+++ /tmp/lvm-monitoring	2012-10-09 14:50:25.974888973 +0100
@@ -18,29 +18,22 @@
 start() {
 	ret=0
 	# TODO do we want to separate out already active groups only?
-	VGSLIST=`$VGS --noheadings -o name 2> /dev/null`
-	for vg in $VGSLIST
-	do
-	    ebegin "Starting LVM monitoring for VG $vg:"
-		$VGCHANGE --monitor y --poll y $vg
-		ret2=$?
-		eend $ret2
-		[ $ret2 -ne 0 ] && ret=$ret2
-	done
+	VGSLIST=`$VGS --noheadings -o name --rows 2> /dev/null`
+	ebegin "Starting LVM monitoring for VGs ${VGSLIST}:"
+	$VGCHANGE --monitor y --poll y  ${VGSLIST}
+	ret=$?
+	eend $ret
 	return $ret
 }
 
 stop() {
 	ret=0
 	# TODO do we want to separate out already active groups only?
-	VGSLIST=`$VGS --noheadings -o name 2> /dev/null`
-	for vg in $VGSLIST
-	do
-	    ebegin "Stopping LVM monitoring for VG $vg:"
-		$VGCHANGE --monitor n $vg
-		ret2=$?
-		eend $ret2
-		[ $ret2 -ne 0 ] && ret=$ret2
-	done
+	VGSLIST=`$VGS --noheadings -o name --rows 2> /dev/null`
+        ebegin "Stopping LVM monitoring for VGs ${VGSLIST}:"
+	$VGCHANGE --monitor y --poll y  ${VGSLIST}
+	ret=$?
+	eend $ret
+
 	return $ret
 }