Skip to content
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

Install required "sssd-ldap" package for 389ds #12313

Merged
merged 1 commit into from Apr 15, 2021
Merged

Conversation

rfan1
Copy link
Contributor

@rfan1 rfan1 commented Apr 15, 2021

Based on bug 1184692, the required "sssd-ldap" package
is not installed as dependency of package "389-ds" and
"sssd" in newer SLES15SP3 drop, so fix it in code.
At the same time, add some post fail hook scripts to collect logs.

Based on bug 1184692, the required "sssd-ldap" package
is not installed as dependency of package "389-ds" and
"sssd" in newer SLES15SP3 drop, so fix it in code
@rfan1 rfan1 changed the title (WIP)Add post fail hook for 389ds Install required "sssd-ldap" package for 389ds Apr 15, 2021
Copy link
Contributor

@lilyeyes lilyeyes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@lilyeyes
Copy link
Contributor

lilyeyes commented Apr 15, 2021

Only one tip: Is sssd-ldap package also available on TW?

@rfan1
Copy link
Contributor Author

rfan1 commented Apr 15, 2021

Only one tip: Is sssd-ldap package also available on TW?

Nice comment, it is there: https://openqa.opensuse.org/tests/1699471#step/tls_389ds_sssd_client/2,
So, I don't think we need to re-run it on TW :)

Copy link
Contributor

@jouyingbin jouyingbin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. thanks for the change.

@coolgw coolgw merged commit ec6749b into os-autoinst:master Apr 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants