New version 1.0.86

update the man pages with the "getreclock" and "setreclock" commands.

Do not allow the "VerifyRecoveryLock" tunable to be changed if there is no reclock file

disable VerifyRecoveryLock when the user modifies the filename

add a control to set the reclock file

update the recovery daemon to read the recovery lock file off the main daemon and handle when the file is changed/enabled/disabled

return NULL and not a "" when there is no reclock file returned from the server

add a control to read the current reclock file from a node

Document that you can run ctdb without a reclock file in the sysconfig file

Allow setting the recovery lock file as "", which means that we do not use a file and that we implicitely also disable the recovery lock checking.

Update the init script to allow starting without a reclock file.

Dont access the reclock file at all if VerifyRecoveryLock is zero and also
make sure the reclock file is closed if the variable is cleared at runtime

Merge root@10.1.1.27:/shared/ctdb/ctdb-git

new version 1.0.85

rename 99.routing to 11.routing so that it executed before the service scripts

new version 1.0.85

rename 99.routing to 11.routing so the eventscript is processed before
NFS and LVS

Fix minor problem in previous initscript commit.

The valgrind start case should not use daemon, since this is specific
to Red Hat.

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

Initscript fixes, mostly for "stop" action.

Use a local variable $ctdbd so that we always run ctdbd from the the
same place and so that we know what to kill.  This variable respects
the $CTDBD environment variable, which may be used to specify an
alternative location for the daemon.

In the important cases use "pkill -0 -f" to check if ctdbd is
running.  Also, remove the special case for killing ctdbd when running
under valgrind.  The regular case will handle this just fine.

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

Clean up handling the of CTDB restarts in testcases.

Glitches during restarts of the CTDB cluster have been causing some
tests to fail.  This is because restarts are initiated in the body of
many tests.  This adds a simple function ctdb_restart_when_done, which
schedules a restart using an existing hook in the test exit code.
This function is now used in tests that need to restart CTDB.

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

Fix minor onnode bugs relating to local daemons.

Commit a0f5148ac749758e2dfbd6099e829c5bf1d900e6 caused a subtle
regression.  Due to the subtlety, this description is much longer than
the 1 line patch that fixes it!  The regression, where a process that
invokes onnode is unexpectedly blocked, is only apparent if the
following conditions are met:

1. $CTDB_NODES_SOCKETS is set;
2. The command passed to onnode attempts to background a process; and
3. onnode is run in certain types of subshell (e.g. foo=$(onnode ...)).

In particular, when testing against local daemons (i.e. condition (1)
is met), tests/simple/07_ctdb_process_exists.sh would fail (because it
does both (2), (3)).

The problem is caused by the use of file descriptor 3 in the code that
allows separate filtering of stdout and stderr.  A backgrounded
process will have this descriptor open and the $(...) construct
appears to wait for all file descriptors to be closed.  This only
happens with local daemons because SSH is replaced by a shell and file
descriptor 3 leaks into that shell.  It does not occur when SSH is
used because the file descriptor does not leak into the remote shell
where the process is backgrounded.

The fix is simply to redirect file descriptor 3 to /dev/null in the
fakessh function, which is used when $CTDB_NODES_SOCKETS is set.

Also fixed is another minor bug when the -o option and
$CTDB_NODES_SOCKETS are used in combination.  The code uses the node
name as a suffix for the output filename(s).  Usually this is an IP
address.  However, when $CTDB_NODES_SOCKETS is in use the node name is
the socket name, which might be a path several directories deep.
Each output file is created via a simple redirection and this would
fail if unexpected directories appear in the filename.  3 possible
fixes were considered:

1. Replace all '/'s in the node name by '_'s.  Nice and simple.
2. Use the basename of the node name.  However, sockets may be in
   different directories but have the same basename.
3. Create all required directories before redirecting.  This is a
   little more complex and probably doesn't meet the user's
   expectations.

Option (1) is implemented here.

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

dont log an error if waitpid returns -1 and errno is ECHILD

dont leak file descriptors when set recmdoe timesout

dont leak file descriptors

in the recovery daemon, check that the recovery master can access the recovery lock file and verify it is not stale from a child process.
This allows us to timeout the operation if the underlying filesystem has become temporarily unresponsive without causing a new recovery.

reduce the timeout we wait for the reclock child process to finish to 5 seconds
before we log an error and abort

increase the timeout before we shutdown when ther ecovery daemon is hung

rename 99.routing to 11.routing
so it is executed before any of the service scripts

New tests for NFS and CIFS tickles.

New tests/complex/ subdirectory contains 2 new tests to ensure that
NFS and CIFS connections are tracked by CTDB and that tickle resets
are sent when a node is disabled.

Changes to ctdb_test_functions.bash to support these tests.

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

Increase threshold in 51_ctdb_bench from 2% to 5%.

The threshold for the difference in the number messages sent in either
direction around the ring of nodes was set to 2%.  Something
environmental is causing this different to sometimes be as high as 3%.
We're confident it isn't a CTDB issue so we're increasing the
threshold to 5%.

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

When we ban a node, only drop the IPs on the node being banned, not on every node

remove unused variable

dont require particular values for NoIPFailback and DeterministicIPs when
using ctdb moveip

improve ctdb moveip so that it does not always trigger a recovery.

try avoiding to cause a recovery when deleting a public ip from a node

when adding an ip, try manually adding and takingover the ip instead of triggering a full recovery to do the same thing

dont list DELETED nodes in the ctdb listnodes output

make it possible to run 'ctdb listnodes' also if the daemon is not running.
in this case, read the nodes file directly instead of asking the local daemon for the list.

add an option -Y to provide machinereadable output to listnodes

From William Jojo <w.jojo[AT]hvcc.edu>

AIX dont have getopt.h by default.
Dont try including this file when building on AIX

new version 1.0.84

teach ONNODE about deleted nodes

new version 1.0.83

idocument how to remove a node from an existing cluster using 'ctdb
reloadnodes'

hide all DELETED nodes from the ctdb command output

lower the loglevel when we long that we skip an evenscript because it is not executable

dont try to queue packets for sending to (recently) deleted nodes since these nodes do not have a queue.

when building the initial vnnmap, make sure to skip any deleted nodes

use num_nodes and the nodes array instead of walking the vnnmap
when counting the number of active nodes

add a new node state : DELETED.

This is used to mark nodes as being DELETED internally in ctdb
so that nodes are not renumbered if / when they are removed from the nodes file.

This is used to be able to do "ctdb reloadnodes" at runtime without
causing nodes to be renumbered.
To do this, instead of deleting a node from the nodes file, just comment it out like

   1.0.0.1
   #1.0.0.2
   1.0.0.3

After removing 1.0.0.2 from the cluster,  the remaining nodes retain their
pnn's from prior to the deletion, namely 0 and 2

Any line in the nodes file that is commented out represents a DELETED pnn

dont remove the socket when the dameon stops. This can race if the
service is immediately restarted

New attempt at TDB transaction nesting allow/disallow.

Make the default be that transaction is not allowed and any attempt to create a nested transaction will fail with TDB_ERR_NESTING.

If an application can cope with transaction nesting and the implicit
semantics of tdb_transaction_commit(), it can enable transaction nesting
by using the TDB_ALLOW_NESTING flag.

Revert "we only need to have transaction nesting disabled when we start the new transaction for the recovery"

This reverts commit bf8dae63d10498e6b6179bbacdd72f1ff0fc60be.

Revert "set the TDB_NO_NESTING flag for the tdb before we start a transaction from within recovery"

This reverts commit 1b2029dbb055ff07367ebc1f307f5241320227b2.

Revert "add TDB_NO_NESTING. When this flag is set tdb will not allow any nested transactions and tdb_transaction_start() will implicitely _cancel() any pending transactions before starting any new ones."

This reverts commit 459e4ee135bd1cd24c15e5325906eb4ecfd550ec.

remove the obsolete ipmux component.
this is replaced by LVS since a long time

fix the git path to the repository

install the 31.clamd script as 644 by default

add 31.clamd to the install and the rpm

From Flavio Carmo Junior <carmo.flavio@gmail.com>

Add an eventscript to manage ClamAV

From Flavio Carmo Junior <carmo.flavio@gmail.com>
(with modifications)

Add a webpage about CLAMAV support in CTDB

document the new support for ClamAV

fix re pattern to accept the new recovery lock times in the statistics output

change the socket we use for sending grautious ARPs from AF_INET/SOCK_PACKET to AF_PACKET/SOCK_RAW

Whitespace changes and using the CTDB_NO_MEMORY() macro changes to
the previous patch.

add missing checks on so far ignored return values

Most of these were found during a review by Jim Meyering <meyering@redhat.com>

structure member node_list_file is not used anywhere

structure member logfile is not used anywhere

fix a configure warning while checking for netfilter.h

added a missing dependency

Change the loglevel of "registered tcp client for ..." to INFO
instead of ERR

From : Flavio Carmo Junior <carmo.flavio@gmail.com>

Add a helper function that checks whether a unix domain socket exists
and there is a daemon LISTENING to it  similar to the existing function
to check for a daemon LISTENING to a tcp/ip socket.

Fix http://ctdb.samba.org/download.html

Remove error messages about a non-existing /var/log/log.ctdb when running ctdb with logging to syslog

add additional log info to track if/why we cant switch to client mode.

Track how long it takes to take out the recovery lock from both the main dameon and also from the recovery daemon.
Log this in "ctdb statistics".

Also add a varaible "RecLockLatencyMs" that will log an error everytime it takes longer than this to access the reclock file.

new version 1.0.82

use scope host when adding the interface to loopback so we dont respond to ARPs for this ip

change the prefix NATGW_ to CTDB_NATGW_

ping pong: fix logic for mmap reads vs. preads

Michael

maketarball.sh: add GPL license header

Michael

makerpms.sh: add GPL license header

Michael

Remove generated binary files.

Noted by Mathieu Parent <math.parent@gmail.com>

Michael

remove NATGW_PRIVATE_IFACE from the documentation since we do not need
it any more.

assign the natgw address to loopback and not the private network so that natgw will still work even when public and private networks are one and the same

add extra debug statements to the log to make it easier to see when a recovery dameon has hung due to the underlying filesystem hanging.

check that a node is banned before trying to unban it.

In 51_ctdb_bench.sh now allows a 2% difference between positive and
negative.  ctdb_bench.c checks to ensure the timer has advanced from 0
before dividing.

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

Avoid floating point divide by 0 in ctdb_fetch.c's bench_fetch().

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

Bug fixes for tests: simple/12_ctdb_getdebug.sh and scripts/test_wrap.

simple/12_ctdb_getdebug.sh now recognises output with multi-digit node
numbers.

Sharing the ctdb directory via NFS and testing on a real cluster by
setting CTDB_TEST_REAL_CLUSTER didn't work by default.  The fix is to
hack scripts/test_wrap so that it tries to find a valid bin directory
next to the directory containing it is in.

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

From: Sumit Bose <sbose@redhat.com>

 fix handling of AC_INIT

Fix lvsmaster and natgwlist nodespecs.

They both need to use a -Y option to ctdb and for natgwlist we only
want the 1st line.

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

Updated onnode docs to reflect recent changes.

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

New lvs/lvsmaster and natgw/natgwlist nodespecs for onnode.

Some code re-factoring to implement this and to make it easy to
implement new ones.  New simpler implementation of echo_nth() no
longer uses deleted get_nth() function.

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

New option "-o <prefix>" saves stdout from each node to file <prefix>.<ip>.

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

Use ctdb_fetch_lock rather than ctdb_call.

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

41.httpd event script workaround for RHEL5-ism.

RHEL5 can SIGKILL httpd when stopping it, causing it to leak
semaphores.  This means that eventually a node runs out of semaphores
and httpd can't be started.  So, before we attempt to start httpd we
clean up any semaphores owned by apache.  We also try to restart httpd
in the monitor event if httpd has gone away.

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

Add a -Y machinereadable flag to "lvsmaster"

in the "lvsmaster" command, return -1 if there is no lvsmaster

 new version 1.0.81

From: Sumit Bose <sbose@redhat.com>

fix handling of AC_INIT and read version from ctdb.spec

ping_pong: add GPL comment header with Tridge's copyright

Michael