Skip to content

Commit

Permalink
ipatests: Fix expectation about GSS error in test for healthcheck
Browse files Browse the repository at this point in the history
As of 1.19.1 MIT krb changed the error returned if no valid
credentials could be obtained(GSS_S_CRED_UNAVAIL->GSS_S_NO_CRED).
To be compatible with previous versions of krb the new expected
error message has been added.

Fixes: https://pagure.io/freeipa/issue/8737
Signed-off-by: Stanislav Levin <[email protected]>
Reviewed-By: Alexander Bokovoy <[email protected]>
Reviewed-By: Rob Crittenden <[email protected]>
  • Loading branch information
stanislavlevin authored and flo-renaud committed Mar 5, 2021
1 parent 87304c7 commit 2c4a160
Showing 1 changed file with 14 additions and 6 deletions.
20 changes: 14 additions & 6 deletions ipatests/test_integration/test_ipahealthcheck.py
Original file line number Diff line number Diff line change
Expand Up @@ -483,11 +483,19 @@ def test_source_ipahealthcheck_ipa_host_check_ipahostkeytab(
ipahealthcheck.ipa.host when dirsrv service is stopped and
running on IPA master
"""
msg = (
"Failed to obtain host TGT: Major (851968): "
"Unspecified GSS failure. "
"Minor code may provide more information, "
"Minor (2529638972): Generic error (see e-text)"
msgs = (
(
"Failed to obtain host TGT: Major (851968): "
"Unspecified GSS failure. "
"Minor code may provide more information, "
"Minor (2529638972): Generic error (see e-text)"
),
(
"Failed to obtain host TGT: Major (458752): "
"No credentials were supplied, or the credentials "
"were unavailable or inaccessible, "
"Minor (2529638972): Generic error (see e-text)"
),
)
restart_service(self.master, "dirsrv")
dirsrv_ipactl_status = 'Directory Service: STOPPED'
Expand All @@ -502,7 +510,7 @@ def test_source_ipahealthcheck_ipa_host_check_ipahostkeytab(
assert returncode == 1
if dirsrv_ipactl_status in result.stdout_text:
assert data[0]["result"] == "ERROR"
assert data[0]["kw"]["msg"] == msg
assert data[0]["kw"]["msg"] in msgs
else:
assert data[0]["result"] == "SUCCESS"

Expand Down

0 comments on commit 2c4a160

Please sign in to comment.