From dc9ef6f49cbbe595d8e1501acb5aeee64cfe4d17 Mon Sep 17 00:00:00 2001 From: Alexander Chadin Date: Tue, 20 Feb 2018 13:52:54 +0300 Subject: [PATCH] workload_stabilization trivial fix This fix allows to compare metric name by value, not by object. Change-Id: I57c50ff97efa43efe4fd81875e481b25e9a18cc6 --- .../strategy/strategies/workload_stabilization.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/watcher/decision_engine/strategy/strategies/workload_stabilization.py b/watcher/decision_engine/strategy/strategies/workload_stabilization.py index 1817275c0..c7e36a50d 100644 --- a/watcher/decision_engine/strategy/strategies/workload_stabilization.py +++ b/watcher/decision_engine/strategy/strategies/workload_stabilization.py @@ -301,7 +301,7 @@ class WorkloadStabilization(base.WorkloadStabilizationBaseStrategy): s_host_vcpus = new_hosts[src_node.uuid]['vcpus'] d_host_vcpus = new_hosts[dst_node.uuid]['vcpus'] for metric in self.metrics: - if metric is 'cpu_util': + if metric == 'cpu_util': new_hosts[src_node.uuid][metric] -= ( self.transform_instance_cpu(instance_load, s_host_vcpus)) new_hosts[dst_node.uuid][metric] += (