ctdb-tests: Drop attempts to pass arguments to ctdbd on (re)start
authorMartin Schwenke <martin@meltin.net>
Tue, 23 Aug 2016 00:23:56 +0000 (10:23 +1000)
committerAmitay Isaacs <amitay@samba.org>
Tue, 23 Aug 2016 15:22:17 +0000 (17:22 +0200)
This is not used.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=12170

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
ctdb/tests/scripts/integration.bash
ctdb/tests/simple/scripts/local_daemons.bash

index 0f12c1f3020a7d5c8e0c66c9f3f3580a638b10d4..b2a3451049c9927836d9b46ec04ab5e7342a2482 100644 (file)
@@ -536,7 +536,7 @@ restart_ctdb_1 ()
 # Restart CTDB on all nodes.  Override for local daemons.
 _restart_ctdb_all ()
 {
-    onnode -p all $CTDB_TEST_WRAPPER restart_ctdb_1 "$@"
+    onnode -p all $CTDB_TEST_WRAPPER restart_ctdb_1
 }
 
 # Nothing needed for a cluster.  Override for local daemons.
@@ -547,8 +547,6 @@ setup_ctdb ()
 
 restart_ctdb ()
 {
-    # "$@" is passed to restart_ctdb_all.
-
     echo -n "Restarting CTDB"
     if $ctdb_test_restart_scheduled ; then
        echo -n " (scheduled)"
@@ -557,7 +555,7 @@ restart_ctdb ()
 
     local i
     for i in $(seq 1 5) ; do
-       _restart_ctdb_all "$@" || {
+       _restart_ctdb_all || {
            echo "Restart failed.  Trying again in a few seconds..."
            sleep_for 5
            continue
index 765655ecaa335b7de9a4aef3abcfe8700adfb099..0ad7eb85e114c25e24252ff13012c3239f71740e 100644 (file)
@@ -95,7 +95,6 @@ setup_ctdb ()
 daemons_start_1 ()
 {
     local pnn="$1"
-    shift # "$@" gets passed to ctdbd
 
     local public_addresses_all="${TEST_VAR_DIR}/public_addresses_all"
     local public_addresses_mine="${TEST_VAR_DIR}/public_addresses.${pnn}"
@@ -147,12 +146,10 @@ EOF
 
 daemons_start ()
 {
-    # "$@" gets passed to ctdbd
-
     echo "Starting $TEST_LOCAL_DAEMONS ctdb daemons..."
 
     for i in $(seq 0 $(($TEST_LOCAL_DAEMONS - 1))) ; do
-       daemons_start_1 $i "$@"
+       daemons_start_1 $i
     done
 }
 
@@ -166,5 +163,5 @@ maybe_stop_ctdb ()
 _restart_ctdb_all ()
 {
     daemons_stop
-    daemons_start "$@"
+    daemons_start
 }