Nishant Puri has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1233718?usp=email ) Change subject: ganesha_mgr: accept short log component names for get/set log ...................................................................... ganesha_mgr: accept short log component names for get/set log The DBus log interfaces use property names with a "COMPONENT_" prefix (e.g. COMPONENT_FSAL), while the config file uses short names (FSAL, ALL, NFS_V4). ganesha_mgr was passing the component name through to DBus unchanged, so "ganesha_mgr get log FSAL" failed and only "ganesha_mgr get log COMPONENT_FSAL" worked. Add _log_component_prop_name() in ganesha_mgr_utils.py to normalize the component name: if it does not already start with "COMPONENT_", prepend it. Use this in LogManager and CondLogManager Get/Set so both short names and COMPONENT_* names work for "get log", "set log", "get log conditional", and "set log conditional". Change-Id: Icc7ac03d9c2dc9bcc1012705078ba7b8b27d7b4e Signed-off-by: Nishant Puri <npuri@redhat.com> --- M src/scripts/ganeshactl/Ganesha/ganesha_mgr_utils.py 1 file changed, 10 insertions(+), 4 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/18/1233718/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1233718?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Icc7ac03d9c2dc9bcc1012705078ba7b8b27d7b4e Gerrit-Change-Number: 1233718 Gerrit-PatchSet: 1 Gerrit-Owner: Nishant Puri <npuri@redhat.com>