ctdb-tests: Improve test portability

"wc -l" on some platforms (e.g. FreeBSD) contains leading spaces and
stops "$num from being a number.  Create a more portable solution and
put it in a function instead of repeating the logic.

Signed-off-by: Martin Schwenke <martin@meltin.net>

diff --git a/ctdb/tests/INTEGRATION/failover/pubips.040.NoIPTakeover.sh b/ctdb/tests/INTEGRATION/failover/pubips.040.NoIPTakeover.sh
index a39e48d0883023b56f306ce9ca9f31cf76299b46..058d7febf5f628d40f253a43a53979ce209b6948 100755 (executable)
--- a/ctdb/tests/INTEGRATION/failover/pubips.040.NoIPTakeover.sh
+++ b/ctdb/tests/INTEGRATION/failover/pubips.040.NoIPTakeover.sh
@@ -22,9 +22,26 @@ echo "Wait until the ips are reallocated"
 sleep_for 30
 try_command_on_node 0 "$CTDB ipreallocate"
 
-num=`try_command_on_node -v 1 "$CTDB ip" | grep -v Public | egrep " 1$" | wc -l`
-echo "Number of addresses on node 1 : $num"
+# sets: num
+count_ips_on_node ()
+{
+       local node="$1"
+
+       ctdb_onnode "$node" ip
+       # outfile is set by ctdb_onnode() above
+       # shellcheck disable=SC2154,SC2126
+       # * || true is needed to avoid command failure when there are no matches
+       # * Using "wc -l | tr -d '[:space:]'" is our standard
+       #   pattern... and "grep -c" requires handling of special case
+       #   for no match
+       num=$(grep -v 'Public' "$outfile" | \
+                     grep " ${node}\$" | \
+                     wc -l | \
+                     tr -d '[:space:]')
+       echo "Number of addresses on node ${node}: ${num}"
+}
 
+count_ips_on_node 1
 
 echo "Turning on NoIPTakeover on all nodes"
 try_command_on_node all "$CTDB setvar NoIPTakeover 1"
@@ -33,12 +50,11 @@ try_command_on_node 1 "$CTDB ipreallocate"
 echo Disable node 1
 try_command_on_node 1 "$CTDB disable"
 try_command_on_node 1 "$CTDB ipreallocate"
-num=`try_command_on_node -v 1 "$CTDB ip" | grep -v Public | egrep " 1$" | wc -l`
-echo "Number of addresses on node 1 : $num"
-[ "$num" != "0" ] && {
-    echo "BAD: node 1 still hosts ip addresses"
-    exit 1
-}
+
+count_ips_on_node 1
+if [ "$num" != "0" ] ; then
+       test_fail "BAD: node 1 still hosts IP addresses"
+fi
 
 
 echo "Enable node 1 again"
@@ -46,12 +62,11 @@ try_command_on_node 1 "$CTDB enable"
 sleep_for 30
 try_command_on_node 1 "$CTDB ipreallocate"
 try_command_on_node 1 "$CTDB ipreallocate"
-num=`try_command_on_node -v 1 "$CTDB ip" | grep -v Public | egrep " 1$" | wc -l`
-echo "Number of addresses on node 1 : $num"
-[ "$num" != "0" ] && {
-    echo "BAD: node took over ip addresses"
-    exit 1
-}
+
+count_ips_on_node 1
+if [ "$num" != "0" ] ; then
+       test_fail "BAD: node 1 took over IP addresses"
+fi
 
 
 echo "OK. ip addresses were not taken over"