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