-
Notifications
You must be signed in to change notification settings - Fork 256
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tests: Update ldap test to use journal utility. #7622
Conversation
src/tests/system/tests/test_ldap.py
Outdated
) | ||
assert log.rc == 0, "'Password change failed.' message is not in log!" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Patrik and Dan added is_match
that returns bool for this case:
match = client.journald.is_match("pam_sss(passwd:chauthtok): User info message: Password change failed.", unit="sssd")
assert match, msg
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
92ff101
to
0fbf38d
Compare
8b29ce0
to
3fb1e64
Compare
3fb1e64
to
d52457d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Failures are not related.
The test is freezing in idmci due to journalctl being in interactive mode.
Switch to use journal utility instead.