socket_wrapper changes and ctdb
asn at samba.org
Wed Nov 15 09:18:26 UTC 2017
On Wednesday, 15 November 2017 10:04:27 CET Amitay Isaacs wrote:
> Hi Andreas,
> On Wed, Nov 15, 2017 at 2:34 AM, Andreas Schneider via samba-technical <
> samba-technical at lists.samba.org> wrote:
> > Hi,
> > I'm currently trying to move the wrappers to the third_party directory.
> > I've
> > also changed the ctdb wscript but it always fails in autobuild now.
> > It fails with:
> > cp: cannot stat â€˜/memdisk/asn/a/b3961373/ctdb/ctdb/tests/var/
> > unit_eventscripts/etc/iproute2/rt_tablesâ€™: No such file or directory
> > Aborted (core dumped)
> > make: *** [autotest] Error 1
> > which seems to be unrelated. The logs are here:
> > https://git.samba.org/asn/samba-autobuild/ctdb.stdout
> > https://git.samba.org/asn/samba-autobuild/ctdb.stderr
> Looks like you uncovered a race condition in sock_daemon code. I am working
> on some improvements in sock_daemon. Will post the patches in a day or two
> after fixing the race condition.
Thanks, then I will wait with pushing till you fix it.
> > Could you please take a look? The repo with the changes is:
> > https://git.samba.org/?p=asn/samba.git;a=shortlog;h=refs/
> > heads/master-cwrap
> Looks like there are some patches missing from this branch. I cannot
> configure or build ctdb using just your branch. It needs attached patch.
I've applied your patch and pushed it upstream. Only one line applied, maybe I
didn't push the latest version to my git repo, sorry.
> Also, on debian with glibc 2.24 socket_wrapper requires pthread dependency
> for pthread_atfork() function. On fedora 26 with glibc 2.25 it does not
> to require pthread dependency.
I've fixed that already too. Pushed latest version to the git repo now.
Thanks for your help!
Andreas Schneider GPG-ID: CC014E3D
Samba Team asn at samba.org
More information about the samba-technical