fix: Allow control plane endpoint updates with empty location (#255)

**What is the purpose of this pull request/Why do we need it?**

This validation is blocking successful reconciliation of kamaji
resources under certain circumstances.
I should've noticed while testing it the first time, but I think my test
data was limited to a single scenario.

**Issue #, if available:**

**Description of changes:**

Remove validation blocking updates to control plane endpoint when
location is not set.

**Special notes for your reviewer:**

I tested this by updating the CRD manually and seeing reconciliation
succeed.

**Checklist:**
- [ ] Documentation updated
- [ ] Unit Tests added
- [ ] E2E Tests added
- [x] Includes
[emojis](https://github.com/kubernetes-sigs/kubebuilder-release-tools?tab=readme-ov-file#kubebuilder-project-versioning)

api/v1alpha1/ionoscloudcluster_types.go

@@ -34,7 +34,7 @@ const (
 	IonosCloudClusterKind = "IonosCloudCluster"
 )
 
-//+kubebuilder:validation:XValidation:rule="self.controlPlaneEndpoint.host == '' || has(self.location)",message="location is required when controlPlaneEndpoint.host is set"
+//+kubebuilder:validation:XValidation:rule="!has(self.loadBalancerProviderRef) || has(self.location)",message="location is required when loadBalancerProviderRef is set"
 
 // IonosCloudClusterSpec defines the desired state of IonosCloudCluster.
 type IonosCloudClusterSpec struct {

api/v1alpha1/ionoscloudcluster_types_test.go

@@ -86,14 +86,15 @@ var _ = Describe("IonosCloudCluster", func() {
 			Expect(k8sClient.Create(context.Background(), cluster)).
 				Should(MatchError(ContainSubstring("credentialsRef.name must be provided")))
 		})
-		It("should not allow creating clusters with empty location when ControlPlaneEndpoint host is set", func() {
+		It("should not allow creating clusters with empty location when loadBalancerProviderRef is set", func() {
 			cluster := defaultCluster()
 			cluster.Spec.Location = ""
 			Expect(k8sClient.Create(context.Background(), cluster)).
-				Should(MatchError(ContainSubstring("location is required when controlPlaneEndpoint.host is set")))
+				Should(MatchError(ContainSubstring("location is required when loadBalancerProviderRef is set")))
 		})
-		It("should allow creating clusters with empty location when ControlPlaneEndpoint host is not set", func() {
+		It("should allow creating clusters with empty location and ControlPlaneEndpoint host when loadBalancerProviderRef is not set", func() {
 			cluster := defaultCluster()
+			cluster.Spec.LoadBalancerProviderRef = nil
 			cluster.Spec.Location = ""
 			cluster.Spec.ControlPlaneEndpoint.Host = ""
 			Expect(k8sClient.Create(context.Background(), cluster)).To(Succeed())
@@ -126,6 +127,16 @@ var _ = Describe("IonosCloudCluster", func() {
 				cluster.Spec.ControlPlaneEndpoint.Host = "example.org"
 				Expect(k8sClient.Update(context.Background(), cluster)).To(Succeed())
 			})
+			It("should not fail when when location is empty and loadBalancerProviderRef is not set", func() {
+				cluster := defaultCluster()
+				cluster.Spec.LoadBalancerProviderRef = nil
+				cluster.Spec.Location = ""
+				cluster.Spec.ControlPlaneEndpoint.Host = ""
+				Expect(k8sClient.Create(context.Background(), cluster)).To(Succeed())
+
+				cluster.Spec.ControlPlaneEndpoint = defaultCluster().Spec.ControlPlaneEndpoint
+				Expect(k8sClient.Update(context.Background(), cluster)).To(Succeed())
+			})
 		})
 	})
 	Context("Status", func() {

config/crd/bases/infrastructure.cluster.x-k8s.io_ionoscloudclusters.yaml

@@ -125,8 +125,8 @@ spec:
             - credentialsRef
             type: object
             x-kubernetes-validations:
-            - message: location is required when controlPlaneEndpoint.host is set
-              rule: self.controlPlaneEndpoint.host == '' || has(self.location)
+            - message: location is required when loadBalancerProviderRef is set
+              rule: '!has(self.loadBalancerProviderRef) || has(self.location)'
           status:
             description: IonosCloudClusterStatus defines the observed state of IonosCloudCluster.
             properties:

config/crd/bases/infrastructure.cluster.x-k8s.io_ionoscloudclustertemplates.yaml

@@ -117,9 +117,9 @@ spec:
                     - credentialsRef
                     type: object
                     x-kubernetes-validations:
-                    - message: location is required when controlPlaneEndpoint.host
-                        is set
-                      rule: self.controlPlaneEndpoint.host == '' || has(self.location)
+                    - message: location is required when loadBalancerProviderRef is
+                        set
+                      rule: '!has(self.loadBalancerProviderRef) || has(self.location)'
                 required:
                 - spec
                 type: object