All of lore.kernel.org
 help / color / mirror / Atom feed
From: Leo Yan <leo.yan@linaro.org>
To: lvm-devel@redhat.com
Subject: [PATCH v1 13/17] tests: idm: Add testing for IDM lock manager failure
Date: Thu,  3 Jun 2021 17:59:21 +0800	[thread overview]
Message-ID: <20210603095925.510302-14-leo.yan@linaro.org> (raw)
In-Reply-To: <20210603095925.510302-1-leo.yan@linaro.org>

If the IDM lock manager fails to access drives, might partially fail to
access drives (e.g. it fails to access one of three drives), or totally
fail to access drives, the lock manager should handle properly for these
cases.  When the drives are partially failure, if the lock manager still
can renew the lease for the locking, then it doesn't need to take any
action for the drive failure; otherwise, if it detects it cannot renew
the locking majority, it needs ti immediately kill the VG from the
lvmlockd.

This patch adds the test for verification the IDM lock manager failure;
the command can be used as below:

  # make check_lvmlockd_idm \
    LVM_TEST_BACKING_DEVICE=/dev/sdp3,/dev/sdl3,/dev/sdq3 \
    LVM_TEST_FAILURE=1 T=idm_ilm_failure.sh

Signed-off-by: Leo Yan <leo.yan@linaro.org>
---
 test/shell/idm_ilm_failure.sh | 80 +++++++++++++++++++++++++++++++++++
 1 file changed, 80 insertions(+)
 create mode 100644 test/shell/idm_ilm_failure.sh

diff --git a/test/shell/idm_ilm_failure.sh b/test/shell/idm_ilm_failure.sh
new file mode 100644
index 000000000..58bed270e
--- /dev/null
+++ b/test/shell/idm_ilm_failure.sh
@@ -0,0 +1,80 @@
+#!/usr/bin/env bash
+
+# Copyright (C) 2020 Seagate, 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., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+
+
+SKIP_WITH_LVMPOLLD=1
+
+. lib/inittest
+
+[ -z "$LVM_TEST_LOCK_TYPE_IDM" ] && skip;
+[ -z "$LVM_TEST_FAILURE" ] && skip;
+
+aux prepare_devs 3
+aux extend_filter_LVMTEST
+
+DRIVE1=`dmsetup deps -o devname $dev1 | awk '{gsub(/[()]/,""); print $4;}' | sed 's/[0-9]*$//'`
+DRIVE2=`dmsetup deps -o devname $dev2 | awk '{gsub(/[()]/,""); print $4;}' | sed 's/[0-9]*$//'`
+DRIVE3=`dmsetup deps -o devname $dev3 | awk '{gsub(/[()]/,""); print $4;}' | sed 's/[0-9]*$//'`
+
+if [ "$DRIVE1" = "$DRIVE2" ] || [ "$DRIVE1" = "$DRIVE3" ] || [ "$DRIVE2" = "$DRIVE3" ]; then
+	die "Need to pass three different drives!?"
+fi
+
+# The previous device-mapper are removed, but LVM still can directly
+# access VGs from the specified physical drives.  So enable drives
+# for these drives.
+aux extend_filter_LVMTEST "a|/dev/$DRIVE1*|" "a|/dev/$DRIVE2*|" "a|/dev/$DRIVE3*|"
+aux lvmconf "devices/allow_changes_with_duplicate_pvs = 1"
+
+vgcreate $SHARED $vg "$dev1" "$dev2" "$dev3"
+
+# Create new logic volume and deactivate it
+lvcreate -a y --zero n -l 1 -n $lv1 $vg
+
+# Inject failure 40% so cannot send partially request to drives
+idm_inject_failure 40
+
+# Wait for 40s, but the lock will not be time out
+sleep 40
+
+# Inject failure with 0% so can access drives
+idm_inject_failure 0
+
+# Deactivate logic volume due to locking failure
+lvchange $vg/$lv1 -a n
+
+# Inject failure 100% so cannot send request to drives
+idm_inject_failure 100
+
+# Wait for 70s but should have no any alive locks
+sleep 70
+
+# Inject failure with 0% so can access drives
+idm_inject_failure 0
+
+# Activate logic volume
+lvchange $vg/$lv1 -a y
+
+# Inject failure so cannot send request to drives
+idm_inject_failure 100
+
+# Wait for 70s but will not time out
+sleep 70
+
+# Inject failure with 0% so can access drives
+idm_inject_failure 0
+
+check grep_lvmlockd_dump "S lvm_$vg kill_vg"
+lvmlockctl --drop $vg
+
+vgchange --lock-start
+vgremove -f $vg
-- 
2.25.1



  parent reply	other threads:[~2021-06-03  9:59 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-03  9:59 [PATCH v1 00/17] LVM2: Enable testing for In-Drive-Mutex Leo Yan
2021-06-03  9:59 ` [PATCH v1 01/17] tests: Enable the testing for IDM locking scheme Leo Yan
2021-06-03  9:59 ` [PATCH v1 02/17] tests: Support multiple backing devices Leo Yan
2021-06-03  9:59 ` [PATCH v1 03/17] tests: Cleanup idm context when prepare devices Leo Yan
2021-06-03  9:59 ` [PATCH v1 04/17] tests: Add checking for lvmlockd log Leo Yan
2021-06-03  9:59 ` [PATCH v1 05/17] tests: stress: Add single thread stress testing Leo Yan
2021-06-03  9:59 ` [PATCH v1 06/17] tests: stress: Add multi-threads stress testing for VG/LV Leo Yan
2021-06-03  9:59 ` [PATCH v1 07/17] tests: stress: Add multi-threads stress testing for PV/VG/LV Leo Yan
2021-06-03  9:59 ` [PATCH v1 08/17] tests: Support idm failure injection Leo Yan
2021-06-03  9:59 ` [PATCH v1 09/17] tests: Add testing for lvmlockd failure Leo Yan
2021-06-03  9:59 ` [PATCH v1 10/17] tests: idm: Add testing for the fabric failure Leo Yan
2021-06-03  9:59 ` [PATCH v1 11/17] tests: idm: Add testing for the fabric failure and timeout Leo Yan
2021-06-03  9:59 ` [PATCH v1 12/17] tests: idm: Add testing for the fabric's half brain failure Leo Yan
2021-06-03  9:59 ` Leo Yan [this message]
2021-06-03  9:59 ` [PATCH v1 14/17] tests: multi-hosts: Add VG testing Leo Yan
2021-06-03  9:59 ` [PATCH v1 15/17] tests: multi-hosts: Add LV testing Leo Yan
2021-06-03  9:59 ` [PATCH v1 16/17] tests: multi-hosts: Test lease timeout with LV exclusive mode Leo Yan
2021-06-03  9:59 ` [PATCH v1 17/17] tests: multi-hosts: Test lease timeout with LV shareable mode Leo Yan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210603095925.510302-14-leo.yan@linaro.org \
    --to=leo.yan@linaro.org \
    --cc=lvm-devel@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.