x/tobiko
Code Issues Proposed changes

Merge "Reduce test_reboot_computes_recovery duration"

Browse Source
This commit is contained in:
Zuul 2020-04-17 00:33:47 +00:00 committed by Gerrit Code Review
commit 856e6460fb
2 changed files with 6 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
tobiko/tests/faults/ha/test_cloud_recovery.py
View File

@ -29,6 +29,7 @@ def overcloud_health_checks(passive_checks_only=False):
# verify VM status is updated after reboot
nova.wait_for_all_instances_status('SHUTOFF')
nova.start_all_instances()
containers.list_node_containers.cache_clear()
containers.assert_all_tripleo_containers_running()
containers.assert_equal_containers_state()

12
tobiko/tripleo/containers.py
View File

@ -41,9 +41,10 @@ def get_container_runtime_name():
container_runtime_name = get_container_runtime_name()
def list_node_containers(client):
@functools.lru_cache()
def list_node_containers(ssh_client):
"""returns a list of containers and their run state"""
client = get_container_client(ssh_client=ssh_client)
if container_runtime_module == podman:
return container_runtime_module.list_podman_containers(client=client)
@ -87,9 +88,7 @@ def list_containers(group=None):
openstack_nodes = topology.list_openstack_nodes(group='overcloud')
for node in openstack_nodes:
ssh_client = node.ssh_client
container_client = get_container_client(ssh_client)
node_containers_list = list_node_containers(client=container_client)
node_containers_list = list_node_containers(ssh_client=node.ssh_client)
containers_list.extend(node_containers_list)
return containers_list
@ -115,8 +114,7 @@ def assert_containers_running(group, expected_containers, full_name=True):
openstack_nodes = topology.list_openstack_nodes(group=group)
for node in openstack_nodes:
container_client = get_container_client(node.ssh_client)
node_containers = list_node_containers(client=container_client)
node_containers = list_node_containers(ssh_client=node.ssh_client)
containers_list_df = pandas.DataFrame(
get_container_states_list(node_containers),
columns=['container_host', 'container_name', 'container_state'])