samba_dnsupdate timeouts (was Re: [PATCH] python indent bugfix in dns_hub.py)

[Volker Lendecke]

On Thu, Feb 07, 2019 at 06:03:07PM +0100, Volker Lendecke via samba-technical wrote:
> On Thu, Feb 07, 2019 at 03:09:03PM +0100, Stefan Metzmacher wrote:
> 
> > The key is that we have this:
> > 
> > while (true) {
> >    msg_ctx = messaging_init();
> >    messaging_register(msg_ctx, NULL, MSG_PONG, pong_message);
> >    messaging_send(msg_ctx, MSG_PING, ...)
> >    TALLOC_FREE(msg_ctx);
> > }
> > 
> > messaging_init() calls messaging_dgm_init() = > socket()/bind()
> > and TALLOC_CTX() calls unlink()
> > 
> > As discussed catching ECONNREFUSED and retry would also be fix for the
> > problem, them my samba_dnsupdate patch is just an optimization.
> 
> Attached find a patchset that *should* fix this issue. gitlab ci
> running.

Sadly this patchset pushes our runtime in gitlab to over 5 hours:

==> /builds/samba-team/devel/samba/samba.stdout <==
[850(7101)/853 at 5h8m2s]
samba.tests.blackbox.bug13653(ad_dc_ntvfs:local)
ERROR: Testsuite[samba.tests.blackbox.bug13653(ad_dc_ntvfs:local)]
REASON: unable to set up environment ad_dc_ntvfs:local - exiting

So this seems to be the wrong approach. In the isolated test run
locally this went fine, but the whole thing just dies under this new
code.

Volker

-- 
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: 0551-370000-0, mailto:kontakt at sernet.de
Gesch.F.: Dr. Johannes Loxen und Reinhild Jung
AG Göttingen: HR-B 2816 - http://www.sernet.de