From 659cbf320749cc13b3fc03a1b059ce88e95d1f75 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Vincent=20Fran=C3=A7oise?= Date: Fri, 27 Jan 2017 14:58:34 +0100 Subject: [PATCH] Idiomatic loop for calculate_num_migrations Change-Id: Iddcb6dfbed2d2793df9c33408858f1a8d4600d10 --- .../strategy/strategies/basic_consolidation.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/watcher/decision_engine/strategy/strategies/basic_consolidation.py b/watcher/decision_engine/strategy/strategies/basic_consolidation.py index c8586d6d8..e96f499cc 100644 --- a/watcher/decision_engine/strategy/strategies/basic_consolidation.py +++ b/watcher/decision_engine/strategy/strategies/basic_consolidation.py @@ -415,11 +415,11 @@ class BasicConsolidation(base.ServerConsolidationBaseStrategy): sorted_score): number_migrations = 0 for mig_instance, __ in sorted_instances: - for j in range(0, len(sorted_score)): + for node_uuid, __ in sorted_score: mig_source_node = self.compute_model.get_node_by_uuid( node_to_release) mig_destination_node = self.compute_model.get_node_by_uuid( - sorted_score[j][0]) + node_uuid) result = self.check_migration( mig_source_node, mig_destination_node, mig_instance)