drone / charts

Helm charts for the Drone platform on Kubernetes
Other
166 stars 116 forks source link

Invalid healthz #64

Closed loeffel-io closed 2 years ago

loeffel-io commented 2 years ago

Hey, pretty unsure about this but is http://:http/healthz expected at the deployment because my gke health checks are not working (same result local and on gke)

Containers:
   server:
    Image:      drone/drone:2.11.1
    Port:       80/TCP
    Host Port:  0/TCP
    Liveness:   http-get http://:http/healthz delay=0s timeout=1s period=10s #success=1 #failure=3
    Readiness:  http-get http://:http/healthz delay=0s timeout=1s period=10s #success=1 #failure=3
loeffel-io commented 2 years ago

kubectl exec pod/drone-65fbcff65-gbh5l -n drone -- wget -O- -q http://10.24.128.130/healthz puts out OK

loeffel-io commented 2 years ago

After some debugging i realised that the drone helm isn't creating the gke load balancer correctly - the gke health check is pointed to / instead of /healthz - manually switching this to /healthz brings drone to life

I try to find out a solution for this

loeffel-io commented 2 years ago

Solution

I set the ingress backend port to 80 instead of http

spec {
  backend {
    service_name = "drone"
    service_port = "http" # before 80
  }
}

Now GKE creates the correct health check path 👍