project-infra: add make targets lint and coverage

.golangci.yaml

@@ -0,0 +1,14 @@
+linters:
+  # initially we disable all linters, since this is an old code base with quite a lot of issues
+  disable-all: true
+  enable:
+  - gocyclo
+#  - staticcheck
+#  - unused
+#  - ineffassign
+#  - govet
+#  - gosimple
+#  - errcheck
+#linters-settings:
+#  gocyclo:
+#    min-complexity: 10

Makefile

@@ -5,11 +5,28 @@ querier := robots/cmd/release-querier
 kubevirtci := robots/cmd/kubevirtci-bumper
 bazelbin := bazelisk
 
-.PHONY: all clean deps-update gazelle-update-repos update-labels $(limiter) $(flake-report-writer) $(querier) $(kubevirtci) $(flake-issue-creator)
+ifndef GOLANGCI_LINT_VERSION
+	GOLANGCI_LINT_VERSION=v1.62.2
+	export GOLANGCI_LINT_VERSION
+endif
+ifndef ARTIFACTS
+	ARTIFACTS=/tmp/artifacts
+	export ARTIFACTS
+endif
+ifndef COVERAGE_OUTPUT_PATH
+	COVERAGE_OUTPUT_PATH=${ARTIFACTS}/coverage.html
+	export COVERAGE_OUTPUT_PATH
+endif
+
+.PHONY: all make-artifacts-dir clean deps-update gazelle-update-repos update-labels install-metrics-binaries lint $(limiter) $(flake-report-writer) $(querier) $(kubevirtci) $(flake-issue-creator)
 all: deps-update $(limiter) $(flake-report-writer) $(querier) $(kubevirtci) $(flake-issue-creator)
 
-clean:
+make-artifacts-dir:
+	mkdir -p ${ARTIFACTS}
+
+clean: install-metrics-binaries
 	$(bazelbin) clean
+	golangci-lint cache clean
 
 $(limiter) $(flake-report-writer) $(querier) $(kubevirtci) $(flake-issue-creator): deps-update
 	$(MAKE) --directory=$@
@@ -40,3 +57,18 @@ test:
 
 update-labels:
 	./hack/labels/update.sh
+
+install-metrics-binaries:
+	if ! command -V golangci-lint; then curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b ${GOPATH}/bin ${GOLANGCI_LINT_VERSION} ; fi
+
+lint: install-metrics-binaries
+	./hack/lint.sh
+
+coverage: make-artifacts-dir
+	if ! command -V covreport; then go install github.com/cancue/covreport@latest; fi
+	go test \
+		./external-plugins/... \
+		./releng/... \
+		./robots/... \
+		-coverprofile=/tmp/coverage.out
+	covreport  -i /tmp/coverage.out -o ${COVERAGE_OUTPUT_PATH}

github/ci/prow-deploy/files/jobs/kubevirt/project-infra/project-infra-presubmits.yaml

@@ -138,6 +138,46 @@ presubmits:
             memory: "4Gi"
         securityContext:
           runAsUser: 0
+  - always_run: false
+    cluster: kubevirt-prow-control-plane
+    decorate: true
+    name: pull-project-infra-lint
+    run_if_changed: "external-plugins/.*|releng/.*|robots/.*"
+    spec:
+      containers:
+      - args:
+        - make lint
+        command:
+        - "/usr/local/bin/runner.sh"
+        - "/bin/sh"
+        - "-ce"
+        image: quay.io/kubevirtci/golang:v20241206-ae344ab
+        resources:
+          requests:
+            memory: "1Gi"
+          limits:
+            memory: "1Gi"
+  - always_run: false
+    cluster: kubevirt-prow-control-plane
+    decorate: true
+    name: pull-project-infra-coverage
+    run_if_changed: "external-plugins/.*|releng/.*|robots/.*"
+    spec:
+      containers:
+      - args:
+        - |
+          GOBIN=/usr/local/bin/ go install github.com/onsi/ginkgo/v2/[email protected]
+          make coverage
+        command:
+        - "/usr/local/bin/runner.sh"
+        - "/bin/sh"
+        - "-ce"
+        image: quay.io/kubevirtci/golang:v20241206-ae344ab
+        resources:
+          requests:
+            memory: "1Gi"
+          limits:
+            memory: "1Gi"
   - name: build-kubevirt-infra-bootstrap-image
     always_run: false
     run_if_changed: "images/kubevirt-infra-bootstrap/.*"

hack/lint.sh

@@ -0,0 +1,30 @@
+#!/usr/bin/env bash
+#
+# This file is part of the KubeVirt project
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+# Copyright the KubeVirt Authors.
+#
+#
+
+set -e
+set -u
+set -o pipefail
+
+golangci-lint --version
+golangci-lint run
+
+if [ $(git grep '//nolint' -- '**/*.go' | wc -l) -gt 0 ]; then
+    echo 'WARNING: nolint directive detected';
+fi

releng/release-tool/release-tool.go

@@ -772,7 +772,7 @@ func (r *releaseData) getReleases() ([]*github.RepositoryRelease, error) {
 	return r.allReleases, nil
 }
 
-func (r *releaseData) autoDetectData(autoReleaseCadance string, autoPromoteAfterDays int) error {
+func (r *releaseData) autoDetectData(autoReleaseCadance string, autoPromoteAfterDays int) error { //nolint:gocyclo
 
 	log.Printf("Attempting to auto detect release for %s/%s", r.org, r.repo)
 
@@ -975,7 +975,7 @@ func (r *releaseData) autoDetectData(autoReleaseCadance string, autoPromoteAfter
 	return nil
 }
 
-func (r *releaseData) verifyTag() error {
+func (r *releaseData) verifyTag() error { //nolint:gocyclo
 	// must be a valid semver version
 	tagSemver, err := semver.NewVersion(r.tag)
 	if err != nil {

robots/pkg/kubevirt/cmd/get/support-matrix.go

@@ -108,7 +108,7 @@ type SupportMatrixTemplateData struct {
 	MapK6tToK8sVersions  map[string]map[string]bool
 }
 
-func GenerateMarkdownForSupportMatrix(_ *cobra.Command, _ []string) error {
+func GenerateMarkdownForSupportMatrix(_ *cobra.Command, _ []string) error { //nolint:gocyclo
 	if err := getSupportMatrixOpts.validateOptions(); err != nil {
 		return fmt.Errorf("failed to validate options: %v", err)
 	}