Skip to content

Commit

Permalink
ipatests: fix healthcheck test without DNS
Browse files Browse the repository at this point in the history
ipa-healthcheck has added a new check for ipa-ca record
missing. The test needs to be adapted to handle the new check.

Fixes: https://pagure.io/freeipa/issue/9459

Signed-off-by: Florence Blanc-Renaud <flo@redhat.com>
Reviewed-By: Rob Crittenden <rcritten@redhat.com>
  • Loading branch information
flo-renaud authored and rcritten committed Oct 4, 2023
1 parent b22605e commit f9075f9
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion ipatests/test_integration/test_ipahealthcheck.py
Original file line number Diff line number Diff line change
Expand Up @@ -1640,13 +1640,19 @@ def test_ipa_dns_systemrecords_check(self):
"Got {count} ipa-ca AAAA records, expected {expected}",
"Expected URI record missing",
}
else:
elif (parse_version(version) < parse_version('0.13')):
expected_msgs = {
"Expected SRV record missing",
"Unexpected ipa-ca address {ipaddr}",
"expected ipa-ca to contain {ipaddr} for {server}",
"Expected URI record missing",
}
else:
expected_msgs = {
"Expected SRV record missing",
"Expected URI record missing",
"missing IP address for ipa-ca server {server}",
}

tasks.install_packages(self.master, HEALTHCHECK_PKG)
returncode, data = run_healthcheck(
Expand Down

0 comments on commit f9075f9

Please sign in to comment.