samba - Sep 2019 - Samba 4.11 "Stale file handle"

Thank you for noticing. I've inherited the config setup and it hasn't
been updated for a long while. Still the changes had no bearing on the
original problem of "stale file handle", not that I expected them to,
it was worth a try though.

On Sun, Sep 22, 2019 at 11:33 AM Rowland penny via samba
<samba at lists.samba.org> wrote:
>
> On 22/09/2019 10:17, Kacper via samba wrote:
> > [global]
> > workgroup = example
> > realm = example.test
> > netbios name = server1
> > server role = active directory domain controller
> > log file = /var/log/samba/smbd.log
> >
> > server services = s3fs, rpc, nbt, wrepl, ldap, cldap, kdc, drepl,
> > winbindd, ntp_signd, kcc, dnsupdate
> > domain logons = Yes
> > wins support = Yes
> > security = user
> > preferred master = auto
> > domain master = yes
> > local master = yes
> > unix extensions = no
> >
> > interfaces = 127.0.0.1 192.168.0.30
> > bind interfaces only = yes
> >
> > [netlogon]
> > path = /usr/local/samba/var/locks/sysvol/example.test/scripts
> > read only = No
> > browsable = No
> >
> > [sysvol]
> > path = /usr/local/samba/var/locks/sysvol
> > read only = No
> > inherit acls = Yes
> > dos filemode = true
> > force unknown acl user = true
> > browsable = No
> >
> > [profiles]
> > path = /usr/local/samba/var/locks/profiles
> > read only = no
> > create mask = 0600
> > directory mask = 0700
> > dos filemode = true
> > browsable = No
> >
> > [share]
> > path = /share
> > read only = No
> > create mask = 0666
> > directory mask = 0777
> > inherit permissions = Yes
> > inherit acls = Yes
> > browsable = Yes
> > hide unreadable = no
> > follow symlinks = yes
> > wide links = yes
> >
> Kacper, that is a smb.conf from a 'borked' DC, you cannot have a DC
set
> up like that.
>
> Remove these lines from [global]
>
> domain logons = Yes
> wins support = Yes
> security = user
> preferred master = auto
> domain master = yes
> local master = yes
> unix extensions = no
>
> Remove these lines from [sysvol]
>
> inherit acls = Yes
> dos filemode = true
> force unknown acl user = true
> browsable = No
>
> Remove these lines from [profiles]
>
> create mask = 0600
> directory mask = 0700
> dos filemode = true
> browsable = No
>
> Remove these lines from [share]
>
> create mask = 0666
> directory mask = 0777
> inherit permissions = Yes
> inherit acls = Yes
> browsable = Yes
> hide unreadable = no
> follow symlinks = yes
> wide links = yes
>
> Then read these:
>
> https://wiki.samba.org/index.php/Setting_up_a_Share_Using_Windows_ACLs
>
> https://wiki.samba.org/index.php/Roaming_Windows_User_Profiles
>
> Rowland
>
>
>
> --
> To unsubscribe from this list go to the following URL and read the
> instructions:  https://lists.samba.org/mailman/options/samba

On 22/09/2019 11:28, Kacper wrote:
> Thank you for noticing. I've inherited the config setup and it
hasn't
> been updated for a long while. Still the changes had no bearing on the
> original problem of "stale file handle", not that I expected them
to,
> it was worth a try though.
>
'stale file handle' is usually a problem with nfs and it means that the 
script is trying to read something, that has been moved/renamed, by its 
old name and this no longer exists.

What does your script do, perhaps there is another way to do it.

Rowland

On 22/09/2019 14:00, Kacper wrote:
> Just tested this out on a completely new Centos VM and the issue is
> still there. The script is just a minimal working example to
> demonstrate the bug. mkstemp is for example a part of Ansible's copy
> module, which we use extensively.
>
And has nothing to do with Samba. I fail to see how something that your 
script does, has anything to do with Samba, your script could be backing 
up anything.

Rowland

I beg to differ since I've identified the commit that caused this
regression to be:

a3caf4140c01136a412c129cae28b433e33ab014 is the first bad commit
commit a3caf4140c01136a412c129cae28b433e33ab014
Author: Ralph Boehme <slow at samba.org>
Date:   Thu Jun 27 14:46:37 2019 +0200

    s3:smbd: use stored file_id in SMB_VFS_FS_FILE_ID()

    Signed-off-by: Ralph Boehme <slow at samba.org>
    Reviewed-by: Jeremy Allison <jra at samba.org>

On Sun, Sep 22, 2019 at 3:16 PM Rowland penny via samba
<samba at lists.samba.org> wrote:
>
> On 22/09/2019 14:00, Kacper wrote:
> > Just tested this out on a completely new Centos VM and the issue is
> > still there. The script is just a minimal working example to
> > demonstrate the bug. mkstemp is for example a part of Ansible's
copy
> > module, which we use extensively.
> >
> And has nothing to do with Samba. I fail to see how something that your
> script does, has anything to do with Samba, your script could be backing
> up anything.
>
> Rowland
>
>
>
> --
> To unsubscribe from this list go to the following URL and read the
> instructions:  https://lists.samba.org/mailman/options/samba