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

Missing critical annotations for managing load balancers #405

Open
danielwoodz opened this issue Nov 17, 2022 · 3 comments
Open

Missing critical annotations for managing load balancers #405

danielwoodz opened this issue Nov 17, 2022 · 3 comments

Comments

@danielwoodz
Copy link

danielwoodz commented Nov 17, 2022

FEATURE REQUEST

The load-balancer annotations only cover like 10% of the configurable surface area of a load balancer.

Critical areas that are not covered:

  • Cannot set allowed cipher suite names
  • Cannot set minimum TLS version
  • Cannot set load balancer to use existing certificate manager certificate ids (should support certificate_ids like terraform resource.oci_load_balancer_listener)
  • Cannot set route rules (ex: redirect all HTTP to HTTPS at the LB, restrict http methods at the LB instead of allowing that into the cluster before being done)
  • Cannot set the display name the load balancer
  • Cannot set the accepted hostnames at the load balancer
  • Cannot associate multiple listeners for different tls certificates for different hostnames.
  • Cannot bind load balancer to existing reserved public IP address

The easiest method of handling this would be to have an option for the cluster annotation to attach to a load balancer OCID, create a backend set, and manage ONLY the backend set. Allow terraform to create and manage the rest of the load balancer. That way as nodes come and go, they can modify the backend set with their IP and ingress port, but not modify the rest of the load balancer settings which are managed by terraform.

Anything else we need to know?

Reference Oracle SR: 3-31268985421

@Nctllnty
Copy link

these problems still exist.

@emoracle
Copy link

with the current tls v1.3 implementation of the LB's this would have been handy

@yuriolisa
Copy link

Can I get this one?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants