Re: backend/director admin states

2018-05-03 Thread Geoff Simmons
On 05/03/2018 10:49 AM, Dridi Boukelmoune wrote: >> Illustrated example: varnishadm backend.list output >> >> Backend name Admin Probe Last change >> boot.b1 probe 0/8 badWed, 02 May 2018 13:22:10 >> GMT >> boot.b2

Re: backend/director admin states

2018-05-03 Thread Poul-Henning Kamp
OK, let me try to explain my mental model of this: Admin health is owned by the CLI and director implementations do not get to meddle with it. (But they do get to see it) Admin=sick means sick, no matter what the director implementation thinks. Admin=healthy means healthy, --//--

Re: backend/director admin states

2018-05-03 Thread Dridi Boukelmoune
> Best proposal for "???" seems to be "auto", even though it is not > entirely on point. > > The implementation can control its view of the health two ways > > A) Call VRT_SetHealth() and set it on the director. > (useful for probes as we know them) > > B) Provide a ->healthy() method

Re: backend/director admin states

2018-05-03 Thread Poul-Henning Kamp
In message <1401d370-56dd-6006-2b3d-21771650f...@uplex.de>, Geoff Simmons write s: >>> Illustrated example: varnishadm backend.list output We should also think about varnishadm backend.list -j ? -- Poul-Henning Kamp | UNIX since Zilog Zeus 3.20 p...@freebsd.org | TCP/IP

Re: backend/director admin states

2018-05-03 Thread Poul-Henning Kamp
In message

Re: backend/director admin states

2018-05-03 Thread Guillaume Quintard
Yp, that's on my plate. Which is rather large... -- Guillaume Quintard On Thu, May 3, 2018, 13:04 Poul-Henning Kamp wrote: > > In message c5q-zxfqjcu0k...@mail.gmail.com> > , Guillaume Quintard writes: > > >All commands

Re: backend/director admin states

2018-05-03 Thread Poul-Henning Kamp
In message , Guillaume Quintard writes: >All commands should have a JSON option, I reckon. Somebody should write more code for it then, I reckon :-) -- Poul-Henning Kamp | UNIX since Zilog Zeus 3.20

Re: backend/director admin states

2018-05-03 Thread Dridi Boukelmoune
On Thu, May 3, 2018 at 1:04 PM, Poul-Henning Kamp wrote: > > In message > > , Guillaume Quintard writes: > >>All commands should have a JSON option, I reckon. > > Somebody should write more code

Re: backend/director admin states

2018-05-03 Thread Guillaume Quintard
All commands should have a JSON option, I reckon. -- Guillaume Quintard On Thu, May 3, 2018, 12:52 Poul-Henning Kamp wrote: > > In message <1401d370-56dd-6006-2b3d-21771650f...@uplex.de>, Geoff Simmons > write > s: > > >>> Illustrated example: varnishadm