openstack
/
kuryr-tempest-plugin
Code Issues Proposed changes

Small improvement in method definition and reference 

Clean-up in get_pod_readiness() method definition and change a reference to
get_pod_readiness() by get_container_readiness().

Change-Id: Icf2b7b99d5142298bd9a11d79ae05e83932d270a
This commit is contained in:
Jon Uriarte 2021-12-17 11:34:08 +00:00
parent e1fdcc0a5b
commit 38f9d65574
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
kuryr_tempest_plugin/tests/scenario/base.py
View File

@ -291,11 +291,9 @@ class BaseKuryrScenarioTest(manager.NetworkScenarioTest):
return (num_pods == ready_pods)
@classmethod
def get_pod_readiness(cls, pod_name, namespace="default",
container_name=None):
def get_pod_readiness(cls, pod_name, namespace="default"):
LOG.info("Checking if pod {} is ready".format(pod_name))
return cls.get_readiness_state(pod_name, namespace=namespace,
container_name=container_name)
return cls.get_readiness_state(pod_name, namespace=namespace)
@classmethod
def get_container_readiness(cls, pod_name, namespace="default",
@ -1420,7 +1418,7 @@ class BaseKuryrScenarioTest(manager.NetworkScenarioTest):
# Wait until kuryr-controller pools are reloaded, i.e.,
# kuryr-controller is ready
res = test_utils.call_until_true(
self.get_pod_readiness, 60, 1, kuryr_pod_name,
self.get_container_readiness, 60, 1, kuryr_pod_name,
namespace=system_namespace, container_name='controller')
self.assertTrue(res, 'Timed out waiting for '
'kuryr-controller to reload pools.')