samba.git/ctdb/tests/tool, branch master

ctdb-tests: Make the fake log timestamp string easy to modify

2014-10-06T10:34:32+00:00

Use a variable to allow easy change of this string in case future
logging changes modify the timestamp format or do not support
timestamping.

Signed-off-by: Martin Schwenke 
Reviewed-by: Volker Lendecke

ctdb-tools-ctdb: Make natgwlist and lvsmaster more resilient

2014-05-29T03:59:37+00:00

Recent changes have caused these commands to attempt to get
capabilities from all nodes before doing further filtering.  This
means that capabilities are unnecessarily fetched from nodes that are
unlikely to be the master.  If such a node does not answer the control
then many nodes can fail to calculate the master node.  In the case of
natgwlist this will cause "monitor" events to fail resulting in
unhealthy nodes.

Restore the behaviour where capabilities are only fetched for a node
that will be the master if it has the desired flags.

Although this masks a problem where a connected node is not replying,
it can help to avoid an outage in some cases.

Add supporting tests and infrastructure.  Infrastructure just lets a
timeout be faked - just for ctdb_ctrl_getcapabilities_stub() so far.
First test checks that this infrastructure works if the first node
times out in natgwlist.  Second test checks the case worked around by
the above fix - that is, no failure when a node with PNN beyond the
NATGW master can time out.

Signed-off-by: Martin Schwenke 
Reviewed-by: Amitay Isaacs 

Autobuild-User(master): Amitay Isaacs 
Autobuild-Date(master): Thu May 29 05:59:37 CEST 2014 on sn-devel-104

ctdb-tools-ctdb: Drop disconnected nodes when filtering by capability

2014-04-14T01:52:40+00:00

Commit ba69742ccd822562ca2135d2466e09bf1216644b missed the point of
filtering disconnected nodes while limiting the nodemap to those in
the NAT gateway group.  It was really to avoid trying to fetch
capabilities from disconnected nodes.  This should be explicitly done
in filter_nodemap_by_capabilities(), otherwise "ctdb natgwlist" simply
fails when there is a disconnected node.

Note that the alternate solution where filter_nodemap_by_flags() is
called before filter_nodemap_by_capabilities() would not be not
correct.  Filtering on flags first can produce a "healthier" set of
nodes where none of them have the NAT gateway capability.

Also extend stub for ctdb_ctrl_getcapabilities() to fail when trying
to get capabilities from a disconnected node and add a corresponding
test to confirm that "ctdb natgwlist" is no longer broken.

Signed-off-by: Martin Schwenke 
Reviewed-by: Amitay Isaacs

ctdb-tests: Add "ctdb listnodes" and "ctdb xpnn" stub tests

2014-03-23T03:20:14+00:00

Tests for xpnn need to implement a stub for ctdb_sys_have_ip().  The
cheapest way of doing this is to read a fake nodemap using the
existing code and check if the IP of the "current" node is the one
being asked about.  However, the fake state initialisation isn't
currently available to without_daemon commands because it is meant to
represent daemon state.  However, it can be made available by moving
the relevant code into a new stub for tevent_context_init().  The stub
still needs to initialise a tevent context - this can be done by
calling a lower level function.

Signed-off-by: Martin Schwenke 
Reviewed-by: Amitay Isaacs

ctdb-tests: New natgwlist tests where nodes capability not set

2014-03-23T03:20:14+00:00

Signed-off-by: Martin Schwenke 
Reviewed-by: Amitay Isaacs

ctdb-tests: Update ctdb stub LVS tests and add some new ones

2014-03-23T03:20:14+00:00

Signed-off-by: Martin Schwenke 
Reviewed-by: Amitay Isaacs

ctdb-tests: Support fake capabilities in CTDB tool stub

2014-03-23T03:20:14+00:00

... and add a test to make sure it works.

Signed-off-by: Martin Schwenke 
Reviewed-by: Amitay Isaacs

ctdb-tests: Remove old, unused copy of a CTDB tool unit test

2014-03-23T03:20:14+00:00

This looks to have got left behind a long time ago when things got
moved around...

Signed-off-by: Martin Schwenke 
Reviewed-by: Amitay Isaacs

ctdb:tests: Rework unit test result filtering

2013-11-27T17:46:17+00:00

Using a variable is too fragile, so use a function instead.

Signed-off-by: Martin Schwenke 
Reviewed-by: Michael Adam

tests/tool: Remove references in libctdb in file and function names

2013-10-04T05:15:35+00:00

Main changes are:

  libctdb_test.c -> ctdb_test_stubs.c
  ctdb_tool_libctdb.c -> ctdb_functest.c

ctdb_tool_stubby.c is gone, replaced with existing ctdb_test.c.

Functions starting with "libctdb_test_" now start with
"ctdb_test_stubs_".

Signed-off-by: Martin Schwenke 
Pair-programmed-with: Amitay Isaacs 

(This used to be ctdb commit 6182bd0c19f215a997efe5272e633b1b1bd0c882)