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

NAS-128571 / 24.10 / Add initial round of changes for SSSD support #13623

Merged
merged 2 commits into from
Apr 26, 2024

Conversation

anodos325
Copy link
Contributor

This commit provides equivalent functionality from SSSD that we were getting from nss-pam-ldapd (nslcd)

  • remove nslcd from build and runtime requirements
  • add truenas-sssd to build and runtime requirements
  • slightly refactor LDAP plugin to restart new sssd service rather than nslcd service.
  • remove mako file for nslcd and create one for sssd
  • remove murmurhash3 python implementation and use one provided by sssd.

TODO:

  • add user and group UI caching for LDAP users
  • improve SSSD status checks
  • add migration to comment-out auxiliary parameters for LDAP plugin

This commit provides equivalent functionality from SSSD that
we were getting from nss-pam-ldapd (nslcd)

* remove nslcd from build and runtime requirements
* add truenas-sssd to build and runtime requirements
* slightly refactor LDAP plugin to restart new sssd service
  rather than nslcd service.
* remove mako file for nslcd and create one for sssd
* remove murmurhash3 python implementation and use one provided
  by sssd.

TODO:
- add user and group UI caching for LDAP users
- improve SSSD status checks
- add migration to comment-out auxiliary parameters for LDAP plugin
@bugclerk
Copy link
Contributor

@bugclerk bugclerk changed the title Add initial round of changes for SSSD support NAS-128571 / 24.10 / Add initial round of changes for SSSD support Apr 26, 2024
Copy link
Contributor

@mgrimesix mgrimesix left a comment

Choose a reason for hiding this comment

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

We can clean up comments, etc later.

@anodos325 anodos325 merged commit f9bf608 into master Apr 26, 2024
2 checks passed
@anodos325 anodos325 deleted the add-sssd branch April 26, 2024 21:15
@bugclerk
Copy link
Contributor

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators Apr 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants