LVM2 ./WHATS_NEW test/t-activate-partial.sh to ...

LVM2 ./WHATS_NEW test/t-activate-partial.sh to ...

[mbroz]

CVSROOT:	/cvs/lvm2
Module name:	LVM2
Changes by:	mbroz@sourceware.org	2010-06-30 14:01:40

Modified files:
	.              : WHATS_NEW 
	test           : t-activate-partial.sh 
	tools          : vgremove.c 

Log message:
	Fix vgremove to allow removal of VG with missing PVs. (2.02.52)

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/WHATS_NEW.diff?cvsroot=lvm2&r1=1.1640&r2=1.1641
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/test/t-activate-partial.sh.diff?cvsroot=lvm2&r1=1.2&r2=1.3
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/tools/vgremove.c.diff?cvsroot=lvm2&r1=1.59&r2=1.60

--- LVM2/WHATS_NEW	2010/06/30 13:11:12	1.1640
+++ LVM2/WHATS_NEW	2010/06/30 14:01:39	1.1641
@@ -1,5 +1,6 @@
 Version 2.02.69 - 
 ================================
+  Fix vgremove to allow removal of VG with missing PVs. (2.02.52)
   Add --metadataignore to pvcreate and vgextend and define in lvm.conf.
   Add {pv|vg}_mda_used_count which displays a count of mdas w/ignored bit clear.
   Add vg_mda_copies which displays the value of --vgmetadatacopies.
--- LVM2/test/t-activate-partial.sh	2010/06/03 12:45:09	1.2
+++ LVM2/test/t-activate-partial.sh	2010/06/30 14:01:39	1.3
@@ -15,3 +15,6 @@
 # also check that vgchange works
 vgchange -a n --partial $vg
 vgchange -a y --partial $vg
+
+# check vgremove
+vgremove -f $vg
\ No newline at end of file
--- LVM2/tools/vgremove.c	2010/01/08 14:03:55	1.59
+++ LVM2/tools/vgremove.c	2010/06/30 14:01:40	1.60
@@ -49,7 +49,7 @@
 		}
 	}
 
-	if (!vg_remove_check(vg)) {
+	if (!force && !vg_remove_check(vg)) {
 		stack;
 		return ECMD_FAILED;
 	}

LVM2 ./WHATS_NEW test/t-activate-partial.sh to ...

[mbroz]

CVSROOT:	/cvs/lvm2
Module name:	LVM2
Changes by:	mbroz@sourceware.org	2010-06-03 12:45:09

Modified files:
	.              : WHATS_NEW 
	test           : t-activate-partial.sh 
	tools          : toollib.c 

Log message:
	Require partial option in lvchange --refresh for partials LVs.
	
	We must not refresh LV if some PVs are missing and partial activation
	was not requested.
	
	Fixes https://bugzilla.redhat.com/show_bug.cgi?id=598886

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/WHATS_NEW.diff?cvsroot=lvm2&r1=1.1603&r2=1.1604
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/test/t-activate-partial.sh.diff?cvsroot=lvm2&r1=1.1&r2=1.2
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/tools/toollib.c.diff?cvsroot=lvm2&r1=1.201&r2=1.202

--- LVM2/WHATS_NEW	2010/06/01 21:47:57	1.1603
+++ LVM2/WHATS_NEW	2010/06/03 12:45:05	1.1604
@@ -1,5 +1,6 @@
 Version 2.02.67 -
 ===============================
+  Require partial option in lvchange --refresh for partial LVs.
   Do not fail lvm_init() if init_logging() or _init_rand() generates an errno.
   Don't merge unchanged persistent cache file before dumping if tool scanned.
   Fix incorrect memory pool deallocation while using vg_read for files.
--- LVM2/test/t-activate-partial.sh	2010/05/12 05:59:24	1.1
+++ LVM2/test/t-activate-partial.sh	2010/06/03 12:45:09	1.2
@@ -9,6 +9,8 @@
 not vgreduce --removemissing $vg
 not lvchange -v -a y $vg/mirror
 lvchange -v --partial -a y $vg/mirror
+not lvchange -v --refresh $vg/mirror
+lvchange -v --refresh --partial $vg/mirror
 
 # also check that vgchange works
 vgchange -a n --partial $vg
--- LVM2/tools/toollib.c	2010/05/21 14:15:39	1.201
+++ LVM2/tools/toollib.c	2010/06/03 12:45:09	1.202
@@ -1248,6 +1248,12 @@
 {
 	int r = 0;
 
+	if (!cmd->partial_activation && (lv->status & PARTIAL_LV)) {
+		log_error("Refusing refresh of partial LV %s. Use --partial to override.",
+			  lv->name);
+		goto out;
+	}
+
 	r = suspend_lv(cmd, lv);
 	if (!r)
 		goto_out;