Skip to content
This repository has been archived by the owner on Mar 8, 2023. It is now read-only.

Enable idmapd service on Ubuntu when NFSv4 configured; disable when not. #54

Merged
merged 3 commits into from
Jan 7, 2016

Conversation

dnlsng
Copy link

@dnlsng dnlsng commented Jan 7, 2016

In the case of $nfs::client::ubuntu::nfs_v4 = false, Puppet would stop the idmapd service but when the host is rebooted, the service will start again as it wasn't disabled. The inverse is also true for $nfs::client::ubuntu::nfs_v4 = true. This caused puppet to make a change on the first run after every reboot. Adding the enable parameter to the service resources should fix this.

dwerder and others added 3 commits October 16, 2015 14:42
Backmerge Develop for release 1.7.0
merge develop branch for 1.7.1
If the idmapd service is disabled, make sure it is enabled on Ubuntu if
NFSv4 is configured. Also make sure that it is disabled if NFSv4 is not
configured otherwise it would start on boot.
@dwerder
Copy link
Member

dwerder commented Jan 7, 2016

Thanks for the PR. Would you like to add teh same change for debian too? Would be nice.

@dwerder dwerder self-assigned this Jan 7, 2016
@dwerder dwerder added the bug label Jan 7, 2016
@dwerder
Copy link
Member

dwerder commented Jan 7, 2016

My fault. Debian dont have this service issue.

dwerder added a commit that referenced this pull request Jan 7, 2016
Enable idmapd service on Ubuntu when NFSv4 configured; disable when not.
@dwerder dwerder merged commit 21c2a1d into echocat:develop Jan 7, 2016
@dnlsng
Copy link
Author

dnlsng commented Jan 7, 2016

No problem. I thought I checked all of the services but missed that Debian one for some reason.

On 7 Jan 2016, at 19:04, Daniel Werdermann notifications@github.com wrote:

Thanks for the PR. Would you like to add teh same change for debian too? Would be nice.


Reply to this email directly or view it on GitHub.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants