openstack
/
ceilometer
Code Issues Proposed changes

Only print polling.yaml file contents as DEBUG 

Since this file can contain sensitive information such
as admin credentials of target resources (e.g. snmp) it
is not a good idea to print this log as INFO since admins
might want to have INFO level logging but without the risk
of it containing this kind of information.

We might want to consider remmoving this log altogether.

Change-Id: Iff5fcc4b6c30142f2c5c9ad26027dc5a6542a881
Closes-Bug: #1811098
This commit is contained in:
Edward Hope-Morley 2019-01-10 16:15:16 +00:00
parent 818dc0f1e0
commit 8881a42af1
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ceilometer/agent.py
View File

@ -61,7 +61,7 @@ class ConfigManagerBase(object):
__name__, 'pipeline/data/' + cfg_file)
with open(cfg_loc) as fap:
conf = yaml.safe_load(fap)
LOG.info("Config file: %s", conf)
LOG.debug("Config file: %s", conf)
return conf