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

Support el8 #69

Merged
merged 3 commits into from
Jun 16, 2020
Merged

Support el8 #69

merged 3 commits into from
Jun 16, 2020

Conversation

anders-larsson
Copy link
Contributor

First commit is to ensure spec tests continue to work for ruby 2.1.9.

Adds support for EL8-based distributions.

@anders-larsson
Copy link
Contributor Author

Oops. I had totally missed #68. Well now you have two pull requests for EL8 @ghoneycutt.

Thanks for your work.

@ghoneycutt
Copy link
Owner

Is nscd even used on EL8? I thought it was deprecated in EL7 in favor of sssd.

@anders-larsson
Copy link
Contributor Author

Hm. Yes it appears that they think sssd and nscd might duplicate caching functionality. I do think it serves a purpose though when used in combination with other applications.

@anders-larsson
Copy link
Contributor Author

How would you like us to proceed with this pull request?

Thanks and Best Regards,

@braddeicide
Copy link
Contributor

In the Redhat documentation they say not to use nscd and sssd together for passwd, group, services and in 8 they say sssd is enabled by default. I don't see any solution other than nscd for hosts however. I believe nscd should work with redhat/centos 8 however only hosts should be enabled by default (at all really)

enable-cache hosts yes
enable-cache passwd no
enable-cache group no
enable-cache netgroup no
enable-cache services no

@anders-larsson
Copy link
Contributor Author

anders-larsson commented Jun 16, 2020

Hi @braddeicide,

Thank you for your contribution. It has been merged. I did squash your commits and change the commit message a bit though. I hope it is OK.

@ghoneycutt ghoneycutt merged commit 4608e91 into ghoneycutt:master Jun 16, 2020
@ghoneycutt
Copy link
Owner

Thanks everyone!!

@ghoneycutt
Copy link
Owner

Released in 1.14.0

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.

3 participants