openstack
/
ceilometer
Code Issues Proposed changes

Fix the handle of plain text outputs 

The handler of Plaintext output was using the wrong data structure to store the output in plaintext. The output needs to be enclosed as a dictionary in a list.

Change-Id: Ib134a5a0de135eb1e00073c4dfc6654ad2d93709
This commit is contained in:
Rafael Weingärtner 2022-10-12 17:15:52 -03:00
parent 5106f4acc4
commit 44caf759d0
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
ceilometer/polling/dynamic_pollster.py
View File

@ -66,11 +66,11 @@ class JsonResponseHandler(object):
class PlainTextResponseHandler(object):
"""This response handler converts a string to a dict {'out'=<string>}"""
"""Response handler converts string to a list of dict [{'out'=<string>}]"""
@staticmethod
def handle(response):
return {'out': str(response)}
return [{'out': str(response)}]
VALID_HANDLERS = {
@ -218,6 +218,8 @@ class PollsterSampleExtractor(object):
extra_metadata = self.definitions.retrieve_extra_metadata(
kwargs['manager'], pollster_sample, kwargs['conf'])
LOG.debug("Extra metadata [%s] collected for sample [%s].",
extra_metadata, pollster_sample)
for key in extra_metadata.keys():
if key in metadata.keys():
LOG.warning("The extra metadata key [%s] already exist in "