On 03/29/2017 11:52 AM, J. Bruce Fields wrote:
On Wed, Mar 29, 2017 at 09:06:50AM -0400, Stephen Gallagher wrote:
> On 03/28/2017 09:54 PM, J. Bruce Fields wrote:
>> I think the thread count is another commonly needed parameter--the
>> default of 8 threads is a little small.
>>
>
> So, this is a feature like the ID-mapping and stuff that I'm beginning
> to think probably should be a separate Ansible module specifically for
> configuring global settings of the NFS server. So perhaps we should
> rename the module described in this proposal to "nfs_share" rather
> than just "nfs" and use the name "nfs" for that purpose.
Note also that a lot of those settings will only take effect on server
restart.
Also, you generally want to restart after an export removal. (The
unexport will work, but often people are unexporting so they can
unmount, and that's not reliable as knfsd can still hold some filesystem
references after the unexport.)
Anyway, maybe none of that's Ansible's problem, but it's worth at least
mentioning in documentation if there's a chance.
Well, if that's a real issue, we can probably have the `unshared` state do a
restart instead of just a reload. Thanks for the feedback!