WiP: Some small fixes

Author: s3rj1k

pkg/cloud/services/networking/network.go

@@ -25,6 +25,7 @@ import (
 	"github.com/gophercloud/gophercloud/v2/openstack/networking/v2/extensions/external"
 	"github.com/gophercloud/gophercloud/v2/openstack/networking/v2/networks"
 	"github.com/gophercloud/gophercloud/v2/openstack/networking/v2/subnets"
+	"k8s.io/apimachinery/pkg/api/equality"
 	"k8s.io/utils/ptr"
 
 	infrav1 "sigs.k8s.io/cluster-api-provider-openstack/api/v1beta1"
@@ -258,16 +259,11 @@ func (s *Service) updateSubnetDNSNameservers(openStackCluster *infrav1.OpenStack
 	desiredNameservers := openStackCluster.Spec.ManagedSubnets[0].DNSNameservers
 	currentNameservers := subnet.DNSNameservers
 
-	needsUpdate := false
+	var needsUpdate bool
 	if len(desiredNameservers) != len(currentNameservers) {
 		needsUpdate = true
 	} else {
-		for i, ns := range desiredNameservers {
-			if i >= len(currentNameservers) || ns != currentNameservers[i] {
-				needsUpdate = true
-				break
-			}
-		}
+		needsUpdate = !equality.Semantic.DeepEqual(currentNameservers, desiredNameservers)
 	}
 
 	if needsUpdate {

pkg/webhooks/openstackcluster_webhook.go

@@ -21,6 +21,7 @@ import (
 	"fmt"
 	"reflect"
 
+	"k8s.io/apimachinery/pkg/api/equality"
 	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/apimachinery/pkg/util/validation/field"
 	"k8s.io/utils/ptr"
@@ -165,7 +166,7 @@ func (*openStackClusterWebhook) ValidateUpdate(_ context.Context, oldObjRaw, new
 				}
 
 				// Check if AllocationPools have changed
-				if !reflect.DeepEqual(oldSubnet.AllocationPools, newSubnet.AllocationPools) {
+				if !equality.Semantic.DeepEqual(oldSubnet.AllocationPools, newSubnet.AllocationPools) {
 					allErrs = append(allErrs, field.Forbidden(
 						field.NewPath("spec", "managedSubnets").Child("allocationPools"),
 						"cannot modify allocation pools in existing subnet",