Fixed metallb#2173

internal/config/validator.go

@@ -66,25 +66,25 @@ func (v *validator) Validate(resources ...client.ObjectList) error {
 	return err
 }
 
-func resetfields(clusterResources ClusterResources) ClusterResources{
-	for i := range (clusterResources.Peers) {
+func resetfields(clusterResources ClusterResources) ClusterResources {
+	for i := range clusterResources.Peers {
 		clusterResources.Peers[i].Spec.BFDProfile = ""
 		clusterResources.Peers[i].Spec.PasswordSecret.Name = ""
 	}
-	for _, bgpAdv := range(clusterResources.BGPAdvs){
-		for _, community := range(bgpAdv.Spec.Communities){
-			if !ExistCommunity(community, clusterResources.Communities){
+	for _, bgpAdv := range clusterResources.BGPAdvs {
+		for _, community := range bgpAdv.Spec.Communities {
+			if !ExistCommunity(community, clusterResources.Communities) {
 				bgpAdv.Spec.Communities = RemoveElement(bgpAdv.Spec.Communities, community)
 			}
 		}
 	}
 	for _, legacyAddrPool := range clusterResources.LegacyAddressPools {
 		for _, bgpAdv := range legacyAddrPool.Spec.BGPAdvertisements {
-			for k := range(bgpAdv.Communities){
+			for k := range bgpAdv.Communities {
 				if !ExistCommunity(bgpAdv.Communities[k], clusterResources.Communities) {
 					bgpAdv.Communities = RemoveElement(bgpAdv.Communities, bgpAdv.Communities[k])
 				}
-			}	
+			}
 		}
 	}
 	return clusterResources
@@ -94,9 +94,9 @@ func NewValidator(validate Validate) apivalidate.ClusterObjects {
 	return &validator{validate: validate}
 }
 
-func ExistCommunity(community string, communities []metallbv1beta1.Community ) bool {
+func ExistCommunity(community string, communities []metallbv1beta1.Community) bool {
 	for i := range communities {
-		for j := range communities[i].Spec.Communities{
+		for j := range communities[i].Spec.Communities {
 			if community == communities[i].Spec.Communities[j].Name {
 				return true
 			}
@@ -106,21 +106,21 @@ func ExistCommunity(community string, communities []metallbv1beta1.Community ) b
 }
 
 func RemoveElement(slice []string, elementToRemove string) []string {
-    // Find the index of the element
-    index := -1
-    for i, v := range slice {
-        if v == elementToRemove {
-            index = i
-            break
-        }
-    }
+	// Find the index of the element
+	index := -1
+	for i, v := range slice {
+		if v == elementToRemove {
+			index = i
+			break
+		}
+	}
 
-    // If the element is found, remove it
-    if index != -1 {
-        // Create a new slice without the element
-        return append(slice[:index], slice[index+1:]...)
-    }
+	// If the element is found, remove it
+	if index != -1 {
+		// Create a new slice without the element
+		return append(slice[:index], slice[index+1:]...)
+	}
 
-    // If the element is not found, return the original slice
-    return slice
+	// If the element is not found, return the original slice
+	return slice
 }