[PATCH] CTDB configuration clean-ups
amitay at gmail.com
Mon Nov 5 23:05:10 UTC 2018
On Mon, Nov 5, 2018 at 11:25 AM Martin Schwenke via samba-technical
<samba-technical at lists.samba.org> wrote:
> This patch set drops ad hoc usage of CTDB_SOCKET and CTDB_PIDFILE
> environment variables. Instead, common path handling is now used
> throughout the code. As documented in tests/README, this
> means that compile-time defaults are used unless CTDB_TEST_MODE and
> CTDB_BASE are set, in which case locations relative to CTDB_BASE are
> However, note that the first commit adds an exception for CTDB_SOCKET to
> the common path handling. This is used when the caller asks for the
> socket path for "ctdbd" (and when CTDB_SOCKET is set).
> Please review and maybe push...
In the patch "ctdb-tests: Use path_socket() in dummy client", there
is no need to set default options.sockpath using
Let the default be NULL. If there is no socket argument, use existing
mem_ctx to set up the default sockpath.
The rest of the patches are fine and Reviewed-by: Amitay Isaacs
<amitay at gmail.com>.
More information about the samba-technical