charts icon indicating copy to clipboard operation
charts copied to clipboard

Published 20 hours ago • verified publisher icon drone

Be a TLS enabled LoadBalancer

Open gaima8 opened this issue 3 years ago • 1 comments

We're using drone as a LoadBalancer service with TLS enabled via the certs extraVolume. There are two problems with this.

  1. There is only one ports to be open via the service, and is hardcoded to be HTTP. The servicePorts {} map makes ports totally configurable.
  2. The liveness probe is hardcoded to talk HTTP to the port HTTP, but with TLS enabled drone responds with a HTTP 307 to plan HTTP requests, causing the liveness probe to fail. I'm using the existence of a certs volume to decide if TLS is enabled, and if so do HTTPS liveness probes.

gaima8 avatar Apr 01 '21 12:04 gaima8

Sorry for the delay here.

I think this would be a nice feature to add. If you can resolve the conflicts, I will review. Thanks!

jimsheldon avatar Oct 21 '22 17:10 jimsheldon