From: Martin Schwenke <martin@meltin.net>
Date: Wed, 6 Jul 2016 04:06:14 +0000 (+1000)
Subject: ctdb-tests: Add reclock event script tests
X-Git-Tag: tdb-1.3.10~286
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=c7a730cbc40f2fb81968e0e8fa44d441d825eb7f;p=thirdparty%2Fsamba.git

ctdb-tests: Add reclock event script tests

Tweak eventscript unit test infrastructure to support.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
---

diff --git a/ctdb/tests/eventscripts/01.reclock.monitor.001.sh b/ctdb/tests/eventscripts/01.reclock.monitor.001.sh
new file mode 100755
index 00000000000..8230528c3fe
--- /dev/null
+++ b/ctdb/tests/eventscripts/01.reclock.monitor.001.sh
@@ -0,0 +1,10 @@
+#!/bin/sh
+
+. "${TEST_SCRIPTS_DIR}/unit.sh"
+
+define_test "unset, check no-op"
+
+CTDB_RECOVERY_LOCK=""
+
+ok_null
+simple_test
diff --git a/ctdb/tests/eventscripts/01.reclock.monitor.002.sh b/ctdb/tests/eventscripts/01.reclock.monitor.002.sh
new file mode 100755
index 00000000000..b40213f3256
--- /dev/null
+++ b/ctdb/tests/eventscripts/01.reclock.monitor.002.sh
@@ -0,0 +1,10 @@
+#!/bin/sh
+
+. "${TEST_SCRIPTS_DIR}/unit.sh"
+
+define_test "set to helper, check no-op"
+
+CTDB_RECOVERY_LOCK="!/some/recover/lock/helper foo"
+
+ok_null
+simple_test
diff --git a/ctdb/tests/eventscripts/01.reclock.monitor.003.sh b/ctdb/tests/eventscripts/01.reclock.monitor.003.sh
new file mode 100755
index 00000000000..2d82a21d1ee
--- /dev/null
+++ b/ctdb/tests/eventscripts/01.reclock.monitor.003.sh
@@ -0,0 +1,10 @@
+#!/bin/sh
+
+. "${TEST_SCRIPTS_DIR}/unit.sh"
+
+define_test "set, exists"
+
+setup_reclock
+
+ok_null
+simple_test
diff --git a/ctdb/tests/eventscripts/01.reclock.monitor.004.sh b/ctdb/tests/eventscripts/01.reclock.monitor.004.sh
new file mode 100755
index 00000000000..fd185d1aa52
--- /dev/null
+++ b/ctdb/tests/eventscripts/01.reclock.monitor.004.sh
@@ -0,0 +1,18 @@
+#!/bin/sh
+
+. "${TEST_SCRIPTS_DIR}/unit.sh"
+
+define_test "set, doesn't exist, 4 times"
+
+setup_reclock
+rm -f "$CTDB_RECOVERY_LOCK"
+
+ok_null
+simple_test
+simple_test
+simple_test
+
+required_result 1 <<EOF
+ERROR: 4 consecutive failures for 01.reclock, marking node unhealthy
+EOF
+simple_test
diff --git a/ctdb/tests/eventscripts/01.reclock.monitor.005.sh b/ctdb/tests/eventscripts/01.reclock.monitor.005.sh
new file mode 100755
index 00000000000..e190e14585a
--- /dev/null
+++ b/ctdb/tests/eventscripts/01.reclock.monitor.005.sh
@@ -0,0 +1,29 @@
+#!/bin/sh
+
+. "${TEST_SCRIPTS_DIR}/unit.sh"
+
+define_test "set, doesn't exist, 4 times"
+
+setup_reclock
+rm -f "$CTDB_RECOVERY_LOCK"
+
+ok_null
+for i in $(seq 1 3) ; do
+	simple_test
+done
+
+for i in $(seq 4 199) ; do
+	required_result 1 <<EOF
+ERROR: ${i} consecutive failures for 01.reclock, marking node unhealthy
+EOF
+	simple_test
+done
+
+required_result 1 <<EOF
+Reclock file "${CTDB_RECOVERY_LOCK}" can not be accessed. Shutting down.
+Filesystem             1024-blocks      Used Available Capacity Mounted on
+/dev/sda1                               1000000     100000     900000         10% /
+CTDB says BYE!
+ERROR: 200 consecutive failures for 01.reclock, marking node unhealthy
+EOF
+simple_test
diff --git a/ctdb/tests/eventscripts/etc/sysconfig/ctdb b/ctdb/tests/eventscripts/etc/sysconfig/ctdb
index fe0436d6f55..e03d8103827 100644
--- a/ctdb/tests/eventscripts/etc/sysconfig/ctdb
+++ b/ctdb/tests/eventscripts/etc/sysconfig/ctdb
@@ -1,4 +1,3 @@
-CTDB_RECOVERY_LOCK="/some/place/on/shared/storage"
 CTDB_DEBUGLEVEL=ERR
 
 if [ -n "$FAKE_CTDB_EXTRA_CONFIG" -a -r "$FAKE_CTDB_EXTRA_CONFIG" ] ; then
diff --git a/ctdb/tests/eventscripts/scripts/local.sh b/ctdb/tests/eventscripts/scripts/local.sh
index 37141c6d14b..6db6e40d513 100644
--- a/ctdb/tests/eventscripts/scripts/local.sh
+++ b/ctdb/tests/eventscripts/scripts/local.sh
@@ -1129,6 +1129,16 @@ program $_rpc_service${_ver:+ version }${_ver} is not available"
 
 ######################################################################
 
+# Recovery lock fakery
+
+setup_reclock ()
+{
+	CTDB_RECOVERY_LOCK=$(mktemp --tmpdir="$EVENTSCRIPTS_TESTS_VAR_DIR")
+	export CTDB_RECOVERY_LOCK
+}
+
+######################################################################
+
 # VSFTPD fakery
 
 setup_vsftpd ()
diff --git a/ctdb/tests/eventscripts/stubs/df b/ctdb/tests/eventscripts/stubs/df
index eb91a4c851d..64eb3814cfe 100755
--- a/ctdb/tests/eventscripts/stubs/df
+++ b/ctdb/tests/eventscripts/stubs/df
@@ -2,20 +2,19 @@
 
 usage ()
 {
-    echo "usage: df -kP <mount-point>"
+    echo "usage: df [-kP] [<mount-point>]"
     exit 1
 }
 
-if [ "$1" != "-kP" ] ; then
-    usage
+if [ "$1" = "-kP" ] ; then
+    shift
 fi
 
-shift
-if [ -z "$1" ] ; then
-    usage
-fi
+case "$1" in
+     -*) usage ;;
+esac
 
-fs="$1"
+fs="${1:-/}"
 
 # Anything starting with CTDB_DBDIR gets canonicalised to CTDB_DBDIR.
 # This helps with the setting of defaults for the filesystem checks.