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(a)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(a)redhat.com>