[nova][all] Adding /healthcheck support in Nova, and better healthcheck in every projects
mnaser at vexxhost.com
Thu Nov 18 01:03:02 UTC 2021
On Wed, Nov 17, 2021 at 5:52 PM Thomas Goirand <zigo at debian.org> wrote:
> 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?
> Thomas Goirand (zigo)
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the openstack-discuss