Autotest 0.11.1: KVM test: Fixing shutdown test
Only close session after we are sure the guest is down on shutdown test. Backport of a fix present on trunk.
Signed-off-by Lucas Meneghel Rodrigues <[email protected]>
---
client/tests/kvm/kvm_tests.py +15 -12
@@ -74,26 +74,29 @@ if not vm.is_alive(): raise error.TestError("VM seems to be dead; Test requires a living VM") - logging.info("Waiting for guest to be up...") + try: + logging.info("Waiting for guest to be up...") - session = kvm_utils.wait_for(vm.ssh_login, 240, 0, 2) - if not session: - raise error.TestFail("Could not log into guest") + session = kvm_utils.wait_for(vm.ssh_login, 240, 0, 2) + if not session: + raise error.TestFail("Could not log into guest") - logging.info("Logged in") + logging.info("Logged in") - # Send the VM's shutdown command - session.sendline(vm.get_params().get("cmd_shutdown")) - session.close() + # Send the VM's shutdown command + session.sendline(vm.get_params().get("cmd_shutdown")) - logging.info("Shutdown command sent; waiting for guest to go down...") + logging.info("Shutdown command sent; waiting for guest to go down...") - if not kvm_utils.wait_for(vm.is_dead, 240, 0, 1): - raise error.TestFail("Guest refuses to go down") + if not kvm_utils.wait_for(vm.is_dead, 240, 0, 1): + raise error.TestFail("Guest refuses to go down") - logging.info("Guest is down") + logging.info("Guest is down") + finally: + session.close() + def run_migration(test, params, env): """ KVM migration test: