From 5e2301b8a75d10e5cbbe11e26e5192b894af6ad7 Mon Sep 17 00:00:00 2001 From: Stephen Gallagher Date: Thu, 24 Sep 2009 08:59:03 -0400 Subject: Temporarily disable automatic config file reread The backends do not honor the reloadConfig SBUS message right now, so if an admin changes the sssd.conf file, it will update only the monitor, potentially leaving the SSSD as a whole in a bad state. This patch will simply comment out monitor_config_file() for the time being until https://fedorahosted.org/sssd/ticket/91 is fixed. --- server/monitor/monitor.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) (limited to 'server/monitor/monitor.c') diff --git a/server/monitor/monitor.c b/server/monitor/monitor.c index e6b6902ed..fa9eb0e82 100644 --- a/server/monitor/monitor.c +++ b/server/monitor/monitor.c @@ -1871,12 +1871,18 @@ int monitor_process_init(struct mt_ctx *ctx, int ret, i; struct sss_domain_info *dom; +#if 0 + This feature is incomplete and can leave the SSSD in a bad state if the + config file is changed while the SSSD is running. + + Uncomment this once the backends are honoring reloadConfig() + /* Watch for changes to the confdb config file */ ret = monitor_config_file(ctx, ctx, config_file, monitor_signal_reconf); if (ret != EOK) { return ret; } - +#endif /* Watch for changes to the DNS resolv.conf */ ret = monitor_config_file(ctx, ctx, RESOLV_CONF_PATH, monitor_update_resolv); -- cgit