samba - Jan 2018 - samba_dlz shutting down on named reload

To a user, samba_dlz just doesn't work correctly with bind.  There are two
obvious options
a) bind is broken somehow and unloads all modules when reloading
b) samba_dlz is somehow broken in the way that it interfaces with bind,
perhaps it misinterprets the signal to 'reload' when it receives it from
bind.

As a user, i have no way of knowing which is it is, all that I know is that
sabma_dlz does not work correctly in a specific setting.  Hence, the samba
bug tracker is the right place to report that.
I would expect the samba developers, who know how to build a bind plugin,
to be able to verify if they are implementing the bind API protocol
correctly, and if so, take it upon themselves, having intimate knowledge of
its implementation, get in touch with the authors of bind, to see why their
software doesn't work as advertised.
At any rate, I would expect a comment on the defect to happen during defect
triage.  There is a triage process, right?

On 17 January 2018 at 10:11, Rowland Penny via samba <samba at
lists.samba.org>
wrote:
> On Wed, 17 Jan 2018 09:52:56 +0000
> Kristján Valur Jónsson via samba <samba at lists.samba.org> wrote:
>
> > Well, I created a bug report for this issue.
> > Seems that none of the developers has picked up on it.
> > https://bugzilla.samba.org/show_bug.cgi?id=13214
> >
> >
>
> Probably because you have reported it in the wrong place, this
> is undoubtedly a Bind bug.
>
> Rowland
>
>
> --
> To unsubscribe from this list go to the following URL and read the
> instructions:  https://lists.samba.org/mailman/options/samba
>


-- 
Kv,
Kristján Valur Jónsson, RVX

Well, I had a cursory glance dlz_bind9.c and also the bind9 source code I
could find (
http://users.isc.org/~each/doxygen/bind9/dlz__dlopen__driver_8c-source.html)
and didn´t see any provision for 'reload' in there.
bind is clearly calling the dlz_destroy() function and not dlz_create(), as
one would expect, for a reload.
So yes, it appears to be a shortcoming of bind.
1) I suggest that the wiki for AD DC be updated to point out this fact.
logrotate scripts may use 'reload' which may be broken.  a
'restart' is
better in that case.
2) I'll report this defect to the ISC.

On 17 January 2018 at 13:30, Kristján Valur Jónsson <kristjan at rvx.is>
wrote:
> To a user, samba_dlz just doesn't work correctly with bind.  There are
two
> obvious options
> a) bind is broken somehow and unloads all modules when reloading
> b) samba_dlz is somehow broken in the way that it interfaces with bind,
> perhaps it misinterprets the signal to 'reload' when it receives it
from
> bind.
>
> As a user, i have no way of knowing which is it is, all that I know is
> that sabma_dlz does not work correctly in a specific setting.  Hence, the
> samba bug tracker is the right place to report that.
> I would expect the samba developers, who know how to build a bind plugin,
> to be able to verify if they are implementing the bind API protocol
> correctly, and if so, take it upon themselves, having intimate knowledge of
> its implementation, get in touch with the authors of bind, to see why their
> software doesn't work as advertised.
> At any rate, I would expect a comment on the defect to happen during
> defect triage.  There is a triage process, right?
>
> On 17 January 2018 at 10:11, Rowland Penny via samba <
> samba at lists.samba.org> wrote:
>
>> On Wed, 17 Jan 2018 09:52:56 +0000
>> Kristján Valur Jónsson via samba <samba at lists.samba.org>
wrote:
>>
>> > Well, I created a bug report for this issue.
>> > Seems that none of the developers has picked up on it.
>> > https://bugzilla.samba.org/show_bug.cgi?id=13214
>> >
>> >
>>
>> Probably because you have reported it in the wrong place, this
>> is undoubtedly a Bind bug.
>>
>> Rowland
>>
>>
>> --
>> To unsubscribe from this list go to the following URL and read the
>> instructions:  https://lists.samba.org/mailman/options/samba
>>
>
>
>
> --
> Kv,
> Kristján Valur Jónsson, RVX
>


-- 
Kv,
Kristján Valur Jónsson, RVX

On 17 January 2018 at 16:00, Kristján Valur Jónsson via samba <
samba at lists.samba.org> wrote:
> Well, I had a cursory glance dlz_bind9.c and also the bind9 source code I
> could find (
> http://users.isc.org/~each/doxygen/bind9/dlz__dlopen__driver
> _8c-source.html)
> and didn´t see any provision for 'reload' in there.
> bind is clearly calling the dlz_destroy() function and not dlz_create(), as
> one would expect, for a reload.
> So yes, it appears to be a shortcoming of bind.
> 1) I suggest that the wiki for AD DC be updated to point out this fact.
> logrotate scripts may use 'reload' which may be broken.  a
'restart' is
> better in that case.
> 2) I'll report this defect to the ISC.
>
> On 17 January 2018 at 13:30, Kristján Valur Jónsson <kristjan at
rvx.is>
> wrote:
>
> > To a user, samba_dlz just doesn't work correctly with bind.  There
are
> two
> > obvious options
> > a) bind is broken somehow and unloads all modules when reloading
> > b) samba_dlz is somehow broken in the way that it interfaces with
bind,
> > perhaps it misinterprets the signal to 'reload' when it
receives it from
> > bind.
> >
> > As a user, i have no way of knowing which is it is, all that I know is
> > that sabma_dlz does not work correctly in a specific setting.  Hence,
the
> > samba bug tracker is the right place to report that.
> > I would expect the samba developers, who know how to build a bind
plugin,
> > to be able to verify if they are implementing the bind API protocol
> > correctly, and if so, take it upon themselves, having intimate
knowledge
> of
> > its implementation, get in touch with the authors of bind, to see why
> their
> > software doesn't work as advertised.
> > At any rate, I would expect a comment on the defect to happen during
> > defect triage.  There is a triage process, right?
> >
> > On 17 January 2018 at 10:11, Rowland Penny via samba <
> > samba at lists.samba.org> wrote:
> >
> >> On Wed, 17 Jan 2018 09:52:56 +0000
> >> Kristján Valur Jónsson via samba <samba at lists.samba.org>
wrote:
> >>
> >> > Well, I created a bug report for this issue.
> >> > Seems that none of the developers has picked up on it.
> >> > https://bugzilla.samba.org/show_bug.cgi?id=13214
> >> >
> >> >
> >>
> >> Probably because you have reported it in the wrong place, this
> >> is undoubtedly a Bind bug.
> >>
> >> Rowland
> >>
> >>
> >> --
> >> To unsubscribe from this list go to the following URL and read the
> >> instructions:  https://lists.samba.org/mailman/options/samba
> >>
> >
> >
> >
> > --
> > Kv,
> > Kristján Valur Jónsson, RVX
> >
>
>
>
> --
> Kv,
> Kristján Valur Jónsson, RVX
> --
> To unsubscribe from this list go to the following URL and read the
> instructions:  https://lists.samba.org/mailman/options/samba
>
Hi Kristjan,

This sounds like the issue I have as per https://lists.samba.org/
archive/samba/2017-October/211662.html

My logrotate does a service named reload though.

I will try swapping my reload for restart and see if that fixes things.

Thank you for your legwork.

Kind regards