lvm2-commits September 2013

lvm2-commits@lists.fedorahosted.org

6 participants
107 discussions

master - tests: no activate LV
by Zdenek Kabelac 16 Sep '13

16 Sep '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=61427af377891b01… Commit: 61427af377891b011089b14154fe0c1d2845fd21 Parent: 2abaf87a9b36cff2f32398a8dfb26609c0f56e14 Author: Zdenek Kabelac <zkabelac(a)redhat.com> AuthorDate: Mon Sep 16 15:38:42 2013 +0200 Committer: Zdenek Kabelac <zkabelac(a)redhat.com> CommitterDate: Mon Sep 16 15:38:42 2013 +0200 tests: no activate LV Since our current vgcfgbackup/restore doesn't deal with difference of active volumes between current and restored set of volumes - run test with inactive LVs. --- test/shell/vgcfgbackup-usage.sh | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/test/shell/vgcfgbackup-usage.sh b/test/shell/vgcfgbackup-usage.sh index b04ac86..314ecdd 100644 --- a/test/shell/vgcfgbackup-usage.sh +++ b/test/shell/vgcfgbackup-usage.sh @@ -22,7 +22,7 @@ vgcfgbackup -f bak-%s >out grep "Volume group \"$vg1\" successfully backed up." out grep "Volume group \"$vg2\" successfully backed up." out # increase seqno -lvcreate -l1 $vg1 +lvcreate -an -Zn -l1 $vg1 vgcfgrestore -f bak-$vg1 $vg1 vgremove -ff $vg1 $vg2

1 0

master - tests: update vgcfgrestore test
by Zdenek Kabelac 16 Sep '13

16 Sep '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=2abaf87a9b36cff2… Commit: 2abaf87a9b36cff2f32398a8dfb26609c0f56e14 Parent: 9742c5192e8871ebde12856170dae70ee861ef40 Author: Zdenek Kabelac <zkabelac(a)redhat.com> AuthorDate: Mon Sep 16 13:46:17 2013 +0200 Committer: Zdenek Kabelac <zkabelac(a)redhat.com> CommitterDate: Mon Sep 16 13:48:44 2013 +0200 tests: update vgcfgrestore test Test vgcfgrestore when seqno of restored mda is smaller then current metadata on device. --- test/shell/vgcfgbackup-usage.sh | 7 +++++-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/test/shell/vgcfgbackup-usage.sh b/test/shell/vgcfgbackup-usage.sh index 7a1e530..b04ac86 100644 --- a/test/shell/vgcfgbackup-usage.sh +++ b/test/shell/vgcfgbackup-usage.sh @@ -1,5 +1,5 @@ #!/bin/sh -# Copyright (C) 2008 Red Hat, Inc. All rights reserved. +# Copyright (C) 2008-2013 Red Hat, Inc. All rights reserved. # # This copyrighted material is made available to anyone wishing to use, # modify, copy, or redistribute it subject to the terms and conditions @@ -18,9 +18,12 @@ vg1=${PREFIX}vg00 vg2=${PREFIX}vg01 vgcreate $vg1 "$dev1" vgcreate $vg2 "$dev2" -vgcfgbackup -f $TESTDIR/bak-%s >out +vgcfgbackup -f bak-%s >out grep "Volume group \"$vg1\" successfully backed up." out grep "Volume group \"$vg2\" successfully backed up." out +# increase seqno +lvcreate -l1 $vg1 +vgcfgrestore -f bak-$vg1 $vg1 vgremove -ff $vg1 $vg2 # vgcfgbackup correctly stores metadata with missing PVs

1 0

master - systemd: run lvm2-activation-net.service after lvm2-activation.service
by Peter Rajnoha 16 Sep '13

16 Sep '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=9742c5192e8871eb… Commit: 9742c5192e8871ebde12856170dae70ee861ef40 Parent: fbb732bd8eb5308eeb1dc7cc6d747552844018ac Author: Peter Rajnoha <prajnoha(a)redhat.com> AuthorDate: Mon Sep 16 11:47:09 2013 +0200 Committer: Peter Rajnoha <prajnoha(a)redhat.com> CommitterDate: Mon Sep 16 11:47:09 2013 +0200 systemd: run lvm2-activation-net.service after lvm2-activation.service The lvm2-activation-net.service was ordered only with respect to iscsi and fcoe service before. In addition to that, we also need ordering with respect to lvm2-activation.service to prevent parallel vgchange -aay runs which may cause some problems during activation. See also https://bugs.gentoo.org/show_bug.cgi?id=480066. With this patch, the ordering is firmly set to: lvm2-activation-early.service -> lvm2-activation.service -> lvm2-activation-net.service Thanks to Alexander Tsoy for the original patch (modified a bit here): https://www.redhat.com/archives/lvm-devel/2013-September/msg00049.html --- WHATS_NEW | 1 + .../lvm2_activation_generator_systemd_red_hat.c | 10 +++++----- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/WHATS_NEW b/WHATS_NEW index a46b0b4..1c4925b 100644 --- a/WHATS_NEW +++ b/WHATS_NEW @@ -1,5 +1,6 @@ Version 2.02.101 - =================================== + Run lvm2-activation-net after lvm2-activation service to prevent parallel run. Add man page entries for lvmdump's -u and -l options. Fix lvm2app segfault while using lvm_list_pvs_free fn if there are no PVs. Improve of clvmd singlenode locking simulation. diff --git a/scripts/lvm2_activation_generator_systemd_red_hat.c b/scripts/lvm2_activation_generator_systemd_red_hat.c index 7e93d7d..f2e2e49 100644 --- a/scripts/lvm2_activation_generator_systemd_red_hat.c +++ b/scripts/lvm2_activation_generator_systemd_red_hat.c @@ -134,16 +134,16 @@ static int generate_unit(const char *dir, int unit) "DefaultDependencies=no\n", f); if (unit == UNIT_NET) { - fputs("After=iscsi.service fcoe.service\n" - "Before=remote-fs.target shutdown.target\n\n" - "[Service]\n" - "ExecStartPre=/usr/bin/udevadm settle\n", f); + fprintf(f, "After=%s iscsi.service fcoe.service\n" + "Before=remote-fs.target shutdown.target\n\n" + "[Service]\n" + "ExecStartPre=/usr/bin/udevadm settle\n", unit_names[UNIT_MAIN]); } else { if (unit == UNIT_EARLY) { fputs("After=systemd-udev-settle.service\n" "Before=cryptsetup.target\n", f); } else - fputs("After=lvm2-activation-early.service cryptsetup.target\n", f); + fprintf(f, "After= %s cryptsetup.target\n", unit_names[UNIT_EARLY]); fputs("Before=local-fs.target shutdown.target\n" "Wants=systemd-udev-settle.service\n\n"

1 0

master - tests: enhance pvmove testing
by Zdenek Kabelac 16 Sep '13

16 Sep '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=fbb732bd8eb5308e… Commit: fbb732bd8eb5308eeb1dc7cc6d747552844018ac Parent: 47b7cc6850091aac9d32070ad3c3a727bae5f188 Author: Zdenek Kabelac <zkabelac(a)redhat.com> AuthorDate: Mon Sep 16 11:18:31 2013 +0200 Committer: Zdenek Kabelac <zkabelac(a)redhat.com> CommitterDate: Mon Sep 16 11:22:04 2013 +0200 tests: enhance pvmove testing Rewrite check lv_on and add new lv_tree_on Move more pvmove test unrelated code out to check & get sections (so they do not obfuscate trace output unnecesserily) Use new lv_tree_on() NOTE: unsure how the snapshot origin should be accounted here. Split pmove-all-segments into separate tests for raid and thins (so the test output properly shows what has been skipped in test) --- test/lib/check.sh | 40 ++++- test/lib/get.sh | 34 ++++ test/shell/pvmove-all-segtypes.sh | 300 +++++----------------------------- test/shell/pvmove-basic.sh | 311 ++++++++++++++++-------------------- test/shell/pvmove-raid-segtypes.sh | 79 +++++++++ test/shell/pvmove-thin-segtypes.sh | 83 ++++++++++ 6 files changed, 410 insertions(+), 437 deletions(-) diff --git a/test/lib/check.sh b/test/lib/check.sh index d3678a9..4847de1 100644 --- a/test/lib/check.sh +++ b/test/lib/check.sh @@ -1,5 +1,5 @@ #!/usr/bin/env bash -# Copyright (C) 2010-2012 Red Hat, Inc. All rights reserved. +# Copyright (C) 2010-2013 Red Hat, Inc. All rights reserved. # # This copyrighted material is made available to anyone wishing to use, # modify, copy, or redistribute it subject to the terms and conditions @@ -54,14 +54,38 @@ mirror_images_redundant() { $(cat check.tmp.all) } +lv_err_list_() { + (echo "$2" | not grep -m 1 -q "$1") || \ + echo "$3 on [ $(echo "$2" | grep "$1" | cut -b3- | tr '\n' ' ')] " +} + +lv_on_diff_() { + declare -a devs=("${!1}") # pass in shell array + local expect=( "${@:4}" ) # make an array starting from 4th args... + local diff_e + + # Find diff between 2 shell arrays, print them as stdin files + diff_e=$(diff <(printf "%s\n" "${expect[@]}" | sort | uniq ) <(printf "%s\n" "${devs[@]}") ) || + die "LV $2/$3 $(lv_err_list_ "^>" "${diff_e}" found)$(lv_err_list_ "^<" "${diff_e}" "not found")." +} + +# list devices for given LV lv_on() { - local lv=$1/$2 - (lvdevices $lv | grep -F "$3") || \ - die "LV $lv expected on $3 but is not:" \ - $(lvdevices $lv) - test $(lvdevices $lv | grep -vF "$3" | wc -l) -eq 0 || \ - die "LV $lv contains unexpected devices:" \ - $(lvdevices $lv) + local devs + + devs=( $(lvdevices "$1/$2" | sort | uniq ) ) + + lv_on_diff_ devs[@] "${@}" +} + +# list devices for given LV and all its subdevices +lv_tree_on() { + local devs + + # Get sorted list of devices + devs=( $(get lv_tree_devices "$1" "$2") ) + + lv_on_diff_ devs[@] "${@}" } mirror_images_on() { diff --git a/test/lib/get.sh b/test/lib/get.sh index 7d7c3d2..7b97c06 100644 --- a/test/lib/get.sh +++ b/test/lib/get.sh @@ -43,6 +43,40 @@ lv_devices() { lv_field "$1" devices -a "${@:2}" | sed 's/([^)]*)//g; s/,/\n/g' } +lv_field_lv_() { + lv_field "$1" "$2" -a --unbuffered | sed 's/\[//; s/]//' +} + +lv_tree_devices_() { + local lv="$1/$2" + local type=$(lv_field "$lv" segtype -a --unbuffered | head -n 1) + local orig=$(lv_field_lv_ "$lv" origin) + # FIXME: should we count in also origins ? + #test -z "$orig" || lv_tree_devices_ $1 $orig + case "$type" in + linear|striped) + lv_devices "$lv" + ;; + mirror|raid*) + local log=$(lv_field_lv_ "$lv" mirror_log) + test -z "$log" || lv_tree_devices_ "$1" "$log" + for i in $(lv_devices "$lv") + do lv_tree_devices_ "$1" "$i"; done + ;; + thin) + lv_tree_devices_ "$1" "$(lv_field_lv_ $lv pool_lv)" + ;; + thin-pool) + lv_tree_devices_ "$1" "$(lv_field_lv_ $lv data_lv)" + lv_tree_devices_ "$1" "$(lv_field_lv_ $lv metadata_lv)" + ;; + esac +} + +lv_tree_devices() { + lv_tree_devices_ "$@" | sort | uniq +} + #set -x unset LVM_VALGRIND "$@" diff --git a/test/shell/pvmove-all-segtypes.sh b/test/shell/pvmove-all-segtypes.sh index 8ea9d39..fcd8913 100644 --- a/test/shell/pvmove-all-segtypes.sh +++ b/test/shell/pvmove-all-segtypes.sh @@ -9,115 +9,16 @@ # along with this program; if not, write to the Free Software Foundation, # Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA -test_description="ensure pvmove works with all segment types" +test_description="ensure pvmove works with all common segment types" . lib/test +test -e LOCAL_CLVMD && skip which mkfs.ext2 || skip which md5sum || skip -test -e LOCAL_CLVMD && skip - -# --------------------------------------------------------------------- -# Utilities -lvdev_() { - echo "$DM_DEV_DIR/$1/$2" -} - -# lv_is_on_pvs <VG> <LV> <PV1> <PV2> ... -lv_is_on_pvs() { - local vg=$1 - local lv_list=( $2 ) - local new_lv_list=() - local run_again=true - - shift 2 - - while $run_again; do - run_again=false - for tmplv in ${lv_list[*]}; do - if [[ $(lvs -a --noheadings -o attr $vg/$tmplv) =~ ^[[:space:]]*r.* ]] || - [[ $(lvs -a --noheadings -o attr $vg/$tmplv) =~ ^[[:space:]]*R.* ]]; then - # Expand RAID - echo "$tmplv is a RAID LV" - new_lv_list=( ${new_lv_list[*]} `lvs -a --noheadings -o name | grep ${tmplv}_r` ) - run_again=true - elif [[ $(lvs -a --noheadings -o attr $vg/$tmplv) =~ ^[[:space:]]*m.* ]] || - [[ $(lvs -a --noheadings -o attr $vg/$tmplv) =~ ^[[:space:]]*M.* ]]; then - # Expand Mirror - echo "$tmplv is a mirror LV" - new_lv_list=( ${new_lv_list[*]} `lvs -a --noheadings -o devices $vg/$tmplv | sed s/,/' '/g` ) - new_lv_list=( ${new_lv_list[*]} `lvs -a --noheadings -o mirror_log $vg/$tmplv | sed s/,/' '/g` ) - run_again=true - elif [[ $(lvs -a --noheadings -o attr $vg/$tmplv) =~ ^[[:space:]]*t.* ]]; then - # Expand Thin Pool - echo "$tmplv is a thin-pool LV" - new_lv_list=( ${new_lv_list[*]} `lvs -a --noheadings -o data_lv,metadata_lv $vg/$tmplv | sed s/,/' '/g` ) - run_again=true - elif [[ $(lvs -a --noheadings -o attr $vg/$tmplv) =~ ^[[:space:]]*V.* ]]; then - # Expand Thin-LV - echo "$tmplv is a thin-LV" - new_lv_list=( ${new_lv_list[*]} `lvs -a --noheadings -o pool_lv $vg/$tmplv | sed s/,/' '/g` ) - run_again=true - elif [[ $(lvs -a --noheadings -o attr $vg/$tmplv) =~ ^[[:space:]]*e.* ]] && - lvs -a --noheadings -o devices $vg/$tmplv | grep $tmplv; then - # Expand Thin Pool Meta, which is also RAID - echo "$tmplv is a thinpool (RAID) metadata" - new_lv_list=( ${new_lv_list[*]} `lvs -a --noheadings -o name | grep ${tmplv}_r` ) - run_again=true - else - new_lv_list=( ${new_lv_list[*]} $tmplv ) - fi - done - printf "\nOld LV list: ${lv_list[*]}\n\n" - - lv_list=() - for tmplv in ${new_lv_list[*]}; do - lv_list=( ${lv_list[*]} $(echo $tmplv | sed s/$.*$/''/ | sed s/\\[/''/ | sed s/]/''/) ) - done - printf "\nExpanded LV list: ${lv_list[*]}\n\n" - - new_lv_list=() - done - - [ -e out-$$ ] && rm out-$$ - for tmplv in ${lv_list[*]}; do - lvs -a -odevices --noheadings $vg/$tmplv | sed 's/,/\n/g' >> out-$$ - done - - #is on all specified devs - for d in $*; do - if ! grep "$d(" out-$$; then - cat out-$$ - false - fi - done - - #isn't on any other dev (we are set -e remember) - cat out-$$ - [ -e tmp-$$ ] && rm tmp-$$ - for d in $*; do - echo "Removing $d" - ! grep -v "$d(" out-$$ >> tmp-$$ - mv tmp-$$ out-$$ - cat out-$$ - done - # out-$$ must be empty - no additional devs allowed - [ ! -s out-$$ ] - rm out-$$ - return 0 -} - -save_dev_sum_() { - mkfs.ext2 $1 > /dev/null && md5sum $1 > md5.$(basename $1) -} - -check_dev_sum_() { - md5sum -c md5.$(basename $1) -} -# --------------------------------------------------------------------- - -aux prepare_vg 5 30 +aux prepare_pvs 5 20 +vgcreate -c n -s 256k $vg $(cat DEVICES) # Each of the following tests does: # 1) Create two LVs - one linear and one other segment type @@ -125,181 +26,68 @@ aux prepare_vg 5 30 # 2) Move both LVs together # 3) Move only the second LV by name -printf "##\n# Testing pvmove of linear LV\n##\n" +# Testing pvmove of linear LV lvcreate -l 2 -n ${lv1}_foo $vg "$dev1" lvcreate -l 2 -n $lv1 $vg "$dev1" -lv_is_on_pvs $vg ${lv1}_foo "$dev1" -lv_is_on_pvs $vg $lv1 "$dev1" -save_dev_sum_ $(lvdev_ $vg $lv1) +check lv_tree_on $vg ${lv1}_foo "$dev1" +check lv_tree_on $vg $lv1 "$dev1" +aux mkdev_md5sum $vg $lv1 pvmove "$dev1" "$dev5" -lv_is_on_pvs $vg ${lv1}_foo "$dev5" -lv_is_on_pvs $vg $lv1 "$dev5" -check_dev_sum_ $(lvdev_ $vg $lv1) +check lv_tree_on $vg ${lv1}_foo "$dev5" +check lv_tree_on $vg $lv1 "$dev5" +check dev_md5sum $vg $lv1 pvmove -n $lv1 "$dev5" "$dev4" -lv_is_on_pvs $vg $lv1 "$dev4" -lv_is_on_pvs $vg ${lv1}_foo "$dev5" -check_dev_sum_ $(lvdev_ $vg $lv1) +check lv_tree_on $vg $lv1 "$dev4" +check lv_tree_on $vg ${lv1}_foo "$dev5" +check dev_md5sum $vg $lv1 lvremove -ff $vg -printf "##\n# Testing pvmove of stripe LV\n##\n" +# Testing pvmove of stripe LV lvcreate -l 2 -n ${lv1}_foo $vg "$dev1" lvcreate -l 4 -i 2 -n $lv1 $vg "$dev1" "$dev2" -lv_is_on_pvs $vg ${lv1}_foo "$dev1" -lv_is_on_pvs $vg $lv1 "$dev1" "$dev2" -save_dev_sum_ $(lvdev_ $vg $lv1) +check lv_tree_on $vg ${lv1}_foo "$dev1" +check lv_tree_on $vg $lv1 "$dev1" "$dev2" +aux mkdev_md5sum $vg $lv1 pvmove "$dev1" "$dev5" -lv_is_on_pvs $vg ${lv1}_foo "$dev5" -lv_is_on_pvs $vg $lv1 "$dev2" "$dev5" -check_dev_sum_ $(lvdev_ $vg $lv1) +check lv_tree_on $vg ${lv1}_foo "$dev5" +check lv_tree_on $vg $lv1 "$dev2" "$dev5" +check dev_md5sum $vg $lv1 pvmove -n $lv1 "$dev5" "$dev4" -lv_is_on_pvs $vg $lv1 "$dev2" "$dev4" -lv_is_on_pvs $vg ${lv1}_foo "$dev5" -check_dev_sum_ $(lvdev_ $vg $lv1) +check lv_tree_on $vg $lv1 "$dev2" "$dev4" +check lv_tree_on $vg ${lv1}_foo "$dev5" +check dev_md5sum $vg $lv1 lvremove -ff $vg -printf "##\n# Testing pvmove of mirror LV\n##\n" +# Testing pvmove of mirror LV lvcreate -l 2 -n ${lv1}_foo $vg "$dev1" lvcreate -l 2 --type mirror -m 1 -n $lv1 $vg "$dev1" "$dev2" -lv_is_on_pvs $vg ${lv1}_foo "$dev1" -lv_is_on_pvs $vg $lv1 "$dev1" "$dev2" -save_dev_sum_ $(lvdev_ $vg $lv1) +check lv_tree_on $vg ${lv1}_foo "$dev1" +check lv_tree_on $vg $lv1 "$dev1" "$dev2" +aux mkdev_md5sum $vg $lv1 pvmove "$dev1" "$dev5" -lv_is_on_pvs $vg ${lv1}_foo "$dev5" -lv_is_on_pvs $vg $lv1 "$dev2" "$dev5" -check_dev_sum_ $(lvdev_ $vg $lv1) +check lv_tree_on $vg ${lv1}_foo "$dev5" +check lv_tree_on $vg $lv1 "$dev2" "$dev5" +check dev_md5sum $vg $lv1 pvmove -n $lv1 "$dev5" "$dev4" -lv_is_on_pvs $vg $lv1 "$dev2" "$dev4" -lv_is_on_pvs $vg ${lv1}_foo "$dev5" -check_dev_sum_ $(lvdev_ $vg $lv1) +check lv_tree_on $vg $lv1 "$dev2" "$dev4" +check lv_tree_on $vg ${lv1}_foo "$dev5" +check dev_md5sum $vg $lv1 lvremove -ff $vg # Dummy LV and snap share dev1, while origin is on dev2 -printf "##\n# Testing pvmove of snapshot LV\n##\n" +# Testing pvmove of snapshot LV lvcreate -l 2 -n ${lv1}_foo $vg "$dev1" lvcreate -l 2 -n $lv1 $vg "$dev2" lvcreate -s $vg/$lv1 -l 2 -n snap "$dev1" -lv_is_on_pvs $vg ${lv1}_foo "$dev1" -lv_is_on_pvs $vg snap "$dev1" -save_dev_sum_ $(lvdev_ $vg snap) +check lv_tree_on $vg ${lv1}_foo "$dev1" +check lv_tree_on $vg snap "$dev1" +aux mkdev_md5sum $vg snap pvmove "$dev1" "$dev5" -lv_is_on_pvs $vg ${lv1}_foo "$dev5" -lv_is_on_pvs $vg snap "$dev5" -check_dev_sum_ $(lvdev_ $vg snap) +check lv_tree_on $vg ${lv1}_foo "$dev5" +check lv_tree_on $vg snap "$dev5" +check dev_md5sum $vg snap pvmove -n snap "$dev5" "$dev4" -lv_is_on_pvs $vg snap "$dev4" -lv_is_on_pvs $vg ${lv1}_foo "$dev5" -check_dev_sum_ $(lvdev_ $vg snap) +check lv_tree_on $vg snap "$dev4" +check lv_tree_on $vg ${lv1}_foo "$dev5" +check dev_md5sum $vg snap lvremove -ff $vg - -## -# RAID -## -if aux target_at_least dm-raid 1 3 5; then - printf "##\n# Testing pvmove of RAID1 LV\n##\n" - lvcreate -l 2 -n ${lv1}_foo $vg "$dev1" - lvcreate -l 2 --type raid1 -m 1 -n $lv1 $vg "$dev1" "$dev2" - lv_is_on_pvs $vg ${lv1}_foo "$dev1" - lv_is_on_pvs $vg $lv1 "$dev1" "$dev2" - save_dev_sum_ $(lvdev_ $vg $lv1) - pvmove "$dev1" "$dev5" - lv_is_on_pvs $vg ${lv1}_foo "$dev5" - lv_is_on_pvs $vg $lv1 "$dev2" "$dev5" - check_dev_sum_ $(lvdev_ $vg $lv1) - pvmove -n $lv1 "$dev5" "$dev4" - lv_is_on_pvs $vg $lv1 "$dev2" "$dev4" - lv_is_on_pvs $vg ${lv1}_foo "$dev5" - check_dev_sum_ $(lvdev_ $vg $lv1) - lvremove -ff $vg - - printf "##\n# Testing pvmove of RAID10 LV\n##\n" - lvcreate -l 2 -n ${lv1}_foo $vg "$dev1" - lvcreate -l 4 --type raid10 -i 2 -m 1 -n $lv1 $vg \ - "$dev1" "$dev2" "$dev3" "$dev4" - lv_is_on_pvs $vg ${lv1}_foo "$dev1" - lv_is_on_pvs $vg $lv1 "$dev1" "$dev2" "$dev3" "$dev4" - save_dev_sum_ $(lvdev_ $vg $lv1) - pvmove "$dev1" "$dev5" - lv_is_on_pvs $vg ${lv1}_foo "$dev5" - lv_is_on_pvs $vg $lv1 "$dev2" "$dev3" "$dev4" "$dev5" - check_dev_sum_ $(lvdev_ $vg $lv1) - pvmove -n $lv1 "$dev5" "$dev1" - lv_is_on_pvs $vg $lv1 "$dev1" "$dev2" "$dev3" "$dev4" - lv_is_on_pvs $vg ${lv1}_foo "$dev5" - check_dev_sum_ $(lvdev_ $vg $lv1) - lvremove -ff $vg - - printf "##\n# Testing pvmove of RAID5 LV\n##\n" - lvcreate -l 2 -n ${lv1}_foo $vg "$dev1" - lvcreate -l 4 --type raid5 -i 2 -n $lv1 $vg \ - "$dev1" "$dev2" "$dev3" - lv_is_on_pvs $vg ${lv1}_foo "$dev1" - lv_is_on_pvs $vg $lv1 "$dev1" "$dev2" "$dev3" - save_dev_sum_ $(lvdev_ $vg $lv1) - pvmove "$dev1" "$dev5" - lv_is_on_pvs $vg ${lv1}_foo "$dev5" - lv_is_on_pvs $vg $lv1 "$dev2" "$dev3" "$dev5" - check_dev_sum_ $(lvdev_ $vg $lv1) - pvmove -n $lv1 "$dev5" "$dev4" - lv_is_on_pvs $vg $lv1 "$dev2" "$dev3" "$dev4" - lv_is_on_pvs $vg ${lv1}_foo "$dev5" - check_dev_sum_ $(lvdev_ $vg $lv1) - lvremove -ff $vg -fi - -## -# Thin -## -if aux target_at_least dm-thin-pool 1 8 0; then - - printf "##\n# Testing pvmove of thin-pool LV\n##\n" - lvcreate -l 2 -n ${lv1}_foo $vg "$dev1" - lvcreate -T $vg/$lv1 -l 4 "$dev1" - lv_is_on_pvs $vg ${lv1}_foo "$dev1" - lv_is_on_pvs $vg $lv1 "$dev1" - save_dev_sum_ $(lvdev_ $vg $lv1) - pvmove "$dev1" "$dev5" - lv_is_on_pvs $vg ${lv1}_foo "$dev5" - lv_is_on_pvs $vg $lv1 "$dev5" - check_dev_sum_ $(lvdev_ $vg $lv1) - pvmove -n $lv1 "$dev5" "$dev4" - lv_is_on_pvs $vg $lv1 "$dev4" - lv_is_on_pvs $vg ${lv1}_foo "$dev5" - check_dev_sum_ $(lvdev_ $vg $lv1) - lvremove -ff $vg - - printf "##\n# Testing pvmove of thin LV\n##\n" - lvcreate -l 2 -n ${lv1}_foo $vg "$dev1" - lvcreate -T $vg/${lv1}_pool -l 4 -V 8 -n $lv1 "$dev1" - lv_is_on_pvs $vg ${lv1}_foo "$dev1" - lv_is_on_pvs $vg $lv1 "$dev1" - save_dev_sum_ $(lvdev_ $vg $lv1) - pvmove "$dev1" "$dev5" - lv_is_on_pvs $vg ${lv1}_foo "$dev5" - lv_is_on_pvs $vg $lv1 "$dev5" - check_dev_sum_ $(lvdev_ $vg $lv1) - pvmove -n $lv1 "$dev5" "$dev4" - lv_is_on_pvs $vg $lv1 "$dev4" - lv_is_on_pvs $vg ${lv1}_foo "$dev5" - check_dev_sum_ $(lvdev_ $vg $lv1) - lvremove -ff $vg - - printf "##\n# Testing pvmove of thin LV on RAID\n##\n" - lvcreate -l 2 -n ${lv1}_foo $vg "$dev1" - lvcreate --type raid1 -m 1 -l 4 -n ${lv1}_raid1_pool $vg "$dev1" "$dev2" - lvcreate --type raid1 -m 1 -l 4 -n ${lv1}_raid1_meta $vg "$dev1" "$dev2" - lvconvert --thinpool $vg/${lv1}_raid1_pool \ - --poolmetadata ${lv1}_raid1_meta - lvcreate -T $vg/${lv1}_raid1_pool -V 8 -n $lv1 - lv_is_on_pvs $vg ${lv1}_foo "$dev1" - lv_is_on_pvs $vg $lv1 "$dev1" "$dev2" - save_dev_sum_ $(lvdev_ $vg $lv1) - pvmove "$dev1" "$dev5" - lv_is_on_pvs $vg ${lv1}_foo "$dev5" - lv_is_on_pvs $vg $lv1 "$dev2" "$dev5" - check_dev_sum_ $(lvdev_ $vg $lv1) - pvmove -n $lv1 "$dev5" "$dev4" - lv_is_on_pvs $vg $lv1 "$dev2" "$dev4" - lv_is_on_pvs $vg ${lv1}_foo "$dev5" - check_dev_sum_ $(lvdev_ $vg $lv1) - lvremove -ff $vg -fi diff --git a/test/shell/pvmove-basic.sh b/test/shell/pvmove-basic.sh index e95fc36..69244be 100644 --- a/test/shell/pvmove-basic.sh +++ b/test/shell/pvmove-basic.sh @@ -1,5 +1,5 @@ #!/bin/sh -# Copyright (C) 2008-2012 Red Hat, Inc. All rights reserved. +# Copyright (C) 2008-2013 Red Hat, Inc. All rights reserved. # Copyright (C) 2007 NEC Corporation # # This copyrighted material is made available to anyone wishing to use, @@ -20,83 +20,67 @@ which md5sum || skip # --------------------------------------------------------------------- # Utilities -lvdev_() { - echo "$DM_DEV_DIR/$1/$2" +create_vg_() { + vgcreate -c n -s 128k $vg $(cat DEVICES) } -lv_is_on_() { - local lv=$1 #allready vg/lv - shift 1 - lvs -a -odevices --noheadings $lv | sed 's/,/\n/g' > out -#is on all specified devs - for d in $*; do grep "$d(" out; done -#isn't on any other dev (we are set -e remember) - for d in $*; do ! grep -v "$d(" out; done - return 0 +# --------------------------------------------------------------------- +# Common environment setup/cleanup for each sub testcases +prepare_lvs_() { + lvcreate -l2 -n $lv1 $vg "$dev1" + check lv_on $vg $lv1 "$dev1" + lvcreate -l9 -i3 -n $lv2 $vg "$dev2" "$dev3" "$dev4" + check lv_on $vg $lv2 "$dev2" "$dev3" "$dev4" + lvextend -l+2 $vg/$lv1 "$dev2" + check lv_on $vg $lv1 "$dev1" "$dev2" + lvextend -l+2 $vg/$lv1 "$dev3" + lvextend -l+2 $vg/$lv1 "$dev1" + check lv_on $vg $lv1 "$dev1" "$dev2" "$dev3" + lvcreate -l1 -n $lv3 $vg "$dev2" + check lv_on $vg $lv3 "$dev2" + aux mkdev_md5sum $vg $lv1 + aux mkdev_md5sum $vg $lv2 + aux mkdev_md5sum $vg $lv3 + get lv_devices "$vg/$lv1" > "${lv1}_devs" + get lv_devices "$vg/$lv2" > "${lv2}_devs" + get lv_devices "$vg/$lv3" > "${lv3}_devs" + lvs -a -o name,size,seg_pe_ranges $vg + vgcfgbackup -f bak-$$ $vg } -save_dev_sum_() { - mkfs.ext2 $1 > /dev/null && md5sum $1 > md5.$(basename $1) +# Restore metadata content, since data are pvmove-ed +# original content should be preserved +restore_lvs_() { + vgcfgrestore -f bak-$$ $vg + vgchange -ay $vg } -check_dev_sum_() { - md5sum -c md5.$(basename $1) +lvs_not_changed_() { + for i in "${@}"; do + get lv_devices "$vg/$i" | tee out + diff "${i}_devs" out || \ + (cat "${i}_devs"; die "Devices for LV $vg/$i differs!") + done } -create_vg_() { - vgcreate -c n -s 128k $vg $(cat DEVICES) +check_and_cleanup_lvs_() { + check dev_md5sum $vg $lv1 + check dev_md5sum $vg $lv2 + check dev_md5sum $vg $lv3 + get lv_field $vg name >out + not grep ^pvmove out + vgchange -an $vg + lvremove -ff $vg + (dm_table | not grep $vg) || \ + die "ERROR: lvremove did leave some some mappings in DM behind!" } + # --------------------------------------------------------------------- # Initialize PVs and VGs -#aux prepare_vg 5 30 aux prepare_pvs 5 5 create_vg_ -# --------------------------------------------------------------------- -# Common environment setup/cleanup for each sub testcases -FIRST="" - -prepare_lvs_() { - lvcreate -l2 -n $lv1 $vg "$dev1" - test -z "$FIRST" && lv_is_on_ $vg/$lv1 "$dev1" - lvcreate -l9 -i3 -n $lv2 $vg "$dev2" "$dev3" "$dev4" - test -z "$FIRST" && lv_is_on_ $vg/$lv2 "$dev2" "$dev3" "$dev4" - lvextend -l+2 $vg/$lv1 "$dev2" - test -z "$FIRST" && lv_is_on_ $vg/$lv1 "$dev1" "$dev2" - lvextend -l+2 $vg/$lv1 "$dev3" - test -z "$FIRST" && lv_is_on_ $vg/$lv1 "$dev1" "$dev2" "$dev3" - lvextend -l+2 $vg/$lv1 "$dev1" - test -z "$FIRST" && lv_is_on_ $vg/$lv1 "$dev1" "$dev2" "$dev3" "$dev1" - lvcreate -l1 -n $lv3 $vg "$dev2" - test -z "$FIRST" && lv_is_on_ $vg/$lv3 "$dev2" - save_dev_sum_ $(lvdev_ $vg $lv1) - save_dev_sum_ $(lvdev_ $vg $lv2) - save_dev_sum_ $(lvdev_ $vg $lv3) - if test -z "$FIRST" ; then - get lv_field $vg/$lv1 devices > ${lv1}_devs - get lv_field $vg/$lv2 devices > ${lv2}_devs - get lv_field $vg/$lv3 devices > ${lv3}_devs - fi - FIRST=done -} - -lv_not_changed_() { - get lv_field $1 devices > out - diff $(basename $1)_devs out -} - -check_and_cleanup_lvs_() { - lvs -a -o+devices $vg - check_dev_sum_ $(lvdev_ $vg $lv1) - check_dev_sum_ $(lvdev_ $vg $lv2) - check_dev_sum_ $(lvdev_ $vg $lv3) - lvs -a -o name $vg > out && ! grep ^pvmove out - lvremove -ff $vg - (dm_table | not grep $vg) || \ - die "ERROR: lvremove did leave some some mappings in DM behind!" -} - #COMM "check environment setup/cleanup" prepare_lvs_ check_and_cleanup_lvs_ @@ -108,262 +92,243 @@ check_and_cleanup_lvs_ # filter by LV #COMM "only specified LV is moved: from pv2 to pv5 only for lv1" -prepare_lvs_ +restore_lvs_ pvmove -i1 -n $vg/$lv1 "$dev2" "$dev5" -lv_is_on_ $vg/$lv1 "$dev1" "$dev5" "$dev3" "$dev1" -lv_not_changed_ $vg/$lv2 -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv1 "$dev1" "$dev5" "$dev3" +lvs_not_changed_ $lv2 $lv3 check_and_cleanup_lvs_ # --- # segments in a LV #COMM "the 1st seg of 3-segs LV is moved: from pv1 of lv1 to pv4" -prepare_lvs_ +restore_lvs_ pvmove -i0 -n $vg/$lv1 "$dev1" "$dev4" -lv_is_on_ $vg/$lv1 "$dev4" "$dev2" "$dev3" "$dev4" -lv_not_changed_ $vg/$lv2 -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv1 "$dev4" "$dev2" "$dev3" +lvs_not_changed_ $lv2 $lv3 check_and_cleanup_lvs_ #COMM "the 2nd seg of 3-segs LV is moved: from pv2 of lv1 to pv4" -prepare_lvs_ +restore_lvs_ pvmove -i0 -n $vg/$lv1 "$dev2" "$dev4" -lv_is_on_ $vg/$lv1 "$dev1" "$dev4" "$dev3" "$dev1" -lv_not_changed_ $vg/$lv2 -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv1 "$dev1" "$dev4" "$dev3" +lvs_not_changed_ $lv2 $lv3 check_and_cleanup_lvs_ #COMM "the 3rd seg of 3-segs LV is moved: from pv3 of lv1 to pv4" -prepare_lvs_ +restore_lvs_ pvmove -i0 -n $vg/$lv1 "$dev3" "$dev4" -lv_is_on_ $vg/$lv1 "$dev1" "$dev2" "$dev4" "$dev1" -lv_not_changed_ $vg/$lv2 -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv1 "$dev1" "$dev2" "$dev4" +lvs_not_changed_ $lv2 $lv3 check_and_cleanup_lvs_ # --- # multiple LVs matching #COMM "1 out of 3 LVs is moved: from pv4 to pv5" -prepare_lvs_ +restore_lvs_ pvmove -i0 "$dev4" "$dev5" -lv_not_changed_ $vg/$lv1 -lv_is_on_ $vg/$lv2 "$dev2" "$dev3" "$dev5" -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv2 "$dev2" "$dev3" "$dev5" +lvs_not_changed_ $lv1 $lv3 check_and_cleanup_lvs_ #COMM "2 out of 3 LVs are moved: from pv3 to pv5" -prepare_lvs_ +restore_lvs_ pvmove -i0 "$dev3" "$dev5" -lv_is_on_ $vg/$lv1 "$dev1" "$dev2" "$dev5" "$dev1" -lv_is_on_ $vg/$lv2 "$dev2" "$dev5" "$dev4" -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv1 "$dev1" "$dev2" "$dev5" +check lv_on $vg $lv2 "$dev2" "$dev5" "$dev4" +lvs_not_changed_ $lv3 check_and_cleanup_lvs_ #COMM "3 out of 3 LVs are moved: from pv2 to pv5" -prepare_lvs_ +restore_lvs_ pvmove -i0 "$dev2" "$dev5" -lv_is_on_ $vg/$lv1 "$dev1" "$dev5" "$dev3" "$dev1" -lv_is_on_ $vg/$lv2 "$dev5" "$dev3" "$dev4" -lv_is_on_ $vg/$lv3 "$dev5" +check lv_on $vg $lv1 "$dev1" "$dev5" "$dev3" +check lv_on $vg $lv2 "$dev5" "$dev3" "$dev4" +check lv_on $vg $lv3 "$dev5" check_and_cleanup_lvs_ # --- # areas of striping #COMM "move the 1st stripe: from pv2 of lv2 to pv1" -prepare_lvs_ +restore_lvs_ pvmove -i0 -n $vg/$lv2 "$dev2" "$dev1" -lv_not_changed_ $vg/$lv1 -lv_is_on_ $vg/$lv2 "$dev1" "$dev3" "$dev4" -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv2 "$dev1" "$dev3" "$dev4" +lvs_not_changed_ $lv1 $lv3 check_and_cleanup_lvs_ #COMM "move the 2nd stripe: from pv3 of lv2 to pv1" -prepare_lvs_ +restore_lvs_ pvmove -i0 -n $vg/$lv2 "$dev3" "$dev1" -lv_not_changed_ $vg/$lv1 -lv_is_on_ $vg/$lv2 "$dev2" "$dev1" "$dev4" -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv2 "$dev2" "$dev1" "$dev4" +lvs_not_changed_ $lv1 $lv3 check_and_cleanup_lvs_ #COMM "move the 3rd stripe: from pv4 of lv2 to pv1" -prepare_lvs_ +restore_lvs_ pvmove -i0 -n $vg/$lv2 "$dev4" "$dev1" -lv_not_changed_ $vg/$lv1 -lv_is_on_ $vg/$lv2 "$dev2" "$dev3" "$dev1" -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv2 "$dev2" "$dev3" "$dev1" +lvs_not_changed_ $lv1 $lv3 check_and_cleanup_lvs_ # --- # partial segment match (source segment splitted) #COMM "match to the start of segment:from pv2:0-0 to pv5" -prepare_lvs_ +restore_lvs_ pvmove -i0 "$dev2":0-0 "$dev5" -lv_not_changed_ $vg/$lv1 -lv_is_on_ $vg/$lv2 "$dev5" "$dev2" "$dev3" "$dev4" -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv2 "$dev5" "$dev2" "$dev3" "$dev4" +lvs_not_changed_ $lv1 $lv3 check_and_cleanup_lvs_ - +#exit 0 #COMM "match to the middle of segment: from pv2:1-1 to pv5" -prepare_lvs_ +restore_lvs_ pvmove -i0 "$dev2":1-1 "$dev5" -lv_not_changed_ $vg/$lv1 -lv_is_on_ $vg/$lv2 "$dev2" "$dev5" "$dev2" "$dev3" "$dev4" -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv2 "$dev2" "$dev3" "$dev4" "$dev5" +lvs_not_changed_ $lv1 $lv3 check_and_cleanup_lvs_ #COMM "match to the end of segment: from pv2:2-2 to pv5" -prepare_lvs_ +restore_lvs_ pvmove -i0 "$dev2":2-2 "$dev5" -lv_not_changed_ $vg/$lv1 -lv_is_on_ $vg/$lv2 "$dev2" "$dev5" "$dev3" "$dev4" -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv2 "$dev2" "$dev5" "$dev3" "$dev4" +lvs_not_changed_ $lv1 $lv3 check_and_cleanup_lvs_ # --- # destination segment splitted #COMM "no destination split: from pv2:0-2 to pv5" -prepare_lvs_ +restore_lvs_ pvmove -i0 "$dev2":0-2 "$dev5" -lv_not_changed_ $vg/$lv1 -lv_is_on_ $vg/$lv2 "$dev5" "$dev3" "$dev4" -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv2 "$dev5" "$dev3" "$dev4" +lvs_not_changed_ $lv1 $lv3 check_and_cleanup_lvs_ #COMM "destination split into 2: from pv2:0-2 to pv5:5-5 and pv4:5-6" -prepare_lvs_ +restore_lvs_ pvmove -i0 --alloc anywhere "$dev2":0-2 "$dev5":5-5 "$dev4":5-6 -lv_not_changed_ $vg/$lv1 -lv_is_on_ $vg/$lv2 "$dev5" "$dev4" "$dev3" "$dev4" -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv2 "$dev5" "$dev4" "$dev3" +lvs_not_changed_ $lv1 $lv3 check_and_cleanup_lvs_ #COMM "destination split into 3: from pv2:0-2 to {pv3,4,5}:5-5" -prepare_lvs_ +restore_lvs_ pvmove -i0 --alloc anywhere "$dev2":0-2 "$dev3":5-5 "$dev4":5-5 "$dev5":5-5 -lv_not_changed_ $vg/$lv1 -lv_is_on_ $vg/$lv2 "$dev3" "$dev4" "$dev5" "$dev3" "$dev4" -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv2 "$dev3" "$dev4" "$dev5" +lvs_not_changed_ $lv1 $lv3 check_and_cleanup_lvs_ # --- # alloc policy (anywhere, contiguous) with both success and failure cases #COMM "alloc normal on same PV for source and destination: from pv3:0-2 to pv3:5-7" -prepare_lvs_ +restore_lvs_ not pvmove -i0 "$dev3":0-2 "$dev3":5-7 # "(cleanup previous test)" -lv_not_changed_ $vg/$lv1 -lv_not_changed_ $vg/$lv2 -lv_not_changed_ $vg/$lv3 +lvs_not_changed_ $lv1 $lv2 $lv3 check_and_cleanup_lvs_ #COMM "alloc anywhere on same PV for source and destination: from pv3:0-2 to pv3:5-7" -prepare_lvs_ +restore_lvs_ pvmove -i0 --alloc anywhere "$dev3":0-2 "$dev3":5-7 -lv_not_changed_ $vg/$lv1 -lv_is_on_ $vg/$lv2 "$dev2" "$dev3" "$dev4" -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv2 "$dev2" "$dev3" "$dev4" +lvs_not_changed_ $lv1 $lv3 check_and_cleanup_lvs_ #COMM "alloc anywhere but better area available: from pv3:0-2 to pv3:5-7 or pv5:5-6,pv4:5-5" -prepare_lvs_ +restore_lvs_ +#lvs -a -o name,size,seg_pe_ranges $vg +#LV2 1.12m @TESTDIR@/dev/mapper/@PREFIX@pv2:0-2 @TESTDIR@/dev/mapper/@PREFIX@pv3:0-2 @TESTDIR@/dev/mapper/@PREFIX@pv4:0-2 + pvmove -i0 --alloc anywhere "$dev3":0-2 "$dev3":5-7 "$dev5":5-6 "$dev4":5-5 -lv_not_changed_ $vg/$lv1 -#lv_is_on_ $vg/$lv2 "$dev2" "$dev5" "$dev4" "$dev4" -lv_not_changed_ $vg/$lv3 + +#lvs -a -o name,size,seg_pe_ranges $vg +# Hmm is this correct ? - why pv2 is split +#LV2 1.12m @TESTDIR@/dev/mapper/@PREFIX@pv2:0-1 @TESTDIR@/dev/mapper/@PREFIX@pv5:5-6 @TESTDIR@/dev/mapper/@PREFIX@pv4:0-1 +#LV2 1.12m @TESTDIR@/dev/mapper/@PREFIX@pv2:2-2 @TESTDIR@/dev/mapper/@PREFIX@pv3:5-5 @TESTDIR@/dev/mapper/@PREFIX@pv4:2-2 +check lv_on $vg $lv2 "$dev2" "$dev3" "$dev4" "$dev5" +lvs_not_changed_ $lv1 $lv3 check_and_cleanup_lvs_ #COMM "alloc contiguous but area not available: from pv2:0-2 to pv5:5-5 and pv4:5-6" -prepare_lvs_ +restore_lvs_ not pvmove -i0 --alloc contiguous "$dev2":0-2 "$dev5":5-5 "$dev4":5-6 # "(cleanup previous test)" -lv_not_changed_ $vg/$lv1 -lv_not_changed_ $vg/$lv2 -lv_not_changed_ $vg/$lv3 +lvs_not_changed_ $lv1 $lv2 $lv3 check_and_cleanup_lvs_ #COMM "alloc contiguous and contiguous area available: from pv2:0-2 to pv5:0-0,pv5:3-5 and pv4:5-6" -prepare_lvs_ +restore_lvs_ pvmove -i0 --alloc contiguous "$dev2":0-2 "$dev5":0-0 "$dev5":3-5 "$dev4":5-6 -lv_not_changed_ $vg/$lv1 -lv_is_on_ $vg/$lv2 "$dev5" "$dev3" "$dev4" -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv2 "$dev5" "$dev3" "$dev4" +lvs_not_changed_ $lv1 $lv3 check_and_cleanup_lvs_ # --- # multiple segments in a LV #COMM "multiple source LVs: from pv3 to pv5" -prepare_lvs_ +restore_lvs_ pvmove -i0 "$dev3" "$dev5" -lv_is_on_ $vg/$lv1 "$dev1" "$dev2" "$dev5" -lv_is_on_ $vg/$lv2 "$dev2" "$dev5" "$dev4" -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv1 "$dev1" "$dev2" "$dev5" +check lv_on $vg $lv2 "$dev2" "$dev5" "$dev4" +lvs_not_changed_ $lv3 check_and_cleanup_lvs_ # --- # move inactive LV #COMM "move inactive LV: from pv2 to pv5" -prepare_lvs_ +restore_lvs_ lvchange -an $vg/$lv1 lvchange -an $vg/$lv3 pvmove -i0 "$dev2" "$dev5" -lv_is_on_ $vg/$lv1 "$dev1" "$dev5" "$dev3" -lv_is_on_ $vg/$lv2 "$dev5" "$dev3" "$dev4" -lv_is_on_ $vg/$lv3 "$dev5" +check lv_on $vg $lv1 "$dev1" "$dev5" "$dev3" +check lv_on $vg $lv2 "$dev5" "$dev3" "$dev4" +check lv_on $vg $lv3 "$dev5" check_and_cleanup_lvs_ # --- # other failure cases #COMM "no PEs to move: from pv3 to pv1" -prepare_lvs_ +restore_lvs_ pvmove -i0 "$dev3" "$dev1" not pvmove -i0 "$dev3" "$dev1" # "(cleanup previous test)" -lv_is_on_ $vg/$lv1 "$dev1" "$dev2" "$dev1" -lv_is_on_ $vg/$lv2 "$dev2" "$dev1" "$dev4" -lv_not_changed_ $vg/$lv3 +check lv_on $vg $lv1 "$dev1" "$dev2" "$dev1" +check lv_on $vg $lv2 "$dev2" "$dev1" "$dev4" +lvs_not_changed_ $lv3 check_and_cleanup_lvs_ #COMM "no space available: from pv2:0-0 to pv1:0-0" -prepare_lvs_ +restore_lvs_ not pvmove -i0 "$dev2":0-0 "$dev1":0-0 # "(cleanup previous test)" -lv_not_changed_ $vg/$lv1 -lv_not_changed_ $vg/$lv2 -lv_not_changed_ $vg/$lv3 +lvs_not_changed_ $lv1 $lv2 $lv3 check_and_cleanup_lvs_ #COMM 'same source and destination: from pv1 to pv1' -prepare_lvs_ +restore_lvs_ not pvmove -i0 "$dev1" "$dev1" #"(cleanup previous test)" -lv_not_changed_ $vg/$lv1 -lv_not_changed_ $vg/$lv2 -lv_not_changed_ $vg/$lv3 +lvs_not_changed_ $lv1 $lv2 $lv3 check_and_cleanup_lvs_ #COMM "sum of specified destination PEs is large enough, but it includes source PEs and the free PEs are not enough" -prepare_lvs_ +restore_lvs_ not pvmove --alloc anywhere "$dev1":0-2 "$dev1":0-2 "$dev5":0-0 2> err #"(cleanup previous test)" grep "Insufficient free space" err -lv_not_changed_ $vg/$lv1 -lv_not_changed_ $vg/$lv2 -lv_not_changed_ $vg/$lv3 +lvs_not_changed_ $lv1 $lv2 $lv3 check_and_cleanup_lvs_ # --------------------------------------------------------------------- #COMM "pvmove abort" -prepare_lvs_ +restore_lvs_ pvmove -i100 -b "$dev1" "$dev3" pvmove --abort check_and_cleanup_lvs_ diff --git a/test/shell/pvmove-raid-segtypes.sh b/test/shell/pvmove-raid-segtypes.sh new file mode 100644 index 0000000..4b720bb --- /dev/null +++ b/test/shell/pvmove-raid-segtypes.sh @@ -0,0 +1,79 @@ +#!/bin/sh +# Copyright (C) 2013 Red Hat, Inc. All rights reserved. +# +# This copyrighted material is made available to anyone wishing to use, +# modify, copy, or redistribute it subject to the terms and conditions +# of the GNU General Public License v.2. +# +# You should have received a copy of the GNU General Public License +# along with this program; if not, write to the Free Software Foundation, +# Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + +test_description="ensure pvmove works with raid segment types" + +. lib/test + +test -e LOCAL_CLVMD && skip +which mkfs.ext2 || skip +which md5sum || skip + +aux target_at_least dm-raid 1 3 5 || skip + +aux prepare_pvs 5 20 +vgcreate -c n -s 128k $vg $(cat DEVICES) + +# Each of the following tests does: +# 1) Create two LVs - one linear and one other segment type +# The two LVs will share a PV. +# 2) Move both LVs together +# 3) Move only the second LV by name + +# Testing pvmove of RAID1 LV +lvcreate -l 2 -n ${lv1}_foo $vg "$dev1" +lvcreate -l 2 --type raid1 -m 1 -n $lv1 $vg "$dev1" "$dev2" +check lv_tree_on $vg ${lv1}_foo "$dev1" +check lv_tree_on $vg $lv1 "$dev1" "$dev2" +aux mkdev_md5sum $vg $lv1 +pvmove "$dev1" "$dev5" +check lv_tree_on $vg ${lv1}_foo "$dev5" +check lv_tree_on $vg $lv1 "$dev2" "$dev5" +check dev_md5sum $vg $lv1 +pvmove -n $lv1 "$dev5" "$dev4" +check lv_tree_on $vg $lv1 "$dev2" "$dev4" +check lv_tree_on $vg ${lv1}_foo "$dev5" +check dev_md5sum $vg $lv1 +lvremove -ff $vg + +# Testing pvmove of RAID10 LV +lvcreate -l 2 -n ${lv1}_foo $vg "$dev1" +lvcreate -l 4 --type raid10 -i 2 -m 1 -n $lv1 $vg \ + "$dev1" "$dev2" "$dev3" "$dev4" +check lv_tree_on $vg ${lv1}_foo "$dev1" +check lv_tree_on $vg $lv1 "$dev1" "$dev2" "$dev3" "$dev4" +aux mkdev_md5sum $vg $lv1 +pvmove "$dev1" "$dev5" +check lv_tree_on $vg ${lv1}_foo "$dev5" +check lv_tree_on $vg $lv1 "$dev2" "$dev3" "$dev4" "$dev5" +check dev_md5sum $vg $lv1 +pvmove -n $lv1 "$dev5" "$dev1" +check lv_tree_on $vg $lv1 "$dev1" "$dev2" "$dev3" "$dev4" +check lv_tree_on $vg ${lv1}_foo "$dev5" +check dev_md5sum $vg $lv1 +lvremove -ff $vg + +# Testing pvmove of RAID5 LV +lvcreate -l 2 -n ${lv1}_foo $vg "$dev1" +lvcreate -l 4 --type raid5 -i 2 -n $lv1 $vg \ + "$dev1" "$dev2" "$dev3" +check lv_tree_on $vg ${lv1}_foo "$dev1" +check lv_tree_on $vg $lv1 "$dev1" "$dev2" "$dev3" +aux mkdev_md5sum $vg $lv1 +pvmove "$dev1" "$dev5" +check lv_tree_on $vg ${lv1}_foo "$dev5" +check lv_tree_on $vg $lv1 "$dev2" "$dev3" "$dev5" +check dev_md5sum $vg $lv1 +pvmove -n $lv1 "$dev5" "$dev4" +check lv_tree_on $vg $lv1 "$dev2" "$dev3" "$dev4" +check lv_tree_on $vg ${lv1}_foo "$dev5" +check dev_md5sum $vg $lv1 +lvremove -ff $vg diff --git a/test/shell/pvmove-thin-segtypes.sh b/test/shell/pvmove-thin-segtypes.sh new file mode 100644 index 0000000..fe07288 --- /dev/null +++ b/test/shell/pvmove-thin-segtypes.sh @@ -0,0 +1,83 @@ +#!/bin/sh +# Copyright (C) 2013 Red Hat, Inc. All rights reserved. +# +# This copyrighted material is made available to anyone wishing to use, +# modify, copy, or redistribute it subject to the terms and conditions +# of the GNU General Public License v.2. +# +# You should have received a copy of the GNU General Public License +# along with this program; if not, write to the Free Software Foundation, +# Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + +test_description="ensure pvmove works with thin segment types" + +. lib/test +test -e LOCAL_CLVMD && skip +which mkfs.ext2 || skip +which md5sum || skip + +aux target_at_least dm-thin-pool 1 8 0 || skip +# for stacking +aux target_at_least dm-raid 1 3 5 || skip + +aux prepare_pvs 5 20 +vgcreate -c n -s 128k $vg $(cat DEVICES) + +# Each of the following tests does: +# 1) Create two LVs - one linear and one other segment type +# The two LVs will share a PV. +# 2) Move both LVs together +# 3) Move only the second LV by name + + +# Testing pvmove of thin-pool LV +lvcreate -l 2 -n ${lv1}_foo $vg "$dev1" +lvcreate -T $vg/$lv1 -l 4 "$dev1" +check lv_tree_on $vg ${lv1}_foo "$dev1" +check lv_tree_on $vg $lv1 "$dev1" +aux mkdev_md5sum $vg $lv1 +pvmove "$dev1" "$dev5" +check lv_tree_on $vg ${lv1}_foo "$dev5" +check lv_tree_on $vg $lv1 "$dev5" +check dev_md5sum $vg $lv1 +pvmove -n $lv1 "$dev5" "$dev4" +check lv_tree_on $vg $lv1 "$dev4" +check lv_tree_on $vg ${lv1}_foo "$dev5" +check dev_md5sum $vg $lv1 +lvremove -ff $vg + +# Testing pvmove of thin LV +lvcreate -l 2 -n ${lv1}_foo $vg "$dev1" +lvcreate -T $vg/${lv1}_pool -l 4 -V 8 -n $lv1 "$dev1" +check lv_tree_on $vg ${lv1}_foo "$dev1" +check lv_tree_on $vg $lv1 "$dev1" +aux mkdev_md5sum $vg $lv1 +pvmove "$dev1" "$dev5" +check lv_tree_on $vg ${lv1}_foo "$dev5" +check lv_tree_on $vg $lv1 "$dev5" +check dev_md5sum $vg $lv1 +pvmove -n $lv1 "$dev5" "$dev4" +check lv_tree_on $vg $lv1 "$dev4" +check lv_tree_on $vg ${lv1}_foo "$dev5" +check dev_md5sum $vg $lv1 +lvremove -ff $vg + +# Testing pvmove of thin LV on RAID +lvcreate -l 2 -n ${lv1}_foo $vg "$dev1" +lvcreate --type raid1 -m 1 -l 4 -n ${lv1}_raid1_pool $vg "$dev1" "$dev2" +lvcreate --type raid1 -m 1 -L 2 -n ${lv1}_raid1_meta $vg "$dev1" "$dev2" +lvconvert --thinpool $vg/${lv1}_raid1_pool \ + --poolmetadata ${lv1}_raid1_meta +lvcreate -T $vg/${lv1}_raid1_pool -V 8 -n $lv1 +check lv_tree_on $vg ${lv1}_foo "$dev1" +check lv_tree_on $vg $lv1 "$dev1" "$dev2" +aux mkdev_md5sum $vg $lv1 +pvmove "$dev1" "$dev5" +check lv_tree_on $vg ${lv1}_foo "$dev5" +check lv_tree_on $vg $lv1 "$dev2" "$dev5" +check dev_md5sum $vg $lv1 +pvmove -n $lv1 "$dev5" "$dev4" +check lv_tree_on $vg $lv1 "$dev2" "$dev4" +check lv_tree_on $vg ${lv1}_foo "$dev5" +check dev_md5sum $vg $lv1 +lvremove -ff $vg

1 0

master - tests: use check for raid test
by Zdenek Kabelac 16 Sep '13

16 Sep '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=47b7cc6850091aac… Commit: 47b7cc6850091aac9d32070ad3c3a727bae5f188 Parent: 9df27c42794652cc663e04c5c751daa549e73194 Author: Zdenek Kabelac <zkabelac(a)redhat.com> AuthorDate: Mon Sep 16 11:15:38 2013 +0200 Committer: Zdenek Kabelac <zkabelac(a)redhat.com> CommitterDate: Mon Sep 16 11:22:04 2013 +0200 tests: use check for raid test Use test-suite built-in check command for this test (creates better logs, and reduces amount of valgrind-ed commands) --- test/shell/lvcreate-raid.sh | 8 ++++---- test/shell/lvcreate-raid10.sh | 5 ++--- 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/test/shell/lvcreate-raid.sh b/test/shell/lvcreate-raid.sh index ee56b9c..214b162 100644 --- a/test/shell/lvcreate-raid.sh +++ b/test/shell/lvcreate-raid.sh @@ -42,8 +42,8 @@ lvremove -ff $vg lvcreate --type raid1 -m 2 -l 2 \ --minrecoveryrate 50 --maxrecoveryrate 100 \ -n $lv1 $vg -[ `lvs --noheadings -o raid_min_recovery_rate $vg/$lv1` -eq 50 ] -[ `lvs --noheadings -o raid_max_recovery_rate $vg/$lv1` -eq 100 ] +check lv_field $vg/$lv1 raid_min_recovery_rate 50 +check lv_field $vg/$lv1 raid_max_recovery_rate 100 aux wait_for_sync $vg $lv1 lvremove -ff $vg @@ -65,8 +65,8 @@ for i in raid4 \ lvcreate --type $i -l 3 -i 3 \ --minrecoveryrate 50 --maxrecoveryrate 100 \ -n $lv1 $vg - [ `lvs --noheadings -o raid_min_recovery_rate $vg/$lv1` -eq 50 ] - [ `lvs --noheadings -o raid_max_recovery_rate $vg/$lv1` -eq 100 ] + check lv_field $vg/$lv1 raid_min_recovery_rate 50 + check lv_field $vg/$lv1 raid_max_recovery_rate 100 aux wait_for_sync $vg $lv1 lvremove -ff $vg done diff --git a/test/shell/lvcreate-raid10.sh b/test/shell/lvcreate-raid10.sh index 933097a..143e869 100644 --- a/test/shell/lvcreate-raid10.sh +++ b/test/shell/lvcreate-raid10.sh @@ -36,15 +36,14 @@ lvremove -ff $vg/$lv1 lvcreate --type raid10 -m 1 -i 2 -l 2 \ --minrecoveryrate 50 --maxrecoveryrate 100 \ -n $lv1 $vg -[ `lvs --noheadings -o raid_min_recovery_rate $vg/$lv1` -eq 50 ] -[ `lvs --noheadings -o raid_max_recovery_rate $vg/$lv1` -eq 100 ] +check lv_field $vg/$lv1 raid_min_recovery_rate 50 +check lv_field $vg/$lv1 raid_max_recovery_rate 100 aux wait_for_sync $vg $lv1 # 2-way mirror, 3-stripes lvcreate --type raid10 -m 1 -i 3 -l 3 -n $lv2 $vg aux wait_for_sync $vg $lv2 - lvremove -ff $vg #

1 0

master - tests: update check and get
by Zdenek Kabelac 16 Sep '13

16 Sep '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=9df27c42794652cc… Commit: 9df27c42794652cc663e04c5c751daa549e73194 Parent: 51d3667cb559e77cd43556851922cb5bef0fc9ae Author: Zdenek Kabelac <zkabelac(a)redhat.com> AuthorDate: Mon Sep 16 11:12:37 2013 +0200 Committer: Zdenek Kabelac <zkabelac(a)redhat.com> CommitterDate: Mon Sep 16 11:22:04 2013 +0200 tests: update check and get Update usage of "" around shell vars. trim needs to trim both sides now. trim also removes debug.log since it's only called when lvm command has finished properly (so if something fails afterward, there is no missleading debug trace in the log) --- test/lib/check.sh | 4 ++-- test/lib/get.sh | 19 +++++++++++-------- 2 files changed, 13 insertions(+), 10 deletions(-) diff --git a/test/lib/check.sh b/test/lib/check.sh index ecb0721..d3678a9 100644 --- a/test/lib/check.sh +++ b/test/lib/check.sh @@ -236,13 +236,13 @@ pv_field() { } vg_field() { - local actual=$(get vg_field $1 "$2" "${@:4}") + local actual=$(get vg_field "$1" "$2" "${@:4}") test "$actual" = "$3" || \ die "vg_field: vg=$1, field=\"$2\", actual=\"$actual\", expected=\"$3\"" } lv_field() { - local actual=$(get lv_field $1 "$2" "${@:4}") + local actual=$(get lv_field "$1" "$2" "${@:4}") test "$actual" = "$3" || \ die "lv_field: lv=$lv, field=\"$2\", actual=\"$actual\", expected=\"$3\"" } diff --git a/test/lib/get.sh b/test/lib/get.sh index 4197f47..7d7c3d2 100644 --- a/test/lib/get.sh +++ b/test/lib/get.sh @@ -1,5 +1,5 @@ #!/bin/sh -# Copyright (C) 2011-2012 Red Hat, Inc. All rights reserved. +# Copyright (C) 2011-2013 Red Hat, Inc. All rights reserved. # # This copyrighted material is made available to anyone wishing to use, # modify, copy, or redistribute it subject to the terms and conditions @@ -18,28 +18,31 @@ # # get lv_devices LV [lvs params] +test -z "$BASH" || set -e -o pipefail -# trims only leading prefix, we should not need trim trailing spaces +# trims only leading prefix and suffix trim_() { - #local var=${var%"${var##*[! ]}"} # remove trailing space characters - echo ${1#"${1%%[! ]*}"} # remove leading space characters + rm -f debug.log # drop log, command was ok + local var=${1%${1##*[! ]}} # remove trailing space characters + echo "${var#${var%%[! ]*}}" # remove leading space characters } pv_field() { - trim_ "$(pvs --noheadings -o $2 ${@:3} $1)" + trim_ "$(pvs --config 'log{prefix=""}' --noheadings -o $2 ${@:3} $1)" } vg_field() { - trim_ "$(vgs --noheadings -o $2 ${@:3} $1)" + trim_ "$(vgs --config 'log{prefix=""}' --noheadings -o $2 ${@:3} $1)" } lv_field() { - trim_ "$(lvs --noheadings -o $2 ${@:3} $1)" + trim_ "$(lvs --config 'log{prefix=""}' --noheadings -o $2 ${@:3} $1)" } lv_devices() { - lv_field $1 devices -a "${@:2}" | sed 's/([^)]*)//g; s/,/ /g' + lv_field "$1" devices -a "${@:2}" | sed 's/([^)]*)//g; s/,/\n/g' } +#set -x unset LVM_VALGRIND "$@"

1 0

master - tests: update die
by Zdenek Kabelac 16 Sep '13

16 Sep '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=51d3667cb559e77c… Commit: 51d3667cb559e77cd43556851922cb5bef0fc9ae Parent: d6090a10f01e6daf5c66e7098a3b0f85560a80ce Author: Zdenek Kabelac <zkabelac(a)redhat.com> AuthorDate: Mon Sep 16 11:05:03 2013 +0200 Committer: Zdenek Kabelac <zkabelac(a)redhat.com> CommitterDate: Mon Sep 16 11:22:04 2013 +0200 tests: update die 'die' evaluates given string - so \n could be used for multiline error report Also remove debug.log since the command finished properly when we call 'die' Note: we should not call 'die' after lvm command failure. --- test/lib/check.sh | 3 ++- test/lib/utils.sh | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/test/lib/check.sh b/test/lib/check.sh index 6eedb47..ecb0721 100644 --- a/test/lib/check.sh +++ b/test/lib/check.sh @@ -25,7 +25,8 @@ test -z "$BASH" || set -e -o pipefail die() { - echo "$@" >&2 + rm -f debug.log + echo -e "$@" >&2 return 1 } diff --git a/test/lib/utils.sh b/test/lib/utils.sh index 47e89bf..0f58b25 100644 --- a/test/lib/utils.sh +++ b/test/lib/utils.sh @@ -15,7 +15,8 @@ IFS_NL=' ' die() { - echo "$@" >&2 + rm -f debug.log + echo -e "$@" >&2 return 1 }

1 0

master - tests: add help function
by Zdenek Kabelac 16 Sep '13

16 Sep '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=d6090a10f01e6daf… Commit: d6090a10f01e6daf5c66e7098a3b0f85560a80ce Parent: 1d068682404c7af35c0b40586a6e5479443fc851 Author: Zdenek Kabelac <zkabelac(a)redhat.com> AuthorDate: Mon Sep 16 11:02:58 2013 +0200 Committer: Zdenek Kabelac <zkabelac(a)redhat.com> CommitterDate: Mon Sep 16 11:22:04 2013 +0200 tests: add help function Add mkdev_md5sum to create and checksum given LV. Add dev_md5sum to verify device has matching md5 sum. --- test/lib/aux.sh | 6 ++++++ test/lib/check.sh | 8 ++++++++ 2 files changed, 14 insertions(+), 0 deletions(-) diff --git a/test/lib/aux.sh b/test/lib/aux.sh index e07397d..6194492 100644 --- a/test/lib/aux.sh +++ b/test/lib/aux.sh @@ -506,6 +506,12 @@ unhide_dev() { lvmconf "$filter" } +mkdev_md5sum() { + rm -f debug.log + mkfs.ext2 "$DM_DEV_DIR/$1/$2" || return 1 + md5sum "$DM_DEV_DIR/$1/$2" > "md5.$1-$2" +} + generate_config() { if test -n "$profile_name"; then config_values=PROFILE_VALUES_$profile_name diff --git a/test/lib/check.sh b/test/lib/check.sh index 8b2cfce..6eedb47 100644 --- a/test/lib/check.sh +++ b/test/lib/check.sh @@ -280,5 +280,13 @@ pvlv_counts() { vg_field $local_vg snap_count $num_snaps } +# Compare md5 check generated from get dev_md5sum +dev_md5sum() { + md5sum -c "md5.$1-$2" || \ + (get lv_field $1/$2 "name,size,seg_pe_ranges" + die "LV $1/$2 has different MD5 check sum!") +} + +#set -x unset LVM_VALGRIND "$@"

1 0

master - TEST: Unaccounted possible output causing failure
by Jonathan Brassow 12 Sep '13

12 Sep '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=1d068682404c7af3… Commit: 1d068682404c7af35c0b40586a6e5479443fc851 Parent: e2268aeb92779580d3da4c5c400a1c353f955571 Author: Jonathan Brassow <jbrassow(a)redhat.com> AuthorDate: Thu Sep 12 13:23:53 2013 -0500 Committer: Jonathan Brassow <jbrassow(a)redhat.com> CommitterDate: Thu Sep 12 13:23:53 2013 -0500 TEST: Unaccounted possible output causing failure lvchange-raid.sh checks to ensure that the 'p'artial flag takes precedence over the 'w'ritemostly flag by disabling and reenabling a device in the array. Most of the time this works fine, but sometimes the kernel can notice the device failure before it is reenabled. In that case, the attr flag will not return to 'w', but to 'r'efresh. This is because 'r'efresh also takes precedence over the 'w'ritemostly flag. So, we also do a quick check for 'r' and not just 'w'. --- test/shell/lvchange-raid.sh | 8 ++++++++ 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/test/shell/lvchange-raid.sh b/test/shell/lvchange-raid.sh index f1b8d72..91c8e8d 100644 --- a/test/shell/lvchange-raid.sh +++ b/test/shell/lvchange-raid.sh @@ -105,7 +105,15 @@ run_writemostly_check() { # Partial flag supercedes writemostly flag aux disable_dev $d0 lvs -a --noheadings -o lv_attr $vg/${lv}_rimage_0 | grep '.*p.$' + + # It is possible for the kernel to detect the failed device before + # we re-enable it. If so, the field will be set to 'r'efresh since + # that also takes precedence over 'w'ritemostly. If this has happened, + # we refresh the LV and then check for 'w'. aux enable_dev $d0 + if lvs -a --noheadings -o lv_attr $vg/${lv}_rimage_0 | grep '.*r.$'; then + lvchange --refresh $vg/$lv + fi lvs -a --noheadings -o lv_attr $vg/${lv}_rimage_0 | grep '.*w.$' # Catch Bad writebehind values

1 0

master - WHATS_NEW: some missing lines
by Peter Rajnoha 12 Sep '13

12 Sep '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=e2268aeb92779580… Commit: e2268aeb92779580d3da4c5c400a1c353f955571 Parent: 6940e24aeeff3d6c5c8da259b74afd908b4716f2 Author: Peter Rajnoha <prajnoha(a)redhat.com> AuthorDate: Thu Sep 12 14:16:54 2013 +0200 Committer: Peter Rajnoha <prajnoha(a)redhat.com> CommitterDate: Thu Sep 12 14:16:54 2013 +0200 WHATS_NEW: some missing lines --- WHATS_NEW | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/WHATS_NEW b/WHATS_NEW index c61070e..a46b0b4 100644 --- a/WHATS_NEW +++ b/WHATS_NEW @@ -1,5 +1,7 @@ Version 2.02.101 - =================================== + Add man page entries for lvmdump's -u and -l options. + Fix lvm2app segfault while using lvm_list_pvs_free fn if there are no PVs. Improve of clvmd singlenode locking simulation. Disallow thinpools on mirror logical volumes. Direct udev to use 3min timeout for LVM devices. Recent udev has default 30s.

1 0

← Newer
1
...
4
5
6
7
8
9
10
11
Older →

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

lvm2-commits September 2013