From 38f9d65574c327a22ff629f9581fcd5dc4d49132 Mon Sep 17 00:00:00 2001 From: Jon Uriarte Date: Fri, 17 Dec 2021 11:34:08 +0000 Subject: [PATCH] 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 --- kuryr_tempest_plugin/tests/scenario/base.py | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/kuryr_tempest_plugin/tests/scenario/base.py b/kuryr_tempest_plugin/tests/scenario/base.py index be427f99..43c9f6a7 100644 --- a/kuryr_tempest_plugin/tests/scenario/base.py +++ b/kuryr_tempest_plugin/tests/scenario/base.py @@ -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.')