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