Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(load-balancer): Different health check options per service/port #412

Open
Tracked by #612
DarkZatarra opened this issue Mar 24, 2023 · 4 comments
Open
Tracked by #612
Labels
enhancement New feature or request pinned

Comments

@DarkZatarra
Copy link

DarkZatarra commented Mar 24, 2023

We have the following code:

kind: Service
metadata:
  name: abcd-com
  namespace: abcd-prod
  annotations:
    load-balancer.hetzner.cloud/name: "farm-waf-razdon-com"
    load-balancer.hetzner.cloud/location: fsn1
    load-balancer.hetzner.cloud/protocol: "tcp"
    load-balancer.hetzner.cloud/uses-proxyprotocol: "true"
    load-balancer.hetzner.cloud/use-private-ip: "true"
    load-balancer.hetzner.cloud/disable-public-network: "false"
    load-balancer.hetzner.cloud/health-check-protocol: "http"
    load-balancer.hetzner.cloud/health-check-http-validate-certificate: "true"
    load-balancer.hetzner.cloud/health-check-http-domain: "abcd.com"
    load-balancer.hetzner.cloud/health-check-http-path: "/hcheck"
spec:
  selector:
    app: abcd-com
  ports:
    - port: 443
      targetPort: 443
      name: "https"
    - port: 80
      targetPort: 80
      name: "http"
  type: LoadBalancer

And we want to have the possibility to run a check on http without validate-certificate and a check on https with validate-certificate.
We tried the following:

kind: Service
metadata:
  name: abcd-com
  namespace: abcd-prod
  annotations:
    load-balancer.hetzner.cloud/name: "farm-waf-razdon-com"
    load-balancer.hetzner.cloud/location: fsn1
    load-balancer.hetzner.cloud/protocol: "tcp"
    load-balancer.hetzner.cloud/uses-proxyprotocol: "true"
    load-balancer.hetzner.cloud/use-private-ip: "true"
    load-balancer.hetzner.cloud/disable-public-network: "false"
    load-balancer.hetzner.cloud/health-check-protocol-443: "https"
    load-balancer.hetzner.cloud/health-check-http-validate-certificate-443: "true"
    load-balancer.hetzner.cloud/health-check-http-domain-443: "abcd.com"
    load-balancer.hetzner.cloud/health-check-http-path-443: "/hcheck"
    load-balancer.hetzner.cloud/health-check-protocol-80: "http"
    load-balancer.hetzner.cloud/health-check-http-validate-certificate-80: "false"
    load-balancer.hetzner.cloud/health-check-http-domain-80: "abcd.com"
    load-balancer.hetzner.cloud/health-check-http-path-80: "/hcheck"
spec:
  selector:
    app: abcd-com
  ports:
    - port: 443
      targetPort: 443
      name: "https"
    - port: 80
      targetPort: 80
      name: "http"
  type: LoadBalancer

But it's seems it's not supported.

There real question is how can we control the healthcheck options per port?

Thanks!

P.S. in first code quote the health-check-protocol set to https gives error - that's why we've set it to http

@samcday
Copy link
Contributor

samcday commented Apr 17, 2023

#324 seems somewhat related to this: making it possible to more finely tune the healthcheck configuration.

@github-actions

This comment was marked as off-topic.

@github-actions github-actions bot added the stale label Jun 16, 2023
@apricote apricote removed the stale label Jun 16, 2023
@github-actions

This comment was marked as off-topic.

@github-actions github-actions bot added the stale label Aug 16, 2023
@apricote apricote removed the stale label Aug 16, 2023

This comment was marked as off-topic.

@github-actions github-actions bot added the stale label Nov 15, 2023
@apricote apricote added enhancement New feature or request pinned and removed stale labels Nov 15, 2023
@apricote apricote changed the title Unable to run healthcheck with different options for http and https feat(load-balancer): Different health check options per service/port Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pinned
Projects
None yet
Development

No branches or pull requests

3 participants