Added backof to dns health probe

Kuadrant · Dec 21, 2023 · 70ebdd6 · 70ebdd6
1 parent c05ddef
commit 70ebdd6
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/testsuite/policy/dns_policy.py b/testsuite/policy/dns_policy.py
@@ -2,7 +2,9 @@
 from dataclasses import dataclass
 from typing import Optional, Literal
 
+import backoff
 import openshift as oc
+from openshift import OpenShiftPythonException
 
 from testsuite.utils import asdict
 from testsuite.gateway import Referencable
@@ -57,6 +59,7 @@ def set_health_check(self, health_check: HealthCheck):
         """Sets health check for DNSPolicy"""
         self.model["spec"]["healthCheck"] = asdict(health_check)
 
+    @backoff.on_exception(backoff.fibo, OpenShiftPythonException, max_tries=13, jitter=None)
     def get_dns_health_probe(self) -> oc.APIObject:
         """Returns DNSHealthCheckProbe object for the created DNSPolicy"""
         with self.context:

diff --git a/testsuite/tests/mgc/dnspolicy/health_check/conftest.py b/testsuite/tests/mgc/dnspolicy/health_check/conftest.py
@@ -1,5 +1,4 @@
 """Conftest for DNSPolicy health checks"""
-import time
 import pytest
 
 from testsuite.gateway import Hostname
@@ -51,5 +50,4 @@ def dns_policy(dns_policy, health_check):
 @pytest.fixture(scope="module")
 def dns_health_probe(dns_policy, route):  # pylint: disable=unused-argument
     """Wait for health check to start monitoring endpoint and return according DNSHealthCheckProbe object"""
-    time.sleep(10)
     return dns_policy.get_dns_health_probe()