skupperproject / skupper

Skupper is an implementation of a Virtual Application Network, enabling rich hybrid cloud communication.
http://skupper.io
Apache License 2.0
589 stars 73 forks source link

Kubernetes probes not set on Skupper pods #944

Open rmallam opened 1 year ago

rmallam commented 1 year ago

Hi,

Currently, Skupper pods, router, Service controller and Site controllers dont have liveness and readiness probes set, The pod show healthy irrespective of the health of the applications, This is causing a lot of confusion when debugging issues.

We are using custom certs and made a mistake with one of the cert, it was very hard for us to understand the issue as the pod was showing healthy and it took sometime for us to debug.

Could we add the probes to all the applications that are part of skupper?

grs commented 1 year ago

The router deployment specifies a liveness probe already. The site- and service-controller do not. We can configure probes for those.