[Samba] nfs4:acedup question: inconsistent documentation?

[Andrew Bartlett]

On Mon, 2021-05-10 at 08:34 +0200, Remy Zandwijk via samba wrote:
> > > According the the release notes of Samba 4.12.0, the default
> > > setting of nfs4:acedup has been changed to 'merge' (see: 
> > > https://wiki.samba.org/index.php/Samba_4.12_Features_added/changed#smb.conf_changes
> > > <
> > > https://wiki.samba.org/index.php/Samba_4.12_Features_added/changed#smb.conf_changes>
> > > ;).
> > > 
> > > However, the manpage on 
> > > https://www.samba.org/samba/docs/current/man-html/vfs_zfsacl.8.html
> > > <
> > > https://www.samba.org/samba/docs/current/man-html/vfs_zfsacl.8.html
> > > > tells me that the default setting is 'dontcare'. The manpage is
> > > valid for Samba 4.13.3, according to the text in the 'VERSION'
> > > section.
> > > 
> > > 
> > > What to believe? The manpage or the release notes?
> > You need to believe the release notes, the default has changed to
> > 'merge'. The manpage needs updating.
> > 
> > Rowland
> 
> Thanks Rowland. I hope the manpage is going to be updated soon?
> 
> Is there a CLI way to show what the default settings of modules are?
> testparm -v doesn't show default settings for modules.

That is the challenge, the 'parametric' options are declared in the
code, so they don't get picked up by the tests we have which try to
keep all this consistent. 

Could you try fixing this for us, and propose a patch?

https://wiki.samba.org/index.php/Contribute

Andrew Bartlett

-- 
Andrew Bartlett (he/him)       https://samba.org/~abartlet/
Samba Team Member (since 2001) https://samba.org
Samba Team Lead, Catalyst IT   https://catalyst.net.nz/services/samba

Samba Development and Support, Catalyst IT - Expert Open Source
Solutions