From 715f6fa1cd2deb9d103d286795243b2ec663c446 Mon Sep 17 00:00:00 2001 From: licanwei Date: Tue, 29 Nov 2016 10:20:23 +0800 Subject: [PATCH] Modify the variable assignment errors The values of 'released_compute_nodes_count' and 'instance_migrations_count' are upside down Change-Id: I0662bdfce575de529eb8c12363be7fa196b1a88c --- .../strategy/strategies/vm_workload_consolidation.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/watcher/decision_engine/strategy/strategies/vm_workload_consolidation.py b/watcher/decision_engine/strategy/strategies/vm_workload_consolidation.py index 75069edd8..d65081759 100644 --- a/watcher/decision_engine/strategy/strategies/vm_workload_consolidation.py +++ b/watcher/decision_engine/strategy/strategies/vm_workload_consolidation.py @@ -534,10 +534,9 @@ class VMWorkloadConsolidation(base.ServerConsolidationBaseStrategy): LOG.debug(info) def post_execute(self): - # self.solution.efficacy = rcu_after['cpu'] self.solution.set_efficacy_indicators( - released_compute_nodes_count=self.number_of_migrations, - instance_migrations_count=self.number_of_released_nodes, + released_compute_nodes_count=self.number_of_released_nodes, + instance_migrations_count=self.number_of_migrations, ) LOG.debug(self.compute_model.to_string())