Update Helm templates to account for lack of external load balancer support #623
Labels
impact/reliability
Something that feels unreliable or flaky
impact/usability
Something that impacts users' ability to use the product easily and intuitively
kind/enhancement
Improvements or new features
needs-triage
Needs attention from the triage team
size/M
Estimated effort to complete (up to 5 days).
The current Helm templates deploy the Nginx ingress controller and expose it with a Service of type LoadBalancer. This is fine for production-grade environments where the Kubernetes clusters have external load balancer support, but this can cause issues when testing the programs on platforms like KinD or Minikube. These platforms typically do not have external load balancer support.
Without external load balancer support, the Service never comes out of a pending state, and this causes Pulumi to report messages like this:
To fix this, we need to update the Helm templates in three ways:
Defaulting to a value of ClusterIP will help tests pass successfully, but are not conducive to actual production deployments. Defaulting to LoadBalancer is more in line with actual production deployments, but tests will fail if external load balancer support isn't present. The recommendation is to default to ClusterIP with comments in the code on how to change that for production deployments. Changes to the associated landing pages on the website may also be needed.
The text was updated successfully, but these errors were encountered: