Test suite: Make "ctdb ip" test backward compatible with older ctdb versions.
authorMartin Schwenke <martin@meltin.net>
Wed, 10 Feb 2010 09:27:53 +0000 (20:27 +1100)
committerMartin Schwenke <martin@meltin.net>
Wed, 10 Feb 2010 09:27:53 +0000 (20:27 +1100)
Recent updates to the test meant that it only worked with the latest
ctdb versions.  This changes things so that we never bother matching
the machine readable header, just the actual data in the output.  It
also takes a slightly more liberal approach in massaging the human
readable output to ensure it matches the machine readable output.

Signed-off-by: Martin Schwenke <martin@meltin.net>
tests/simple/11_ctdb_ip.sh

index 59c5d52eac40f956260481ee9b247f11eaaf40eb..e85884541a92905299781fd0d5e0cc3c7074858e 100755 (executable)
@@ -32,21 +32,14 @@ set -e
 cluster_is_healthy
 
 echo "Getting list of public IPs..."
-try_command_on_node -v 1 $CTDB ip -n all
+try_command_on_node -v 1 "$CTDB ip -n all | tail -n +2"
 ips=$(echo "$out" | sed \
-       -e '1d' \
        -e 's@ node\[@ @' \
-       -e 's@\].*$@:@')
-machineout1=":Public IP:Node:ActiveInterface:AvailableInterfaces:ConfiguredInterfaces:"
-machineout2=$(echo "$out" | sed \
-       -e '1d' \
-       -e 's@^@:@' \
-       -e 's@ node\[@:@' \
-       -e 's@\] active\[@:@' \
-       -e 's@\] available\[@:@' \
-       -e 's@\] configured\[@:@' \
-       -e 's@\]$@:@')
-machineout=`echo -e "$machineout1\n$machineout2"`
+       -e 's@\].*$@@')
+machineout=$(echo "$out" | sed -r \
+       -e 's@^| |$@:@g' \
+       -e 's@[[:alpha:]]+\[@@g' \
+       -e 's@\]@@g')
 
 while read ip pnn ; do
     try_command_on_node $pnn "ip addr show"
@@ -60,7 +53,7 @@ done <<<"$ips" # bashism to avoid problem setting variable in pipeline.
 
 [ "$testfailures" != 1 ] && echo "Looks good!"
 
-cmd="$CTDB -Y ip -n all"
+cmd="$CTDB -Y ip -n all | tail -n +2"
 echo "Checking that \"$cmd\" produces expected output..."
 
 try_command_on_node 1 "$cmd"