From a62553a6a5c5238d981d67ff6dc938cc04491afd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?G=C3=A1bor=20Antal?= Date: Thu, 10 Dec 2015 17:48:25 +0100 Subject: [PATCH] Remove unreachable code in basic_consolidation.py In basic_consolidation.py there is a method called calculate_score_vm() which had two return statements following each other. The second one (which equals the first one) is unreachable as the first one returns. Change-Id: Ia4877c22188fae6217e07597a2dd939633414349 Closes-Bug: #1524911 --- watcher/decision_engine/strategy/basic_consolidation.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/watcher/decision_engine/strategy/basic_consolidation.py b/watcher/decision_engine/strategy/basic_consolidation.py index 4db16ad2e..cd8dc23e2 100644 --- a/watcher/decision_engine/strategy/basic_consolidation.py +++ b/watcher/decision_engine/strategy/basic_consolidation.py @@ -321,10 +321,6 @@ class BasicConsolidation(BaseStrategy): 0, 0) - return self.calculate_weight(model, vm, total_cores_used, - 0, - 0) - def print_utilization(self, model): if model is None: raise ClusteStateNotDefined()