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 | |
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
-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 " |