only install -kiss

Author: camilamacedo86

.github/workflows/lint-sample.yml

@@ -46,7 +46,6 @@ jobs:
         with:
           version: v2.2.2
           working-directory: ${{ matrix.folder }}
-          install-mode: goinstall
       - name: Run linter via makefile target
         working-directory: ${{ matrix.folder }}
         run: make lint

docs/book/src/cronjob-tutorial/testdata/project/.github/workflows/lint.yml

@@ -21,4 +21,3 @@ jobs:
         uses: golangci/golangci-lint-action@v8
         with:
           version: v2.2.2
-          install-mode: goinstall

docs/book/src/cronjob-tutorial/testdata/project/.golangci.yml

@@ -31,10 +31,6 @@ linters:
       kubeapilinter:
         path: "./bin/kube-api-linter.so"
         description: "Kube API Linter plugin"
-        original-url: "sigs.k8s.io/kube-api-linter"
-        settings:
-          linters: { }
-          lintersConfig: { }
   exclusions:
     generated: lax
     rules:

docs/book/src/cronjob-tutorial/testdata/project/Makefile

@@ -225,22 +225,20 @@ golangci-lint: $(GOLANGCI_LINT) ## Download golangci-lint locally if necessary.
 $(GOLANGCI_LINT): $(LOCALBIN)
 	$(call go-install-tool,$(GOLANGCI_LINT),github.com/golangci/golangci-lint/v2/cmd/golangci-lint,$(GOLANGCI_LINT_VERSION))
 
-# To lint Kubernetes API definitions.
+# To lint Kubernetes API definitions (CRDs).
 # More info: https://github.com/kubernetes-sigs/kube-api-linter
+# Build kube-api-linter plugin (used by golangci-lint as a custom plugin)
 KUBE_API_LINTER_PLUGIN := $(LOCALBIN)/kube-api-linter.so
-KUBE_API_LINTER_BUILD_DIR := ./hack/kube-api-linter
 
 .PHONY: kube-api-linter
-kube-api-linter: $(KUBE_API_LINTER_PLUGIN) ## Build the kube-api-linter plugin
-$(KUBE_API_LINTER_PLUGIN): $(KUBE_API_LINTER_BUILD_DIR)/go.mod
-	cd $(KUBE_API_LINTER_BUILD_DIR) && \
-	go build -buildmode=plugin -o "$(abspath $(KUBE_API_LINTER_PLUGIN))" sigs.k8s.io/kube-api-linter/pkg/plugin
-$(KUBE_API_LINTER_BUILD_DIR)/go.mod:
-	@echo "Setting up local module for kube-api-linter plugin..."
-	mkdir -p $(KUBE_API_LINTER_BUILD_DIR)
-	cd $(KUBE_API_LINTER_BUILD_DIR) && \
-	go mod init local-kube-api-linter && \
-	go get sigs.k8s.io/kube-api-linter@latest
+kube-api-linter: ## Build kube-api-linter plugin as .so
+	@echo "Building kube-api-linter plugin..."
+	rm -rf ./hack/kube-api-linter
+	mkdir -p ./hack/kube-api-linter
+	cd ./hack/kube-api-linter && \
+		go mod init tmp-kube-api-linter && \
+		go get sigs.k8s.io/kube-api-linter@latest && \
+		go build -buildmode=plugin -o "$(KUBE_API_LINTER_PLUGIN)" sigs.k8s.io/kube-api-linter/pkg/plugin
 
 # go-install-tool will 'go install' any package with custom target and name of binary, if it doesn't exist
 # $1 - target path with name of binary

docs/book/src/cronjob-tutorial/testdata/project/api/v1/cronjob_types.go

@@ -65,6 +65,7 @@ import (
 // CronJobSpec defines the desired state of CronJob
 type CronJobSpec struct {
 	// schedule in Cron format, see https://en.wikipedia.org/wiki/Cron.
+	// +kubebuilder:validation:MinLength=0
 	// +required
 	Schedule string `json:"schedule"`
 
@@ -80,7 +81,6 @@ type CronJobSpec struct {
 	// - "Forbid": forbids concurrent runs, skipping next run if previous run hasn't finished yet;
 	// - "Replace": cancels currently running job and replaces it with a new one
 	// +optional
-	// +kubebuilder:default:=Allow
 	ConcurrencyPolicy ConcurrencyPolicy `json:"concurrencyPolicy,omitempty"`
 
 	// suspend tells the controller to suspend subsequent executions, it does
@@ -89,7 +89,6 @@ type CronJobSpec struct {
 	Suspend *bool `json:"suspend,omitempty"`
 
 	// jobTemplate defines the job that will be created when executing a CronJob.
-	// +required
 	JobTemplate batchv1.JobTemplateSpec `json:"jobTemplate"`
 
 	// successfulJobsHistoryLimit defines the number of successful finished jobs to retain.
@@ -147,9 +146,6 @@ type CronJobStatus struct {
 
 	// active defines a list of pointers to currently running jobs.
 	// +optional
-	// +listType=atomic
-	// +kubebuilder:validation:MaxItems=10
-	// +kubebuilder:validation:MinItems=0
 	Active []corev1.ObjectReference `json:"active,omitempty"`
 
 	// lastScheduleTime defines when was the last time the job was successfully scheduled.

docs/book/src/cronjob-tutorial/testdata/project/config/crd/bases/batch.tutorial.kubebuilder.io_cronjobs.yaml

@@ -27,7 +27,6 @@ spec:
           spec:
             properties:
               concurrencyPolicy:
-                default: Allow
                 enum:
                 - Allow
                 - Forbid
@@ -3799,6 +3798,7 @@ spec:
                     type: object
                 type: object
               schedule:
+                minLength: 0
                 type: string
               startingDeadlineSeconds:
                 format: int64
@@ -3835,10 +3835,7 @@ spec:
                       type: string
                   type: object
                   x-kubernetes-map-type: atomic
-                maxItems: 10
-                minItems: 0
                 type: array
-                x-kubernetes-list-type: atomic
               lastScheduleTime:
                 format: date-time
                 type: string

docs/book/src/cronjob-tutorial/testdata/project/dist/chart/templates/crd/batch.tutorial.kubebuilder.io_cronjobs.yaml

@@ -33,7 +33,6 @@ spec:
           spec:
             properties:
               concurrencyPolicy:
-                default: Allow
                 enum:
                 - Allow
                 - Forbid
@@ -3805,6 +3804,7 @@ spec:
                     type: object
                 type: object
               schedule:
+                minLength: 0
                 type: string
               startingDeadlineSeconds:
                 format: int64
@@ -3841,10 +3841,7 @@ spec:
                       type: string
                   type: object
                   x-kubernetes-map-type: atomic
-                maxItems: 10
-                minItems: 0
                 type: array
-                x-kubernetes-list-type: atomic
               lastScheduleTime:
                 format: date-time
                 type: string

docs/book/src/cronjob-tutorial/testdata/project/dist/install.yaml

@@ -35,7 +35,6 @@ spec:
           spec:
             properties:
               concurrencyPolicy:
-                default: Allow
                 enum:
                 - Allow
                 - Forbid
@@ -3807,6 +3806,7 @@ spec:
                     type: object
                 type: object
               schedule:
+                minLength: 0
                 type: string
               startingDeadlineSeconds:
                 format: int64
@@ -3843,10 +3843,7 @@ spec:
                       type: string
                   type: object
                   x-kubernetes-map-type: atomic
-                maxItems: 10
-                minItems: 0
                 type: array
-                x-kubernetes-list-type: atomic
               lastScheduleTime:
                 format: date-time
                 type: string

docs/book/src/cronjob-tutorial/testdata/project/go.mod

@@ -9,7 +9,6 @@ require (
 	k8s.io/api v0.33.0
 	k8s.io/apimachinery v0.33.0
 	k8s.io/client-go v0.33.0
-	k8s.io/utils v0.0.0-20241104100929-3ea5e8cea738
 	sigs.k8s.io/controller-runtime v0.21.0
 )
 
@@ -90,6 +89,7 @@ require (
 	k8s.io/component-base v0.33.0 // indirect
 	k8s.io/klog/v2 v2.130.1 // indirect
 	k8s.io/kube-openapi v0.0.0-20250318190949-c8a335a9a2ff // indirect
+	k8s.io/utils v0.0.0-20241104100929-3ea5e8cea738 // indirect
 	sigs.k8s.io/apiserver-network-proxy/konnectivity-client v0.31.2 // indirect
 	sigs.k8s.io/json v0.0.0-20241010143419-9aa6b5e7a4b3 // indirect
 	sigs.k8s.io/randfill v1.0.0 // indirect

docs/book/src/cronjob-tutorial/testdata/project/internal/webhook/v1/cronjob_webhook_test.go

@@ -22,7 +22,6 @@ import (
 
 	batchv1 "tutorial.kubebuilder.io/project/api/v1"
 	// TODO (user): Add any additional imports if needed
-	"k8s.io/utils/ptr"
 )
 
 var _ = Describe("CronJob Webhook", func() {
@@ -41,8 +40,8 @@ var _ = Describe("CronJob Webhook", func() {
 			Spec: batchv1.CronJobSpec{
 				Schedule:                   schedule,
 				ConcurrencyPolicy:          batchv1.AllowConcurrent,
-				SuccessfulJobsHistoryLimit: ptr.To(int32(3)),
-				FailedJobsHistoryLimit:     ptr.To(int32(1)),
+				SuccessfulJobsHistoryLimit: new(int32),
+				FailedJobsHistoryLimit:     new(int32),
 			},
 		}
 		*obj.Spec.SuccessfulJobsHistoryLimit = 3
@@ -52,8 +51,8 @@ var _ = Describe("CronJob Webhook", func() {
 			Spec: batchv1.CronJobSpec{
 				Schedule:                   schedule,
 				ConcurrencyPolicy:          batchv1.AllowConcurrent,
-				SuccessfulJobsHistoryLimit: ptr.To(int32(3)),
-				FailedJobsHistoryLimit:     ptr.To(int32(1)),
+				SuccessfulJobsHistoryLimit: new(int32),
+				FailedJobsHistoryLimit:     new(int32),
 			},
 		}
 		*oldObj.Spec.SuccessfulJobsHistoryLimit = 3