fix failing jobs + review comments

apis/v1alpha3/cloudprovider_types.go

@@ -27,7 +27,8 @@ limitations under the License.
 package v1alpha3
 
 // CPIConfig is the vSphere cloud provider's configuration.
-// Depracated: will be removed in v1alpha4.
+//
+// Deprecated: will be removed in v1alpha4.
 type CPIConfig struct {
 	// Global is the vSphere cloud provider's global configuration.
 	// +optional

apis/v1alpha3/vspherecluster_types.go

@@ -22,14 +22,22 @@ import (
 	clusterv1alpha3 "sigs.k8s.io/cluster-api/api/v1alpha3"
 )
 
+const (
+	// ClusterFinalizer allows ReconcileVSphereCluster to clean up vSphere
+	// resources associated with VSphereCluster before removing it from the
+	// API server.
+	ClusterFinalizer = "vspherecluster.infrastructure.cluster.x-k8s.io"
+)
+
 // VSphereClusterSpec defines the desired state of VSphereCluster.
 type VSphereClusterSpec struct {
 	// Server is the address of the vSphere endpoint.
 	Server string `json:"server,omitempty"`
 
 	// Insecure is a flag that controls whether to validate the
 	// vSphere server's certificate.
-	// Depracated: will be removed in v1alpha4
+	//
+	// Deprecated: will be removed in v1alpha4.
 	// +optional
 	Insecure *bool `json:"insecure,omitempty"`
 
@@ -38,9 +46,9 @@ type VSphereClusterSpec struct {
 	// +optional
 	Thumbprint string `json:"thumbprint,omitempty"`
 
-	// CloudProviderConfiguration holds the cluster-wide configuration for the
-	// Depracated: will be removed in v1alpha4
-	// vSphere cloud provider.
+	// CloudProviderConfiguration holds the cluster-wide configuration for the vSphere cloud provider.
+	//
+	// Deprecated: will be removed in v1alpha4.
 	CloudProviderConfiguration CPIConfig `json:"cloudProviderConfiguration,omitempty"`
 
 	// ControlPlaneEndpoint represents the endpoint used to communicate with the control plane.
@@ -52,7 +60,8 @@ type VSphereClusterSpec struct {
 	// When a LoadBalancerRef is provided, the VSphereCluster.Status.Ready field
 	// will not be true until the referenced resource is Status.Ready and has a
 	// non-empty Status.Address value.
-	// Depracated: will be removed in v1alpha4
+	//
+	// Deprecated: will be removed in v1alpha4.
 	// +optional
 	LoadBalancerRef *corev1.ObjectReference `json:"loadBalancerRef,omitempty"`