Skip to content

Commit

Permalink
azure: ipatests: Skip more healthcheck tests requiring SUCCESS
Browse files Browse the repository at this point in the history
The available free space on Azure's VM is about 20%(time to time),
this makes ipa-healthcheck tests flaky.
  • Loading branch information
stanislavlevin committed Jul 18, 2023
1 parent ed7c937 commit 3e548b6
Showing 1 changed file with 9 additions and 3 deletions.
12 changes: 9 additions & 3 deletions ipatests/test_integration/test_ipahealthcheck.py
Original file line number Diff line number Diff line change
Expand Up @@ -353,6 +353,8 @@ def test_human_output(self):
https://pagure.io/freeipa/issue/8892
"""
if not_enough_disk_space(self.master, "/"):
pytest.skip("requires at least 20% of free space for '/'")
returncode, output = run_healthcheck(self.master, output_type="human",
failures_only=True)
assert returncode == 0
Expand Down Expand Up @@ -580,6 +582,8 @@ def test_ipahealthcheck_ca_not_configured(self):
self.replicas[0], 'ipauser1', first='Test', last='User',
)

if not_enough_disk_space(self.replicas[0], "/"):
pytest.skip("requires at least 20% of free space for '/'")
returncode, data = run_healthcheck(self.replicas[0],
failures_only=True)
assert returncode == 0
Expand Down Expand Up @@ -687,11 +691,11 @@ def test_ipa_healthcheck_no_errors(self):
Ensure that on a default installation with KRA and DNS
installed ipa-healthcheck runs with no errors.
"""
if not_enough_disk_space(self.master, "/"):
pytest.skip("requires at least 20% of free space for '/'")

cmd = tasks.install_kra(self.master)
assert cmd.returncode == 0

if not_enough_disk_space(self.master, "/"):
pytest.skip("requires at least 20% of free space for '/'")
returncode, _unused = run_healthcheck(
self.master,
failures_only=True
Expand All @@ -716,6 +720,8 @@ def test_ipa_healthcheck_no_errors_with_overrides(self):
'output_type=human'
])
)
if not_enough_disk_space(self.master, "/"):
pytest.skip("requires at least 20% of free space for '/'")
returncode, output = run_healthcheck(
self.master, failures_only=True, config=config_file
)
Expand Down

0 comments on commit 3e548b6

Please sign in to comment.