Repository navigation
Conversation
|
Known gaps, not addressed here:
|
|
Thanks for explicitly documenting the ClusterIP test-harness limitation above. Our review of commit38d0f3191403d3f6ceb280abc09daae4ced0ca7f confirms the distinction: TESTING=True falls through to the LoadBalancer-address lookup, while normal in-cluster operation uses discovery DNS. We therefore regard this as a known test-coverage limitation, not a production defect. Would you prefer to track support for ClusterIP connectivity in the external e2e harness as a separate follow-up? Our planned EKS verification will run the operator in-cluster and cover ClusterIP connectivity and suspend/resume; those live checks have not run yet. |
|
Yes, a separate follow-up. Production connections don't depend on the exposure: |
Without Traefik, every cluster requested a cloud load balancer, even when only in-cluster clients connect. Resume now waits for a load balancer address only with exposure=loadbalancer. Closes #918 Assisted-by: Claude Code (LLM-assisted)
Suspend deletes the data service so a suspended cluster does not pay for a load balancer. A clusterip service has no such cost, and recreating it on resume gives it a new cluster IP. Assisted-by: Claude Code (LLM-assisted)
The operator reaches the cluster through the headless discovery service. A clusterip cluster has no external address, so routing get_host through the data service would break its health check. Assisted-by: Claude Code (LLM-assisted)
38d0f31 to
d073943
Compare
Summary of changes
In-cluster clients don't need a cloud load balancer, but
loadbalancer(the default) always requests one, andtraefikneedsexternalDNSplus Traefik CRDs.clusteripserves the data Service as a plain ClusterIP. Exposure switches reuse the in-place Service patch, so the #851 NLB cleanup still applies. Resume only waits for an LB address withloadbalancer, otherwise aclusteripcluster would never resume. Suspend keeps aclusteripService: it costs nothing, and recreating it would change the ClusterIP that in-cluster clients may have cached.Checklist
CHANGES.rst