Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add Fedora 18+ and Kerberized NFS4 support #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add Fedora 18+ and Kerberized NFS4 support #9

wants to merge 1 commit into from

Conversation

dmwarren
Copy link

Very nice module, thank you! These changes allow the module to really meet all our site's NFS needs.

New:

  • Remove support for "redhat" in favour of "rhel/centos" and "fedora" in order to support for systemd and new NFS service names on Fedora
  • Manage rpc.gssd, rpc.svcgssd on Red Hat systems (running/enabled/flags stored in /etc/sysconfig/nfs)
  • Manage the Kerberos realm in /etc/idmapd.conf
  • Manage /etc/idmapd.conf with ERB templates instead of augeas

@jhoblitt
Copy link
Contributor

'fedora' is part of $::osfamily == "RedHat' and 'rhel' is not a valid $::operatingsystem. Why was this change needed? It looks like you've just copied the same code over and over for centos/fedora/rhel?

This patch does not update / breaks the spec tests.

@haraldsk
Copy link
Owner

Back from vacation!

There's a lot of good work in here, @dmwarren .

But as @jhoblitt stated the $osfamily stuff and tests need to be fixed before I can merge it.

@traylenator
Copy link
Contributor

RHEL 7 if course similar, though different again.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants