[PATCH] Improving CTDB logging

Martin Schwenke martin at meltin.net
Mon Dec 5 07:09:36 UTC 2016


On Mon, 5 Dec 2016 16:17:40 +1100, Amitay Isaacs <amitay at gmail.com>
wrote:

> Hi,
> 
> Here is a patchset that gets rid of CTDB's special DEBUG levels and
> switches over to using Samba's DEBUG levels,  It also gets rid of special
> debug_string handling in the DEBUG_CALLBACK backends of CTDB.  This brings
> us closer to using Samba's DEBUG as is without requireing special
> DEBUG_CALLBACK backends.
> 
> The main ctdb daemon handles the logging for all processes (except recovery
> daemon).  This is undesirable and each process ideally does it's own
> logging.  The logging api is now refactored to allow any ctdb process to do
> its own logging.
> 
> However, we cannot yet utilize Samba's DEBUG since it's broken and Martin
> is working on it.
> 
> Here are the major changes in this patchset.
> 
> - Drop tevent debugging (it's not needed anymore)
> - Switch to using Samba's DEBUG levels
> - Refactor logging api to common/ from server/ so all processes can use it
> - Get rid of debug_extra
> - Add new ctdb_vfork_exec() to replace ctdb_vfork_with_logging()
> 
> I have already added Reviewed-by tags for Maritn's patches.
> 
> Please review and push.

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

Pushed...

peace & happiness,
martin



More information about the samba-technical mailing list