Merge pull request #10593 from vavuthu/remove_double_retry

remove double retry decorator
red-hat-storage · Oct 14, 2024 · 7686692 · 7686692
2 parents add6461 + 4a8205d
commit 7686692
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/ocs_ci/ocs/resources/mcg.py b/ocs_ci/ocs/resources/mcg.py
@@ -882,14 +882,13 @@ def exec_mcg_cmd(self, cmd, namespace=None, use_yes=False, **kwargs):
  result.stderr = result.stderr.decode()
  return result
 
+ @property
  @retry(
- (CommandFailed, subprocess.TimeoutExpired),
- tries=5,
- delay=15,
+ exception_to_check=(CommandFailed, KeyError, subprocess.TimeoutExpired),
+ tries=10,
+ delay=6,
  backoff=1,
  )
- @property
- @retry(exception_to_check=(CommandFailed, KeyError), tries=10, delay=6, backoff=1)
  def status(self):
  """
  Verify the status of NooBaa, and its default backing store and bucket class