Merge pull request #7397 from killianmuldoon/fix/deprecation-notice

🌱 Update Deprecated comments to correctly trigger staticcheck linter

Author: k8s-ci-robot

.golangci.yml

@@ -185,6 +185,10 @@ issues:
   # changes in PRs and avoid nitpicking.
   exclude-use-default: false
   exclude-rules:
+  # Specific exclude rules for deprecated fields that are still part of the codebase. These should be removed as the referenced deprecated item is removed from the project.
+  - linters:
+      - staticcheck
+    text: "SA1019: (bootstrapv1.ClusterStatus|clusterv1.ClusterTopologyManagedFieldsAnnotation|scope.Config.Spec.UseExperimentalRetryJoin|DockerMachine.Spec.Bootstrapped|machineStatus.Bootstrapped) is deprecated"
   - linters:
     - revive
     text: "exported: exported method .*\\.(Reconcile|SetupWithManager|SetupWebhookWithManager) should have comment or be unexported"

api/v1alpha4/common_types.go

@@ -27,6 +27,7 @@ const (
 	ClusterLabelName = "cluster.x-k8s.io/cluster-name"
 
 	// ClusterTopologyLabelName is the label set on all the object which are managed as part of a ClusterTopology.
+	//
 	// Deprecated: use ClusterTopologyOwnedLabel instead.
 	ClusterTopologyLabelName = "cluster.x-k8s.io/topology"
 
@@ -117,11 +118,13 @@ var (
 
 const (
 	// MachineNodeNameIndex is used by the Machine Controller to index Machines by Node name, and add a watch on Nodes.
+	//
 	// Deprecated: Use api/v1alpha4/index.MachineNodeNameField instead.
 	MachineNodeNameIndex = "status.nodeRef.name"
 
 	// MachineProviderIDIndex is used to index Machines by ProviderID. It's useful to find Machines
 	// in a management cluster from Nodes in a workload cluster.
+	//
 	// Deprecated: Use api/v1alpha4/index.MachineProviderIDField instead.
 	MachineProviderIDIndex = "spec.providerID"
 )

api/v1beta1/common_types.go

@@ -40,7 +40,9 @@ const (
 	// to easily discover which fields have been set by templates + patches/variables at a given reconcile;
 	// instead, it is not necessary to store managed paths for typed objets (e.g. Cluster, MachineDeployments)
 	// given that the topology controller explicitly sets a well-known, immutable list of fields at every reconcile.
+	//
 	// Deprecated: Topology controller is now using server side apply and this annotation will be removed in a future release.
+	// When removing also remove from staticcheck exclude-rules for SA1019 in golangci.yml.
 	ClusterTopologyManagedFieldsAnnotation = "topology.cluster.x-k8s.io/managed-field-paths"
 
 	// ClusterTopologyMachineDeploymentLabelName is the label set on the generated  MachineDeployment objects

api/v1beta1/condition_consts.go

@@ -138,6 +138,7 @@ const (
 
 	// MachineHealthCheckSuccededCondition is set on machines that have passed a healthcheck by the MachineHealthCheck controller.
 	// In the event that the health check fails it will be set to False.
+	//
 	// Deprecated: This const is going to be removed in a next release. Use MachineHealthCheckSucceededCondition instead.
 	MachineHealthCheckSuccededCondition ConditionType = "HealthCheckSucceeded"
 

bootstrap/kubeadm/api/v1alpha4/kubeadm_types.go

@@ -180,6 +180,7 @@ type ImageMeta struct {
 
 // ClusterStatus contains the cluster status. The ClusterStatus will be stored in the kubeadm-config
 // ConfigMap in the cluster, and then updated by kubeadm when additional control plane instance joins or leaves the cluster.
+//
 // Deprecated: ClusterStatus has been removed from kubeadm v1beta3 API; This type is preserved only to support
 // conversion to older versions of the kubeadm API.
 type ClusterStatus struct {

bootstrap/kubeadm/api/v1beta1/kubeadm_types.go

@@ -192,6 +192,7 @@ type ImageMeta struct {
 
 // ClusterStatus contains the cluster status. The ClusterStatus will be stored in the kubeadm-config
 // ConfigMap in the cluster, and then updated by kubeadm when additional control plane instance joins or leaves the cluster.
+//
 // Deprecated: ClusterStatus has been removed from kubeadm v1beta3 API; This type is preserved only to support
 // conversion to older versions of the kubeadm API.
 type ClusterStatus struct {

bootstrap/kubeadm/api/v1beta1/kubeadmconfig_types.go

@@ -97,7 +97,9 @@ type KubeadmConfigSpec struct {
 	//
 	// For more information, refer to https://github.com/kubernetes-sigs/cluster-api/pull/2763#discussion_r397306055.
 	// +optional
+	//
 	// Deprecated: This experimental fix is no longer needed and this field will be removed in a future release.
+	// When removing also remove from staticcheck exclude-rules for SA1019 in golangci.yml
 	UseExperimentalRetryJoin bool `json:"useExperimentalRetryJoin,omitempty"`
 
 	// Ignition contains Ignition specific configuration.

bootstrap/kubeadm/config/crd/bases/bootstrap.cluster.x-k8s.io_kubeadmconfigs.yaml

@@ -49,6 +49,7 @@ type Provider struct {
 
 	// WatchedNamespace indicates the namespace where the provider controller is watching.
 	// If empty the provider controller is watching for objects in all namespaces.
+	//
 	// Deprecated: in clusterctl v1alpha4 all the providers watch all the namespaces; this field will be removed in a future version of this API
 	// +optional
 	WatchedNamespace string `json:"watchedNamespace,omitempty"`