From b829fae6320c5798990f7d8bbd829d22839a6234 Mon Sep 17 00:00:00 2001 From: Liam Young Date: Tue, 5 Jul 2022 12:37:28 +0100 Subject: [PATCH] Fix VM retryer (#812) I recently added an option to have zaza delete and relaunch a guest if it failed to be provisioned or failed a connectivity test. However, in the code to remove the an instance I accidentaly called the method that checks that a resource has gone and not the method that actually does the removal. This patch fixes that. NOTE: Only the Trilio tests use this retryer function atm --- zaza/openstack/configure/guest.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/zaza/openstack/configure/guest.py b/zaza/openstack/configure/guest.py index 90e28e9..9ceeaf8 100644 --- a/zaza/openstack/configure/guest.py +++ b/zaza/openstack/configure/guest.py @@ -68,7 +68,7 @@ def launch_instance_retryer(instance_key, **kwargs): keystone_session = openstack_utils.get_overcloud_keystone_session() nova_client = openstack_utils.get_nova_session_client(keystone_session) vm = nova_client.servers.find(name=vm_name) - openstack_utils.resource_removed( + openstack_utils.delete_resource( nova_client.servers, vm.id, msg="Waiting for the Nova VM {} to be deleted".format(vm.name))