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

example for LDAP integration with Active Directory #3580

Merged
merged 5 commits into from
Jan 8, 2025

Conversation

Comment on lines 26 to 36

. Inside the container, execute the following steps:
+

.. To set configuration options, open the file:
+
----
/etc/sssd/sssd.conf
----
+
. Set [literal]``krb5_keytab`` in ``domain/$domain`` to [literal]``/etc/sssd/krb5.keytab``
.. Set [literal]``krb5_keytab`` in ``domain/$domain`` to [literal]``/etc/sssd/krb5.keytab``
Copy link
Contributor

Choose a reason for hiding this comment

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

Given the example below I think this part better be removed and replaced by something more generic. It also sends mixed messages where example has different keytab path.

Inside the container edit `/etc/sssd/sssd.conf` according to your configuration. 
See example below.

@keichwa keichwa marked this pull request as draft January 8, 2025 11:53
@keichwa keichwa marked this pull request as ready for review January 8, 2025 13:01
@keichwa keichwa requested a review from aaannz January 8, 2025 13:02
@keichwa
Copy link
Contributor Author

keichwa commented Jan 8, 2025

@aaannz can you please check again?

@keichwa keichwa merged commit ba62449 into master Jan 8, 2025
1 check failed
@keichwa keichwa deleted the admin-sssd-#1233696-keichwa branch January 8, 2025 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants