Deploying the helm chart 0.6.1 with githubwebjhookserver fails as /github-webhook-server is not found in the container
docker run --rm -it --entrypoint /github-webhook-server summerwind/actions-runner-controller:v0.17.0
docker: Error response from daemon: OCI runtime create failed: container_linux.go:370: starting container process caused: exec: "/github-webhook-server": stat /github-webhook-server: no such file or directory: unknown.
@Puneeth-n Thanks for reporting! In short - Could you try setting image.tag to canary on install?
I should have added a big note about that, but it requires the unreleased version of actions-runner-controller image, which is only available via canary tag today. Also see https://github.com/summerwind/actions-runner-controller/issues/366#issuecomment-789418036
@mumoshu latest tage works. BTW is there any health check route? I can't seem to find one. I am running on EKS and the root path seems to return 500
@Puneeth-n Thanks for confirming! Re: health check route, good point. We don't have one today. Probably the best thing you could do right now would be to use TCP check on the root path.
I'm assuming that you're trying to setup health-checking from ELB/ALB/NLB to the github webhook server, right?
At least we should add readinessProbe to the github-webhook-server pods, so that your NodePort service is able to stop flowing traffic to not ready pods.
Would you need another kind of health check, e.g. for setting up ELB?
I'm assuming that you're trying to setup health-checking from ELB/ALB/NLB to the github webhook server, right?
@mumoshu yea for ALB. May be I will use the /metrics endpoint and ovverride the healthcheck port for now.
I'm assuming that you're trying to setup health-checking from ELB/ALB/NLB to the github webhook server, right?
@mumoshu yea for ALB. May be I will use the
/metricsendpoint and ovverride the healthcheck port for now.
never mind. does not work
@Puneeth-n #378 adds the universal GET handler thanks to @robwhitby, which can also be used for health checks from LBs.
Would you mind giving it a shot with the canary controller image? Currently, GET /, GET /ok, or whatever path works.
Thanks. I will give it a spin
@Puneeth-n #378 adds the universal GET handler thanks to @robwhitby, which can also be used for health checks from LBs.
Would you mind giving it a shot with the
canarycontroller image? Currently,GET /,GET /ok, or whatever path works.
@mumoshu works! :) Thanks
Most helpful comment
@Puneeth-n Thanks for reporting! In short - Could you try setting
image.tagtocanaryon install?I should have added a big note about that, but it requires the unreleased version of actions-runner-controller image, which is only available via
canarytag today. Also see https://github.com/summerwind/actions-runner-controller/issues/366#issuecomment-789418036