[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug 62318] healthcheck

On Fri, Aug 24, 2018 at 11:57 AM Christophe JAILLET
<christophe.jaillet@xxxxxxxxxx> wrote:
> Le 24/08/2018 à 16:40, Jim Jagielski a écrit :
> > I was wondering if someone wanted to provide a sanity check
> > on the above PR and what's "expected" by the health check code.
> >
> > It would be very easy to adjust so that hcinterval was not
> > the time between successive checks but the interval between
> > the end of one and the start of another, but I'm not sure that
> > is as useful. In other words, I think the current behavior
> > is right (but think the docs need to be updated), but am
> > willing to have my mind changed :)
> >
> Hi Jim,
> the current behavior is also what I would expect.
> If I configure a check every 10s, I would expect 6 checks each minute,
> even if the test itself takes time to perform.

Bug describes something else IIUC.  Because the watchdog calls us 10
times per second, it continuously sees that the worker hasn't been
health checked within the desired interval and queues up a check, it
doesn't know one is queued.