enable optionalorrequired linter

Signed-off-by: sivchari <shibuuuu5@gmail.com>

Author: sivchari

.golangci-kal.yml

@@ -19,6 +19,7 @@ linters-settings:
           - "conditions" # Ensure conditions have the correct json tags and markers.
           - "integers" # Ensure only int32 and int64 are used for integers.
           - "statussubresource" # All root objects that have a `status` field should have a status subresource.
+          - "optionalorrequired" # Every field should be marked as `+optional` or `+required`.
 
           # Per discussion in July 2024, we are keeping phase fields for now.
           # See https://github.com/kubernetes-sigs/cluster-api/pull/10897#discussion_r1685929508
@@ -30,7 +31,6 @@ linters-settings:
           # - "jsontags" # Ensure every field has a json tag.
           # - "maxlength" # Ensure all strings and arrays have maximum lengths/maximum items.
           # - "nobools" # Bools do not evolve over time, should use enums instead.
-          # - "optionalorrequired" # Every field should be marked as `+optional` or `+required`.
           # - "requiredfields" # Required fields should not be pointers, and should not have `omitempty`.
           disable:
           - "*" # We will manually enable new linters after understanding the impact. Disable all by default.
@@ -41,9 +41,6 @@ linters-settings:
             useProtobuf: Forbid # We don't use protobuf, so protobuf tags are not required.
         # jsonTags:
         #   jsonTagRegex: "^[a-z][a-z0-9]*(?:[A-Z][a-z0-9]*)*$" # The default regex is appropriate for our use case.
-        # optionalOrRequired:
-        #   preferredOptionalMarker: optional | kubebuilder:validation:Optional # The preferred optional marker to use, fixes will suggest to use this marker. Defaults to `optional`.
-        #   preferredRequiredMarker: required | kubebuilder:validation:Required # The preferred required marker to use, fixes will suggest to use this marker. Defaults to `required`.
         # requiredFields:
         #   pointerPolicy: Warn | SuggestFix # Defaults to `SuggestFix`. We want our required fields to not be pointers.
 
@@ -58,7 +55,7 @@ issues:
   max-issues-per-linter: 0
   exclude-rules:
   # KAL should only run on API folders.
-  - path-except: "api/*"
+  - path-except: "api//*"
     linters:
       - kal
   - path: "api/v1beta1/*|api/v1alpha1/*"
@@ -73,3 +70,7 @@ issues:
     text: "field Prefix should not use an int, int8 or int16. Use int32 or int64 depending on bounding requirements"
     linters:
       - kal
+  - path: "api/v1alpha1/*|api/v1alpha3/*|api/v1beta1/*"
+    text: "optionalorrequired"
+    linters:
+      - kal