On 11/17/21 10:54 PM, Dan Smith wrote:
>> I don't think we rely on /healthcheck -- there's nothing healthy about
>> an API endpoint blindly returning a 200 OK.
>>
>> You might as well just hit / and accept 300 as a code and that's
>> exactly the same behaviour. I support what Sean is bringing up here
>> and I don't think it makes sense to have a noop /healthcheck that
>> always gives a 200 OK...seems a bit useless imho
>
> Yup, totally agree. Our previous concerns over a healthcheck that
> checked all of nova returning too much info to be useful (for something
> trying to figure out if an individual worker is healthy) apply in
> reverse to one that returns too little to be useful.
>
> I agree, what Sean is working on is the right balance and that we should
> focus on that.
>
> --Dan
>
That's not the only thing it does. It also is capable of being disabled,
which is useful for maintenance: one can gracefully remove an API node
for removal this way, which one cannot do with the root.
I feel like this should be handled by whatever layer that needs to drain requests for maintenance, otherwise also it might just be the same as turning off the service, no?
Cheers,
Thomas Goirand (zigo)