Replace deprecated string set through generic set

sbueringer · sbueringer · commit 00eefa3abec3 · 2023-01-26T15:27:47.000+01:00
Signed-off-by: Stefan Büringer buringerst@vmware.com
diff --git a/.golangci.yml b/.golangci.yml
@@ -268,10 +268,6 @@ issues:
     - staticcheck
     text: "SA1019: in.(.+) is deprecated"
     path: .*(api|types)\/.*\/conversion.*\.go$
-  - linters:
-    - staticcheck
-    text: "SA1019: sets.String is deprecated"
-    path: .*(cmd|controllers|controlplane|internal)\/.*\.go$
   - linters:
       - revive
     # Checking if an error is nil to just after return the error or nil is redundant
diff --git a/cmd/clusterctl/client/cluster/components.go b/cmd/clusterctl/client/cluster/components.go
@@ -146,7 +146,7 @@ func (p *providerComponents) Delete(options DeleteOptions) error {
 
 	// Filter the resources according to the delete options
 	resourcesToDelete := []unstructured.Unstructured{}
-	namespacesToDelete := sets.NewString()
+	namespacesToDelete := sets.Set[string]{}
 	instanceNamespacePrefix := fmt.Sprintf("%s-", options.Provider.Namespace)
 	for _, obj := range resources {
 		// If the CRDs should NOT be deleted, skip it;
diff --git a/cmd/clusterctl/client/cluster/crd_migration.go b/cmd/clusterctl/client/cluster/crd_migration.go
@@ -77,7 +77,7 @@ func (m *crdMigrator) run(ctx context.Context, newCRD *apiextensionsv1.CustomRes
 	log := logf.Log
 
 	// Gets the list of version supported by the new CRD
-	newVersions := sets.NewString()
+	newVersions := sets.Set[string]{}
 	for _, version := range newCRD.Spec.Versions {
 		newVersions.Insert(version.Name)
 	}
@@ -105,11 +105,11 @@ func (m *crdMigrator) run(ctx context.Context, newCRD *apiextensionsv1.CustomRes
 		return false, errors.Errorf("unable to upgrade CRD %q because the new CRD does not contain the storage version %q of the current CRD, thus not allowing CR migration", newCRD.Name, currentStorageVersion)
 	}
 
-	currentStatusStoredVersions := sets.NewString(currentCRD.Status.StoredVersions...)
+	currentStatusStoredVersions := sets.Set[string]{}.Insert(currentCRD.Status.StoredVersions...)
 
 	// If the new CRD still contains all current stored versions, nothing to do
 	// as no previous storage version will be dropped.
-	if newVersions.HasAll(currentStatusStoredVersions.List()...) {
+	if newVersions.HasAll(currentStatusStoredVersions.UnsortedList()...) {
 		log.V(2).Info("CRD migration check passed", "name", newCRD.Name)
 		return false, nil
 	}
@@ -123,7 +123,7 @@ func (m *crdMigrator) run(ctx context.Context, newCRD *apiextensionsv1.CustomRes
 	// exposed by the apiserver.
 	storedVersionsToDelete := currentStatusStoredVersions.Difference(newVersions)
 	storedVersionsToPreserve := currentStatusStoredVersions.Intersection(newVersions)
-	log.Info("CR migration required", "kind", newCRD.Spec.Names.Kind, "storedVersionsToDelete", strings.Join(storedVersionsToDelete.List(), ","), "storedVersionsToPreserve", strings.Join(storedVersionsToPreserve.List(), ","))
+	log.Info("CR migration required", "kind", newCRD.Spec.Names.Kind, "storedVersionsToDelete", strings.Join(sets.List(storedVersionsToDelete), ","), "storedVersionsToPreserve", strings.Join(sets.List(storedVersionsToPreserve), ","))
 
 	if err := m.migrateResourcesForCRD(ctx, currentCRD, currentStorageVersion); err != nil {
 		return false, err
diff --git a/cmd/clusterctl/client/cluster/installer.go b/cmd/clusterctl/client/cluster/installer.go
@@ -350,11 +350,11 @@ func simulateInstall(providerList *clusterctlv1.ProviderList, components reposit
 }
 
 func (i *providerInstaller) Images() []string {
-	ret := sets.NewString()
+	ret := sets.Set[string]{}
 	for _, components := range i.installQueue {
 		ret = ret.Insert(components.Images()...)
 	}
-	return ret.List()
+	return sets.List(ret)
 }
 
 func newProviderInstaller(configClient config.Client, repositoryClientFactory RepositoryClientFactory, proxy Proxy, providerMetadata InventoryClient, providerComponents ComponentsClient) *providerInstaller {
diff --git a/cmd/clusterctl/client/cluster/inventory.go b/cmd/clusterctl/client/cluster/inventory.go
@@ -338,14 +338,14 @@ func (p *inventoryClient) GetDefaultProviderName(providerType clusterctlv1.Provi
 	}
 
 	// Group the providers by name, because we consider more instance of the same provider not relevant for the answer.
-	names := sets.NewString()
+	names := sets.Set[string]{}
 	for _, p := range providerList.FilterByType(providerType) {
 		names.Insert(p.ProviderName)
 	}
 
 	// If there is only one provider, this is the default
 	if names.Len() == 1 {
-		return names.List()[0], nil
+		return sets.List(names)[0], nil
 	}
 
 	// There is no provider or more than one provider of this type; in both cases, a default provider name cannot be decided.
@@ -359,13 +359,13 @@ func (p *inventoryClient) GetProviderVersion(provider string, providerType clust
 	}
 
 	// Group the provider instances by version.
-	versions := sets.NewString()
+	versions := sets.Set[string]{}
 	for _, p := range providerList.FilterByProviderNameAndType(provider, providerType) {
 		versions.Insert(p.Version)
 	}
 
 	if versions.Len() == 1 {
-		return versions.List()[0], nil
+		return sets.List(versions)[0], nil
 	}
 
 	// The default version for this provider cannot be decided.
@@ -379,13 +379,13 @@ func (p *inventoryClient) GetProviderNamespace(provider string, providerType clu
 	}
 
 	// Group the providers by namespace
-	namespaces := sets.NewString()
+	namespaces := sets.Set[string]{}
 	for _, p := range providerList.FilterByProviderNameAndType(provider, providerType) {
 		namespaces.Insert(p.Namespace)
 	}
 
 	if namespaces.Len() == 1 {
-		return namespaces.List()[0], nil
+		return sets.List(namespaces)[0], nil
 	}
 
 	// The default provider namespace cannot be decided.
diff --git a/cmd/clusterctl/client/cluster/mover.go b/cmd/clusterctl/client/cluster/mover.go
@@ -669,7 +669,7 @@ func (o *objectMover) ensureNamespaces(graph *objectGraph, toProxy Proxy) error
 	}
 
 	ensureNamespaceBackoff := newWriteBackoff()
-	namespaces := sets.NewString()
+	namespaces := sets.Set[string]{}
 	for _, node := range graph.getMoveNodes() {
 		// ignore global/cluster-wide objects
 		if node.isGlobal {
diff --git a/cmd/clusterctl/client/cluster/objectgraph_test.go b/cmd/clusterctl/client/cluster/objectgraph_test.go
@@ -2337,7 +2337,7 @@ func Test_objectGraph_setGlobalIdentityTenants(t *testing.T) {
 
 func deduplicateObjects(objs []client.Object) []client.Object {
 	res := []client.Object{}
-	uniqueObjectKeys := sets.NewString()
+	uniqueObjectKeys := sets.Set[string]{}
 	for _, o := range objs {
 		if !uniqueObjectKeys.Has(string(o.GetUID())) {
 			res = append(res, o)
diff --git a/cmd/clusterctl/client/cluster/proxy.go b/cmd/clusterctl/client/cluster/proxy.go
@@ -232,7 +232,7 @@ func (k *proxy) ListResources(labels map[string]string, namespaces ...string) ([
 
 	// Exclude from discovery the objects from the cert-manager/provider's CRDs.
 	// Those objects are not part of the components, and they will eventually be removed when removing the CRD definition.
-	crdsToExclude := sets.String{}
+	crdsToExclude := sets.Set[string]{}
 
 	crdList := &apiextensionsv1.CustomResourceDefinitionList{}
 	if err := retryWithExponentialBackoff(newReadBackoff(), func() error {
diff --git a/cmd/clusterctl/client/cluster/topology.go b/cmd/clusterctl/client/cluster/topology.go
@@ -692,7 +692,7 @@ func clusterClassUsesTemplate(cc *clusterv1.ClusterClass, templateRef *corev1.Ob
 }
 
 func uniqueNamespaces(objs []*unstructured.Unstructured) []string {
-	ns := sets.NewString()
+	ns := sets.Set[string]{}
 	for _, obj := range objs {
 		// Namespace objects do not have metadata.namespace set, but we can add the
 		// name of the obj to the namespace list, as it is another unique namespace.
@@ -707,7 +707,7 @@ func uniqueNamespaces(objs []*unstructured.Unstructured) []string {
 		// objects from different namespaces.
 		ns.Insert(obj.GetNamespace())
 	}
-	return ns.List()
+	return sets.List(ns)
 }
 
 func hasUniqueVersionPerGroupKind(objs []*unstructured.Unstructured) bool {
diff --git a/cmd/clusterctl/client/cluster/upgrader.go b/cmd/clusterctl/client/cluster/upgrader.go
@@ -257,7 +257,7 @@ func (u *providerUpgrader) createCustomPlan(upgradeItems []UpgradeItem) (*Upgrad
 	}
 
 	// Builds the custom upgrade plan, by adding all the upgrade items after checking consistency with the targetContract.
-	upgradeInstanceNames := sets.NewString()
+	upgradeInstanceNames := sets.Set[string]{}
 	upgradePlan := &UpgradePlan{
 		Contract: targetContract,
 	}
diff --git a/cmd/clusterctl/client/cluster/upgrader_info.go b/cmd/clusterctl/client/cluster/upgrader_info.go
@@ -149,7 +149,7 @@ func newUpgradeInfo(metadata *clusterctlv1.Metadata, currentVersion *version.Ver
 
 // getContractsForUpgrade return the list of API Version of Cluster API (contract) version available for a provider upgrade.
 func (i *upgradeInfo) getContractsForUpgrade() []string {
-	contractsForUpgrade := sets.NewString()
+	contractsForUpgrade := sets.Set[string]{}
 	for _, releaseSeries := range i.metadata.ReleaseSeries {
 		// Drop the release series if older than the current version, because not relevant for upgrade.
 		if i.currentVersion.Major() > releaseSeries.Major || (i.currentVersion.Major() == releaseSeries.Major && i.currentVersion.Minor() > releaseSeries.Minor) {
@@ -158,7 +158,7 @@ func (i *upgradeInfo) getContractsForUpgrade() []string {
 		contractsForUpgrade.Insert(releaseSeries.Contract)
 	}
 
-	return contractsForUpgrade.List()
+	return sets.List(contractsForUpgrade)
 }
 
 // getLatestNextVersion returns the next available version for a provider within the target API Version of Cluster API (contract).
diff --git a/cmd/clusterctl/client/delete_test.go b/cmd/clusterctl/client/delete_test.go
@@ -42,7 +42,7 @@ func Test_clusterctlClient_Delete(t *testing.T) {
 		name          string
 		fields        fields
 		args          args
-		wantProviders sets.String
+		wantProviders sets.Set[string]
 		wantErr       bool
 	}{
 		{
@@ -63,7 +63,7 @@ func Test_clusterctlClient_Delete(t *testing.T) {
 					DeleteAll:               true, // delete all the providers
 				},
 			},
-			wantProviders: sets.NewString(),
+			wantProviders: sets.Set[string]{},
 			wantErr:       false,
 		},
 		{
@@ -84,7 +84,7 @@ func Test_clusterctlClient_Delete(t *testing.T) {
 					DeleteAll:               false,
 				},
 			},
-			wantProviders: sets.NewString(
+			wantProviders: sets.Set[string]{}.Insert(
 				capiProviderConfig.Name(),
 				clusterctlv1.ManifestLabel(controlPlaneProviderConfig.Name(), controlPlaneProviderConfig.Type()),
 				clusterctlv1.ManifestLabel(infraProviderConfig.Name(), infraProviderConfig.Type())),
@@ -108,7 +108,7 @@ func Test_clusterctlClient_Delete(t *testing.T) {
 					DeleteAll:               false,
 				},
 			},
-			wantProviders: sets.NewString(
+			wantProviders: sets.Set[string]{}.Insert(
 				capiProviderConfig.Name(),
 				clusterctlv1.ManifestLabel(bootstrapProviderConfig.Name(), bootstrapProviderConfig.Type()),
 				clusterctlv1.ManifestLabel(controlPlaneProviderConfig.Name(), controlPlaneProviderConfig.Type())),
@@ -132,7 +132,7 @@ func Test_clusterctlClient_Delete(t *testing.T) {
 					DeleteAll:               false,
 				},
 			},
-			wantProviders: sets.NewString(
+			wantProviders: sets.Set[string]{}.Insert(
 				capiProviderConfig.Name(),
 				clusterctlv1.ManifestLabel(bootstrapProviderConfig.Name(), bootstrapProviderConfig.Type()),
 				clusterctlv1.ManifestLabel(controlPlaneProviderConfig.Name(), controlPlaneProviderConfig.Type()),
@@ -157,7 +157,7 @@ func Test_clusterctlClient_Delete(t *testing.T) {
 					DeleteAll:               false,
 				},
 			},
-			wantProviders: sets.NewString(
+			wantProviders: sets.Set[string]{}.Insert(
 				clusterctlv1.ManifestLabel(controlPlaneProviderConfig.Name(), controlPlaneProviderConfig.Type()),
 				clusterctlv1.ManifestLabel(infraProviderConfig.Name(), infraProviderConfig.Type())),
 			wantErr: false,
@@ -182,7 +182,7 @@ func Test_clusterctlClient_Delete(t *testing.T) {
 			g.Expect(err).NotTo(HaveOccurred())
 			g.Expect(c.List(ctx, gotProviders)).To(Succeed())
 
-			gotProvidersSet := sets.NewString()
+			gotProvidersSet := sets.Set[string]{}
 			for _, gotProvider := range gotProviders.Items {
 				gotProvidersSet.Insert(gotProvider.Name)
 			}
diff --git a/cmd/clusterctl/client/repository/template.go b/cmd/clusterctl/client/repository/template.go
@@ -164,7 +164,7 @@ func MergeTemplates(templates ...Template) (Template, error) {
 	}
 
 	for _, tmpl := range templates {
-		merged.variables = sets.NewString(merged.variables...).Union(sets.NewString(tmpl.Variables()...)).List()
+		merged.variables = sets.List(sets.Set[string]{}.Insert(merged.variables...).Union(sets.Set[string]{}.Insert(tmpl.Variables()...)))
 
 		for key, val := range tmpl.VariableMap() {
 			if v, ok := merged.variableMap[key]; !ok || v == nil {
diff --git a/controllers/remote/cluster_cache_tracker.go b/controllers/remote/cluster_cache_tracker.go
@@ -169,7 +169,7 @@ func (t *ClusterCacheTracker) GetRESTConfig(ctc context.Context, cluster client.
 type clusterAccessor struct {
 	cache   *stoppableCache
 	client  client.Client
-	watches sets.String
+	watches sets.Set[string]
 	config  *rest.Config
 }
 
@@ -338,7 +338,7 @@ func (t *ClusterCacheTracker) newClusterAccessor(ctx context.Context, cluster cl
 		cache:   cache,
 		config:  config,
 		client:  delegatingClient,
-		watches: sets.NewString(),
+		watches: sets.Set[string]{},
 	}, nil
 }
 
diff --git a/controllers/remote/cluster_cache_tracker_fake.go b/controllers/remote/cluster_cache_tracker_fake.go
@@ -41,10 +41,9 @@ func NewTestClusterCacheTracker(log logr.Logger, cl client.Client, scheme *runti
 	}
 
 	testCacheTracker.clusterAccessors[objKey] = &clusterAccessor{
-
 		cache:   nil,
 		client:  delegatingClient,
-		watches: sets.NewString(watchObjects...),
+		watches: sets.Set[string]{}.Insert(watchObjects...),
 	}
 	return testCacheTracker
 }
diff --git a/controlplane/kubeadm/internal/etcd/util/util.go b/controlplane/kubeadm/internal/etcd/util/util.go
@@ -47,7 +47,7 @@ func MemberNames(members []*etcd.Member) []string {
 // This function only checks that set of names of each member
 // within the lists is the same.
 func MemberEqual(members1, members2 []*etcd.Member) bool {
-	names1 := sets.NewString(MemberNames(members1)...)
-	names2 := sets.NewString(MemberNames(members2)...)
+	names1 := sets.Set[string]{}.Insert(MemberNames(members1)...)
+	names2 := sets.Set[string]{}.Insert(MemberNames(members2)...)
 	return names1.Equal(names2)
 }
diff --git a/controlplane/kubeadm/internal/etcd_client_generator.go b/controlplane/kubeadm/internal/etcd_client_generator.go
@@ -93,7 +93,7 @@ func (c *EtcdClientGenerator) forLeader(ctx context.Context, nodeNames []string)
 		return nil, errors.New("invalid argument: forLeader can't be called with an empty list of nodes")
 	}
 
-	nodes := sets.NewString()
+	nodes := sets.Set[string]{}
 	for _, n := range nodeNames {
 		nodes.Insert(n)
 	}
@@ -118,7 +118,7 @@ func (c *EtcdClientGenerator) forLeader(ctx context.Context, nodeNames []string)
 // getLeaderClient provides an etcd client connected to the leader. It returns an
 // errEtcdNodeConnection if there was a connection problem with the given etcd
 // node, which should be considered non-fatal by the caller.
-func (c *EtcdClientGenerator) getLeaderClient(ctx context.Context, nodeName string, allNodes sets.String) (*etcd.Client, error) {
+func (c *EtcdClientGenerator) getLeaderClient(ctx context.Context, nodeName string, allNodes sets.Set[string]) (*etcd.Client, error) {
 	// Get a temporary client to the etcd instance hosted on the node.
 	client, err := c.forFirstAvailableNode(ctx, []string{nodeName})
 	if err != nil {
diff --git a/controlplane/kubeadm/internal/workload_cluster.go b/controlplane/kubeadm/internal/workload_cluster.go
@@ -137,7 +137,7 @@ var _ WorkloadCluster = &Workload{}
 
 func (w *Workload) getControlPlaneNodes(ctx context.Context) (*corev1.NodeList, error) {
 	controlPlaneNodes := &corev1.NodeList{}
-	controlPlaneNodeNames := sets.NewString()
+	controlPlaneNodeNames := sets.Set[string]{}
 
 	for _, label := range []string{labelNodeRoleOldControlPlane, labelNodeRoleControlPlane} {
 		nodes := &corev1.NodeList{}
diff --git a/controlplane/kubeadm/internal/workload_cluster_conditions.go b/controlplane/kubeadm/internal/workload_cluster_conditions.go
@@ -514,11 +514,11 @@ type aggregateFromMachinesToKCPInput struct {
 func aggregateFromMachinesToKCP(input aggregateFromMachinesToKCPInput) {
 	// Aggregates machines for condition status.
 	// NB. A machine could be assigned to many groups, but only the group with the highest severity will be reported.
-	kcpMachinesWithErrors := sets.NewString()
-	kcpMachinesWithWarnings := sets.NewString()
-	kcpMachinesWithInfo := sets.NewString()
-	kcpMachinesWithTrue := sets.NewString()
-	kcpMachinesWithUnknown := sets.NewString()
+	kcpMachinesWithErrors := sets.Set[string]{}
+	kcpMachinesWithWarnings := sets.Set[string]{}
+	kcpMachinesWithInfo := sets.Set[string]{}
+	kcpMachinesWithTrue := sets.Set[string]{}
+	kcpMachinesWithUnknown := sets.Set[string]{}
 
 	for i := range input.controlPlane.Machines {
 		machine := input.controlPlane.Machines[i]
@@ -545,7 +545,7 @@ func aggregateFromMachinesToKCP(input aggregateFromMachinesToKCPInput) {
 
 	// In case of at least one machine with errors or KCP level errors (nodes without machines), report false, error.
 	if len(kcpMachinesWithErrors) > 0 {
-		input.kcpErrors = append(input.kcpErrors, fmt.Sprintf("Following machines are reporting %s errors: %s", input.note, strings.Join(kcpMachinesWithErrors.List(), ", ")))
+		input.kcpErrors = append(input.kcpErrors, fmt.Sprintf("Following machines are reporting %s errors: %s", input.note, strings.Join(sets.List(kcpMachinesWithErrors), ", ")))
 	}
 	if len(input.kcpErrors) > 0 {
 		conditions.MarkFalse(input.controlPlane.KCP, input.condition, input.unhealthyReason, clusterv1.ConditionSeverityError, strings.Join(input.kcpErrors, "; "))
@@ -554,13 +554,13 @@ func aggregateFromMachinesToKCP(input aggregateFromMachinesToKCPInput) {
 
 	// In case of no errors and at least one machine with warnings, report false, warnings.
 	if len(kcpMachinesWithWarnings) > 0 {
-		conditions.MarkFalse(input.controlPlane.KCP, input.condition, input.unhealthyReason, clusterv1.ConditionSeverityWarning, "Following machines are reporting %s warnings: %s", input.note, strings.Join(kcpMachinesWithWarnings.List(), ", "))
+		conditions.MarkFalse(input.controlPlane.KCP, input.condition, input.unhealthyReason, clusterv1.ConditionSeverityWarning, "Following machines are reporting %s warnings: %s", input.note, strings.Join(sets.List(kcpMachinesWithWarnings), ", "))
 		return
 	}
 
 	// In case of no errors, no warning, and at least one machine with info, report false, info.
 	if len(kcpMachinesWithWarnings) > 0 {
-		conditions.MarkFalse(input.controlPlane.KCP, input.condition, input.unhealthyReason, clusterv1.ConditionSeverityWarning, "Following machines are reporting %s info: %s", input.note, strings.Join(kcpMachinesWithInfo.List(), ", "))
+		conditions.MarkFalse(input.controlPlane.KCP, input.condition, input.unhealthyReason, clusterv1.ConditionSeverityWarning, "Following machines are reporting %s info: %s", input.note, strings.Join(sets.List(kcpMachinesWithInfo), ", "))
 		return
 	}
 
@@ -572,7 +572,7 @@ func aggregateFromMachinesToKCP(input aggregateFromMachinesToKCPInput) {
 
 	// Otherwise, if there is at least one machine with unknown, report unknown.
 	if len(kcpMachinesWithUnknown) > 0 {
-		conditions.MarkUnknown(input.controlPlane.KCP, input.condition, input.unknownReason, "Following machines are reporting unknown %s status: %s", input.note, strings.Join(kcpMachinesWithUnknown.List(), ", "))
+		conditions.MarkUnknown(input.controlPlane.KCP, input.condition, input.unknownReason, "Following machines are reporting unknown %s status: %s", input.note, strings.Join(sets.List(kcpMachinesWithUnknown), ", "))
 		return
 	}
 
diff --git a/hack/tools/tilt-prepare/main.go b/hack/tools/tilt-prepare/main.go
@@ -244,7 +244,7 @@ func allowK8sConfig(ts *tiltSettings) error {
 		return nil
 	}
 
-	allowed := sets.NewString(ts.AllowedContexts...)
+	allowed := sets.Set[string]{}.Insert(ts.AllowedContexts...)
 	if !allowed.Has(config.CurrentContext) {
 		return errors.Errorf("context %s from the KubeConfig file is not allowed", config.CurrentContext)
 	}
diff --git a/internal/controllers/clusterclass/clusterclass_controller.go b/internal/controllers/clusterclass/clusterclass_controller.go
@@ -138,7 +138,7 @@ func (r *Reconciler) reconcile(ctx context.Context, clusterClass *clusterv1.Clus
 	// For example the same KubeadmConfigTemplate could be referenced in multiple MachineDeployment
 	// classes.
 	errs := []error{}
-	reconciledRefs := sets.NewString()
+	reconciledRefs := sets.Set[string]{}
 	outdatedRefs := map[*corev1.ObjectReference]*corev1.ObjectReference{}
 	for i := range refs {
 		ref := refs[i]
diff --git a/internal/controllers/machinedeployment/machinedeployment_sync.go b/internal/controllers/machinedeployment/machinedeployment_sync.go
diff --git a/internal/controllers/topology/cluster/scope/upgradetracker.go b/internal/controllers/topology/cluster/scope/upgradetracker.go
diff --git a/internal/hooks/tracking.go b/internal/hooks/tracking.go
diff --git a/internal/topology/check/compatibility.go b/internal/topology/check/compatibility.go
diff --git a/internal/topology/variables/clusterclass_variable_validation.go b/internal/topology/variables/clusterclass_variable_validation.go
diff --git a/internal/webhooks/clusterclass.go b/internal/webhooks/clusterclass.go
diff --git a/internal/webhooks/patch_validation.go b/internal/webhooks/patch_validation.go
diff --git a/test/e2e/clusterctl_upgrade.go b/test/e2e/clusterctl_upgrade.go
diff --git a/test/framework/controlplane_helpers.go b/test/framework/controlplane_helpers.go
diff --git a/util/log/log.go b/util/log/log.go

Original file line number	Diff line number	Diff line change
`@@ -350,11 +350,11 @@ func simulateInstall(providerList *clusterctlv1.ProviderList, components reposit`
`350`	`350`	`}`
`351`	`351`
`352`	`352`	`func (i *providerInstaller) Images() []string {`
`353`		`- ret := sets.NewString()`
	`353`	`+ ret := sets.Set[string]{}`
`354`	`354`	`for _, components := range i.installQueue {`
`355`	`355`	`ret = ret.Insert(components.Images()...)`
`356`	`356`	`}`
`357`		`- return ret.List()`
	`357`	`+ return sets.List(ret)`
`358`	`358`	`}`
`359`	`359`
`360`	`360`	`func newProviderInstaller(configClient config.Client, repositoryClientFactory RepositoryClientFactory, proxy Proxy, providerMetadata InventoryClient, providerComponents ComponentsClient) *providerInstaller {`
Original file line number	Diff line number	Diff line change
`@@ -338,14 +338,14 @@ func (p *inventoryClient) GetDefaultProviderName(providerType clusterctlv1.Provi`
`338`	`338`	`}`
`339`	`339`
`340`	`340`	`// Group the providers by name, because we consider more instance of the same provider not relevant for the answer.`
`341`		`- names := sets.NewString()`
	`341`	`+ names := sets.Set[string]{}`
`342`	`342`	`for _, p := range providerList.FilterByType(providerType) {`
`343`	`343`	`names.Insert(p.ProviderName)`
`344`	`344`	`}`
`345`	`345`
`346`	`346`	`// If there is only one provider, this is the default`
`347`	`347`	`if names.Len() == 1 {`
`348`		`- return names.List()[0], nil`
	`348`	`+ return sets.List(names)[0], nil`
`349`	`349`	`}`
`350`	`350`
`351`	`351`	`// There is no provider or more than one provider of this type; in both cases, a default provider name cannot be decided.`
`@@ -359,13 +359,13 @@ func (p *inventoryClient) GetProviderVersion(provider string, providerType clust`
`359`	`359`	`}`
`360`	`360`
`361`	`361`	`// Group the provider instances by version.`
`362`		`- versions := sets.NewString()`
	`362`	`+ versions := sets.Set[string]{}`
`363`	`363`	`for _, p := range providerList.FilterByProviderNameAndType(provider, providerType) {`
`364`	`364`	`versions.Insert(p.Version)`
`365`	`365`	`}`
`366`	`366`
`367`	`367`	`if versions.Len() == 1 {`
`368`		`- return versions.List()[0], nil`
	`368`	`+ return sets.List(versions)[0], nil`
`369`	`369`	`}`
`370`	`370`
`371`	`371`	`// The default version for this provider cannot be decided.`
`@@ -379,13 +379,13 @@ func (p *inventoryClient) GetProviderNamespace(provider string, providerType clu`
`379`	`379`	`}`
`380`	`380`
`381`	`381`	`// Group the providers by namespace`
`382`		`- namespaces := sets.NewString()`
	`382`	`+ namespaces := sets.Set[string]{}`
`383`	`383`	`for _, p := range providerList.FilterByProviderNameAndType(provider, providerType) {`
`384`	`384`	`namespaces.Insert(p.Namespace)`
`385`	`385`	`}`
`386`	`386`
`387`	`387`	`if namespaces.Len() == 1 {`
`388`		`- return namespaces.List()[0], nil`
	`388`	`+ return sets.List(namespaces)[0], nil`
`389`	`389`	`}`
`390`	`390`
`391`	`391`	`// The default provider namespace cannot be decided.`
Original file line number	Diff line number	Diff line change
`@@ -669,7 +669,7 @@ func (o objectMover) ensureNamespaces(graph objectGraph, toProxy Proxy) error`
`669`	`669`	`}`
`670`	`670`
`671`	`671`	`ensureNamespaceBackoff := newWriteBackoff()`
`672`		`- namespaces := sets.NewString()`
	`672`	`+ namespaces := sets.Set[string]{}`
`673`	`673`	`for _, node := range graph.getMoveNodes() {`
`674`	`674`	`// ignore global/cluster-wide objects`
`675`	`675`	`if node.isGlobal {`
Original file line number	Diff line number	Diff line change
`@@ -692,7 +692,7 @@ func clusterClassUsesTemplate(cc clusterv1.ClusterClass, templateRef corev1.Ob`
`692`	`692`	`}`
`693`	`693`
`694`	`694`	`func uniqueNamespaces(objs []*unstructured.Unstructured) []string {`
`695`		`- ns := sets.NewString()`
	`695`	`+ ns := sets.Set[string]{}`
`696`	`696`	`for _, obj := range objs {`
`697`	`697`	`// Namespace objects do not have metadata.namespace set, but we can add the`
`698`	`698`	`// name of the obj to the namespace list, as it is another unique namespace.`
`@@ -707,7 +707,7 @@ func uniqueNamespaces(objs []*unstructured.Unstructured) []string {`
`707`	`707`	`// objects from different namespaces.`
`708`	`708`	`ns.Insert(obj.GetNamespace())`
`709`	`709`	`}`
`710`		`- return ns.List()`
	`710`	`+ return sets.List(ns)`
`711`	`711`	`}`
`712`	`712`
`713`	`713`	`func hasUniqueVersionPerGroupKind(objs []*unstructured.Unstructured) bool {`
Original file line number	Diff line number	Diff line change
`@@ -257,7 +257,7 @@ func (u providerUpgrader) createCustomPlan(upgradeItems []UpgradeItem) (Upgrad`
`257`	`257`	`}`
`258`	`258`
`259`	`259`	`// Builds the custom upgrade plan, by adding all the upgrade items after checking consistency with the targetContract.`
`260`		`- upgradeInstanceNames := sets.NewString()`
	`260`	`+ upgradeInstanceNames := sets.Set[string]{}`
`261`	`261`	`upgradePlan := &UpgradePlan{`
`262`	`262`	`Contract: targetContract,`
`263`	`263`	`}`