[bridgeconfig] AI suggested fixes.

store Advertised values to the copy and not to the original object.
isIPv6 should be true in ipv6 case.

Signed-off-by: Nadia Pinaeva <[email protected]>

Author: npinaeva

go-controller/pkg/node/bridgeconfig/bridgeconfig.go

@@ -44,7 +44,7 @@ func (netConfig *BridgeUDNConfiguration) ShallowCopy() *BridgeUDNConfiguration {
 		Subnets:     netConfig.Subnets,
 		NodeSubnets: netConfig.NodeSubnets,
 	}
-	netConfig.Advertised.Store(netConfig.Advertised.Load())
+	copy.Advertised.Store(netConfig.Advertised.Load())
 	return copy
 }
 

go-controller/pkg/node/bridgeconfig/bridgeconfig_testutil.go

@@ -80,7 +80,7 @@ func CheckAdvertisedUDNSvcIsolationOVSFlows(flows []string, netConfig *BridgeUDN
 		Expect(err).ToNot(HaveOccurred())
 		protoPrefix = "ip"
 	} else {
-		matchingIPFamilySubnet, err = util.MatchFirstIPNetFamily(false, udnAdvertisedSubnets)
+		matchingIPFamilySubnet, err = util.MatchFirstIPNetFamily(true, udnAdvertisedSubnets)
 		Expect(err).ToNot(HaveOccurred())
 		protoPrefix = "ip6"
 	}