From caf8c4dc68a729940b1ff0a38ea2c39f61546345 Mon Sep 17 00:00:00 2001 From: Anton Beloglazov Date: Mon, 20 Apr 2015 21:55:16 +1000 Subject: [PATCH] Added a check for non-empty list of VMs to migrate in the global manager --- neat/globals/manager.py | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/neat/globals/manager.py b/neat/globals/manager.py index 2b3382e..44af1d2 100644 --- a/neat/globals/manager.py +++ b/neat/globals/manager.py @@ -383,6 +383,10 @@ def execute_underload(config, state, host): if not vm in vms_ram: del vms_to_migrate[i] + if not vms_to_migrate: + log.info('No VMs to migrate - completed the underload request') + return state + for vm in vms_cpu.keys(): if not vm in vms_ram: del vms_cpu[vm] @@ -545,6 +549,10 @@ def execute_overload(config, state, host, vm_uuids): if not vm in vms_ram: del vms_to_migrate[i] + if not vms_to_migrate: + log.info('No VMs to migrate - completed the overload request') + return state + for vm in vms_cpu.keys(): if not vm in vms_ram: del vms_cpu[vm]