fix deprecated warnings and restrict KAL linter exclusions to v1alpha2 APIs

Author: eljohnson92

.golangci-kal.yml

@@ -15,19 +15,26 @@ linters:
         type: "module"
         description: Kube API LInter lints Kube like APIs based on API conventions and best practices.
         settings:
-          linters:
-            # TODO: remove linters from this list as we add support from them
-            disable:
-              # integers linter will force type change from int to int32/int64, we are going to wait for v1beta1 to enable this
-              - "integers"
-              # optionalfields requires changing fields to pointers in a way that might be hard to support in a backwards compatible way
-              - "optionalfields"
-          lintersConfig: { }
+          linters: {}
+          lintersConfig:
+            optionalfields:
+              pointers:
+                preference: WhenRequired
   exclusions:
     rules:
       - path-except: "^api/"
         linters:
           - kubeapilinter
+      # integers linter will force type change from int to int32/int64, we are going to wait for v1beta1 to enable this
+      - path: "api/v1alpha2/*"
+        text: "integers"
+        linters:
+          - kubeapilinter
+      # optionalfields requires changing fields to pointers in a way that might be hard to support in a backwards compatible way
+      - path: "api/v1alpha2/*"
+        text: "optionalfields"
+        linters:
+          - kubeapilinter
 issues:
   max-issues-per-linter: 0
   max-same-issues: 0

api/v1alpha2/linodemachine_types.go

@@ -528,7 +528,9 @@ type LinodeMachineStatus struct {
 	Addresses []clusterv1.MachineAddress `json:"addresses,omitempty"`
 
 	// cloudinitMetadataSupport determines whether to use cloud-init or not.
-	// Deprecated: Stackscript no longer in use, so this field is not used.
+	//
+	// Deprecated: stackscript are no longer in use, so this field is not used.
+	//
 	// +kubebuilder:deprecatedversion:warning="CloudinitMetadataSupport is deprecated"
 	// +optional
 	// +kubebuilder:default=true

api/v1alpha2/linodeobjectstoragekey_types.go

@@ -91,14 +91,18 @@ type LinodeObjectStorageKeySpec struct {
 	GeneratedSecret `json:"generatedSecret"`
 
 	// secretType instructs the controller what type of secret to generate containing access key details.
-	// Deprecated: Use generatedSecret.type.
+	//
+	// Deprecated: secretType is no longer supported, Use generatedSecret.type.
+	//
 	// +kubebuilder:validation:Enum=Opaque;addons.cluster.x-k8s.io/resource-set
 	// +kubebuilder:deprecatedversion:warning="secretType deprecated by generatedSecret.type"
 	// +optional
 	SecretType corev1.SecretType `json:"secretType,omitempty"`
 
 	// secretDataFormat instructs the controller how to format the data stored in the secret containing access key details.
-	// Deprecated: Use generatedSecret.format.
+	//
+	// Deprecated: secretDataFormat is no longer supported, please use generatedSecret.format.
+	//
 	// +kubebuilder:deprecatedversion:warning="secretDataFormat deprecated by generatedSecret.format"
 	// +optional
 	SecretDataFormat map[string]string `json:"secretDataFormat,omitempty"`

config/crd/bases/infrastructure.cluster.x-k8s.io_linodemachines.yaml

@@ -941,7 +941,8 @@ spec:
                 default: true
                 description: |-
                   cloudinitMetadataSupport determines whether to use cloud-init or not.
-                  Deprecated: Stackscript no longer in use, so this field is not used.
+
+                  Deprecated: stackscript are no longer in use, so this field is not used.
                 type: boolean
               conditions:
                 description: conditions define the current service state of the LinodeMachine.

config/crd/bases/infrastructure.cluster.x-k8s.io_linodeobjectstoragekeys.yaml

@@ -145,12 +145,14 @@ spec:
                   type: string
                 description: |-
                   secretDataFormat instructs the controller how to format the data stored in the secret containing access key details.
-                  Deprecated: Use generatedSecret.format.
+
+                  Deprecated: secretDataFormat is no longer supported, please use generatedSecret.format.
                 type: object
               secretType:
                 description: |-
                   secretType instructs the controller what type of secret to generate containing access key details.
-                  Deprecated: Use generatedSecret.type.
+
+                  Deprecated: secretType is no longer supported, Use generatedSecret.type.
                 enum:
                 - Opaque
                 - addons.cluster.x-k8s.io/resource-set