[PATCHES] Logging to multiple debug backends

Jeremy Allison jra at samba.org
Tue Mar 17 18:00:15 MDT 2015


On Tue, Mar 17, 2015 at 04:58:23PM -0700, Christof Schmitt wrote:
> On Tue, Mar 17, 2015 at 04:30:53PM -0700, Jeremy Allison wrote:
> > On Tue, Mar 17, 2015 at 04:27:05PM -0700, Christof Schmitt wrote:
> > > 
> > > I assume that you are trying to get the code ready for moving some of
> > > the ctdb logging backends. Since this is only an API internal to
> > > debug.c, we can change that anytime. I would propose that we agree on a
> > > format used in the 'logging' parameter for passing additional options,
> > > and make sure that we don't need to change that format going forward.
> > > Then we could work towards getting the current patch set in master.
> > > 
> > > Moving the ctdb logging backends can be done later. As the required
> > > changes are only internal to debug.c, we can do that at any time.
> > > 
> > > Does this sound like a plan?
> > 
> > Sorry, I've completely lost the plot on this patchset now :-).
> > 
> > Do you have a completed patchset you'd like reviewing ?
> 
> It is only one patchset:
> https://lists.samba.org/archive/samba-technical/2015-March/106224.html
> 
> The only pending change is changing the token syntax in the logging
> parameter, as discussed with Martin, from
> 
> name[:level]
> 
> to
> 
> name[:option][@level]
> 
> The logging config would then look like:
> logging = syslog at 1 file at 5 lttng at 10
> 
> The :option part would be ignored at the moment, but once we look into
> moving some logging code from ctdb under this backend framework, the
> backends from ctdb will use the option field.
> 
> I will add the change probably tomorrow and resubmit the patchset.
> 
Thanks Christof, I am now following along again :-).


More information about the samba-technical mailing list