fix: Do not return error if serviceLoadBalancer field is not set (#611)

**What problem does this PR solve?**:
I noticed that the error handling was incorrect.

I also noticed that the field tag did not get updated in
#604,
so I made that update here.

Author: dlipovetsky

api/v1alpha1/addon_types.go

@@ -72,7 +72,7 @@ type Addons struct {
 	// +kubebuilder:validation:Optional
 	CSIProviders *CSI `json:"csi,omitempty"`
 
-	// +optional
+	// +kubebuilder:validation:Optional
 	ServiceLoadBalancer *ServiceLoadBalancer `json:"serviceLoadBalancer,omitempty"`
 }
 

pkg/handlers/generic/lifecycle/serviceloadbalancer/handler.go

@@ -69,12 +69,20 @@ func (c *ServiceLoadBalancerHandler) AfterControlPlaneInitialized(
 	)
 
 	varMap := variables.ClusterVariablesToVariablesMap(req.Cluster.Spec.Topology.Variables)
-
 	slb, err := variables.Get[v1alpha1.ServiceLoadBalancer](
 		varMap,
 		c.variableName,
 		c.variablePath...)
 	if err != nil {
+		if variables.IsNotFoundError(err) {
+			log.
+				Info(
+					"Skipping ServiceLoadBalancer, field is not specified",
+					"error",
+					err,
+				)
+			return
+		}
 		log.Error(
 			err,
 			"failed to read ServiceLoadBalancer provider from cluster definition",