public inbox for lvm2-cvs@sourceware.org
help / color / mirror / Atom feed
* LVM2 ./WHATS_NEW test/t-activate-partial.sh to ...
@ 2010-06-03 12:45 mbroz
0 siblings, 0 replies; 2+ messages in thread
From: mbroz @ 2010-06-03 12:45 UTC (permalink / raw)
To: lvm-devel, lvm2-cvs
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;
^ permalink raw reply [flat|nested] 2+ messages in thread
* LVM2 ./WHATS_NEW test/t-activate-partial.sh to ...
@ 2010-06-30 14:01 mbroz
0 siblings, 0 replies; 2+ messages in thread
From: mbroz @ 2010-06-30 14:01 UTC (permalink / raw)
To: lvm-devel, lvm2-cvs
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;
}
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-06-30 14:01 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-06-03 12:45 LVM2 ./WHATS_NEW test/t-activate-partial.sh to mbroz
2010-06-30 14:01 mbroz
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).