[SSSD] [PATCHES] Allow changing log-level without restarting SSSD

Jakub Hrozek jhrozek at redhat.com
Wed May 4 15:00:49 UTC 2011


On 05/02/2011 08:28 PM, Stephen Gallagher wrote:
> These patches are dependent on the patch provided in my earlier email
> "Override config file debug_level with command-line"
> 
> Patch 0002: Create common sss_monitor_init()
> 
> This was implemented almost identically for both the responders
> and the providers. It is easier to maintain as a single routine.
> 
> This patch also adds the ability to provide a private context to
> attach to the sbus_connection for later use.
> 
> 

I think that sss_monitor_init() should either store conn into *mon_conn
or we should remove the struct sbus_connection* from structures that
contain them - they appear not to be used anywhere.

> 
> Patch 0003: Allow changing the log level without restart
> 
> We will now re-read the confdb debug_level value when processing
> the monitor_common_logrotate() function, which occurs when the
> monitor receives a SIGHUP.
> 
> 
> 
> For now, this can be tested by using ldbedit
> on /var/lib/sss/db/confdb.ldb to change the debug_level values and then
> send 'kill -HUP $(pidof sssd)'. In the future, I will write a
> command-line tool to effect this change more easily.
> 

This one looks good to me, ack

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 261 bytes
Desc: OpenPGP digital signature
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20110504/9767bdd4/attachment.sig>


More information about the sssd-devel mailing list