Nishant Puri has uploaded this change for review.
log: reset conditional logging state on config reload
On SIGHUP/reload, global_client_ip_list and global_export_id_list were
never cleared before re-parsing the LOG { Conditional {} } block. This
caused two bugs: stale client/export entries persisted if removed from
config, and reloading with unchanged entries triggered duplicate errors
that aborted the entire reload.
Extract reset_conditional_logging_state() to clear both global lists,
reset all conditional component log levels to NIV_FULL_DEBUG, reset
match policy to COND_LOG_MATCH_ANY, and set conditional_logging_configured
to false. Call it at the start of read_log_config() so every reload
starts from a clean slate and applies only what is present in the current
config file. Simplify dbus_conditional_log_reset() to reuse the same helper
instead of duplicating the logic.
Also add missing CONF_INDEX_TOKEN entries for COMPONENT_RDMA and
COMPONENT_GRPC in conditional_params[] to prevent them from being
incorrectly forced to NIV_NULL when not explicitly configured.
Change-Id: I4b1c2ca0cc3cefca556839ead49f03d234725efd
Signed-off-by: Nishant Puri <npuri@redhat.com>
---
M src/log/log_functions.c
1 file changed, 64 insertions(+), 36 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/43/1241843/1
To view, visit change 1241843. To unsubscribe, or for help writing mail filters, visit settings.