diff options
author | Rafael Weingärtner <rafael@apache.org> | 2022-10-12 17:15:52 -0300 |
---|---|---|
committer | Rafael Weingärtner <rafael@apache.org> | 2022-10-12 17:45:06 -0300 |
commit | 44caf759d09eea7a55f4f1aeccea8e257b357e13 (patch) | |
tree | 751121be5454ed604949014a15b72b0e400b2f6c /ceilometer | |
parent | 5106f4acc46f87bccfc2174840a9ec29145be0dd (diff) | |
download | ceilometer-44caf759d09eea7a55f4f1aeccea8e257b357e13.tar.gz |
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
Diffstat (limited to 'ceilometer')
-rw-r--r-- | ceilometer/polling/dynamic_pollster.py | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/ceilometer/polling/dynamic_pollster.py b/ceilometer/polling/dynamic_pollster.py index 0030c607..07edc5ee 100644 --- a/ceilometer/polling/dynamic_pollster.py +++ b/ceilometer/polling/dynamic_pollster.py @@ -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 " |