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

Bug health checks fail to create if a hostname is used #91

Closed
maleck13 opened this issue Apr 23, 2024 · 1 comment
Closed

Bug health checks fail to create if a hostname is used #91

maleck13 opened this issue Apr 23, 2024 · 1 comment
Labels

Comments

@maleck13
Copy link
Collaborator

When a health check is reconciled, it will only work if an IP is used. This means when using an AWS loadbalancer no health check can be defined via DNSPolicy. When you define one, it will result in the following error

ERROR	Reconciler error	{"controller": "dnsrecord", "controllerGroup": "kuadrant.io", "controllerKind": "DNSRecord", "DNSRecord": {"name":"prod-web-api","namespace":"ingress-gateway"}, "namespace": "ingress-gateway", "name": "prod-web-api", "reconcileID": "ae615743-522f-4b3d-9ba9-a8e9d27621c8", "error": "InvalidInput: Invalid XML ; javax.xml.stream.XMLStreamException: org.xml.sax.SAXParseException; lineNumber: 1; columnNumber: 401; cvc-datatype-valid.1.2.3: 'ad17cad5895114caca43bdff1e676c2e-1588753257.eu-west-1.elb.amazonaws.com' is not a valid value of union type 'IPAddress'.\n\tstatus code: 400, request id: 6aff7834-14b5-4d55-b882-f693b1803ed8"}sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler	/go/pkg/mod/sigs.k8s.io/[email protected]/pkg/internal/controller/controller.go:329sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem	/go/pkg/mod/sigs.k8s.io/[email protected]/pkg/internal/controller/controller.go:266sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2	/go/pkg/mod/sigs.k8s.io/[email protected]/pkg/internal/controller/controller.go:227

This is because AWS expects an IP for a health check but we are attempting to create it using the host.

Options

  • We should improve the error reporting on DNSRecord and propagate it back to the DNSPolicy
  • Decide if we want to do a look up of the IP of a hostname and use that / those for the health check setup
@maleck13 maleck13 changed the title bug health checks fail if a CNAME host is used Bug health checks fail to create if a CNAME host is used Apr 23, 2024
@maleck13 maleck13 changed the title Bug health checks fail to create if a CNAME host is used Bug health checks fail to create if a hostname is used Apr 23, 2024
@maleck13
Copy link
Collaborator Author

maleck13 commented Jul 1, 2024

Closing as we intend to move away from this design #141

@maleck13 maleck13 closed this as not planned Won't fix, can't repro, duplicate, stale Jul 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

No branches or pull requests

1 participant