[PATCH] Add config file handling to CTDB (finally!)

Martin Schwenke martin at meltin.net
Sat May 12 08:29:58 UTC 2018


On Sat, 12 May 2018 14:31:07 +1000, Amitay Isaacs via samba-technical
<samba-technical at lists.samba.org> wrote:

> Here is a patch set that adds config file handling to CTDB.  The conf
> api allows to define configuration options and validation functions
> for each component.  This also adds a configuration validation tool.
> 
> The idea is to centralize all CTDB configuration. The config file will
> follow the samba config file format.  This will allow us to get rid of
> the adhoc configuration file handling via ctdbd_wrapper for main ctdbd
> and event scripts.
> 
> Other things in the patch set are:
> 
> - abstraction to handle command-line options and commands/arguments for tools
> - abstraction for all the paths (also handles them for testing)
> 
> All the testing now will be based on CTDB_TEST_MODE and the paths will
> be based on CTDB_BASE.
> 
> Please review and push.

Reviewed-by: Martin Schwenke <martin at meltin.net>

I did 2 very minor things:

* Added your Reviewed-by: tag to the "Add config options for logging"
  commit with my authorship.  I can't imagine this being an issue
  because you've looked through it many times and you posted it.  ;-)

* Removed a blank line from the end of path_tool.c

Pushed...

I'm excited...

peace & happiness,
martin



More information about the samba-technical mailing list