From 13b89c8dd2bfa04d5b09aaaa699c698c2b1831d0 Mon Sep 17 00:00:00 2001 From: Fanis Kalimullin Date: Mon, 7 Aug 2017 08:47:09 +0000 Subject: [PATCH] get_config_opts method was overwritten outlet_temperature strategy relies on a datasource config parameter, which can be either "ceilometer" or "gnocchi". This patch overrides get_config_opts method of base class to allow specify datasource. Change-Id: I551401039e26816568a04c7f2151d5b3c7ed269a Closes-Bug: #1709024 --- .../strategy/strategies/outlet_temp_control.py | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/watcher/decision_engine/strategy/strategies/outlet_temp_control.py b/watcher/decision_engine/strategy/strategies/outlet_temp_control.py index bbafd02b0..bc657ef98 100644 --- a/watcher/decision_engine/strategy/strategies/outlet_temp_control.py +++ b/watcher/decision_engine/strategy/strategies/outlet_temp_control.py @@ -30,6 +30,7 @@ telemetries to measure thermal/workload status of server. import datetime +from oslo_config import cfg from oslo_log import log from watcher._i18n import _ @@ -160,6 +161,16 @@ class OutletTempControl(base.ThermalOptimizationBaseStrategy): def granularity(self): return self.input_parameters.get('granularity', 300) + @classmethod + def get_config_opts(cls): + return [ + cfg.StrOpt( + "datasource", + help="Data source to use in order to query the needed metrics", + default="ceilometer", + choices=["ceilometer", "gnocchi"]) + ] + def calc_used_resource(self, node): """Calculate the used vcpus, memory and disk based on VM flavors""" instances = self.compute_model.get_node_instances(node)