Use new 1.23.6 builder (#3652)

* Use new 1.23.6 builder

Signed-off-by: Alex Kalenyuk <[email protected]>

* Bump linter to 1.60.3 for go 1.23 support

Signed-off-by: Alex Kalenyuk <[email protected]>

* Disable linter failures over G115

Signed-off-by: Alex Kalenyuk <[email protected]>

* Fix lint issues related to error format formatting

Signed-off-by: Alex Kalenyuk <[email protected]>

* Address remaining lint failures

len is enough/sprintf not really used

Signed-off-by: Alex Kalenyuk <[email protected]>

---------

Signed-off-by: Alex Kalenyuk <[email protected]>

Author: akalenyu

.golangci.yml

@@ -49,3 +49,8 @@ linters-settings:
     forbid-focus-container: true
   nakedret:
     max-func-lines: 0
+  gosec:
+    excludes:
+      # This is changing things up quite a bit and requires evaluation
+      # per conversion
+      - G115

go.mod

@@ -1,6 +1,6 @@
 module kubevirt.io/containerized-data-importer
 
-go 1.22.0
+go 1.23.0
 
 require (
 	cloud.google.com/go/storage v1.38.0

hack/build/config.sh

@@ -29,7 +29,7 @@ FUNC_TEST_PROXY="cdi-func-test-proxy"
 FUNC_TEST_POPULATOR="cdi-func-test-sample-populator"
 
 # update this whenever new builder tag is created
-BUILDER_IMAGE=${BUILDER_IMAGE:-quay.io/kubevirt/kubevirt-cdi-bazel-builder:2409021308-e6939c6c2}
+BUILDER_IMAGE=${BUILDER_IMAGE:-quay.io/kubevirt/kubevirt-cdi-bazel-builder:2503021312-67d1f3d4d}
 
 BINARIES="cmd/${OPERATOR} cmd/${CONTROLLER} cmd/${IMPORTER} cmd/${CLONER} cmd/${APISERVER} cmd/${UPLOADPROXY} cmd/${UPLOADSERVER} cmd/${OPERATOR} tools/${FUNC_TEST_INIT} tools/${FUNC_TEST_REGISTRY_INIT} tools/${FUNC_TEST_BAD_WEBSERVER} tools/${FUNC_TEST_PROXY} tools/${FUNC_TEST_POPULATOR}"
 CDI_PKGS="cmd/ pkg/ test/"

hack/build/run-linters.sh

@@ -1,6 +1,6 @@
 #!/bin/sh -ex
 
-GOLANGCI_VERSION="${GOLANGCI_VERSION:-v1.58.0}"
+GOLANGCI_VERSION="${GOLANGCI_VERSION:-v1.60.3}"
 MONITORING_LINTER_VERSION="${MONITORING_LINTER_VERSION:-e2be790}"
 
 go install "github.com/golangci/golangci-lint/cmd/golangci-lint@${GOLANGCI_VERSION}"

pkg/apiserver/webhooks/cdi-validate.go

@@ -34,7 +34,7 @@ import (
 	sdkapi "kubevirt.io/controller-lifecycle-operator-sdk/api"
 )
 
-const uninstallErrorMsg = "Rejecting the uninstall request, since there are still DataVolumes present. Either delete all DataVolumes or change the uninstall strategy before uninstalling CDI."
+const uninstallErrorMsg = "rejecting the uninstall request, since there are still %d DataVolumes present. Either delete all DataVolumes or change the uninstall strategy before uninstalling CDI"
 
 type cdiValidatingWebhook struct {
 	client cdiclient.Interface
@@ -69,8 +69,8 @@ func (wh *cdiValidatingWebhook) Admit(ar admissionv1.AdmissionReview) *admission
 			return toAdmissionResponseError(err)
 		}
 
-		if len(dvs.Items) > 0 {
-			return toAdmissionResponseError(fmt.Errorf(uninstallErrorMsg))
+		if numDvs := len(dvs.Items); numDvs > 0 {
+			return toAdmissionResponseError(fmt.Errorf(uninstallErrorMsg, numDvs))
 		}
 	}
 

pkg/apiserver/webhooks/cdi-validate_test.go

@@ -155,7 +155,7 @@ var _ = Describe("CDI Delete Webhook", func() {
 
 			resp := validateCDIs(ar, newDataVolumeWithName("foo"))
 			Expect(resp.Allowed).To(BeFalse())
-			Expect(resp.Result.Message).To(ContainSubstring("Rejecting the uninstall request, since there are still DataVolumes present."))
+			Expect(resp.Result.Message).To(ContainSubstring("rejecting the uninstall request, since there are still 1 DataVolumes present."))
 		})
 
 		It("should reject with DataVolumes present and oldobject not populated", func() {
@@ -187,7 +187,7 @@ var _ = Describe("CDI Delete Webhook", func() {
 
 			resp := validateCDIs(ar, cdi, newDataVolumeWithName("foo"))
 			Expect(resp.Allowed).To(BeFalse())
-			Expect(resp.Result.Message).To(ContainSubstring("Rejecting the uninstall request, since there are still DataVolumes present."))
+			Expect(resp.Result.Message).To(ContainSubstring("rejecting the uninstall request, since there are still 1 DataVolumes present."))
 		})
 
 		It("should allow error CDI to be deleted with DataVolumes present", func() {

pkg/controller/datavolume/controller-base.go

@@ -716,7 +716,7 @@ func (r *ReconcilerBase) validatePVC(dv *cdiv1.DataVolume, pvc *corev1.Persisten
 	if pvc.DeletionTimestamp != nil {
 		msg := fmt.Sprintf(MessageResourceMarkedForDeletion, pvc.Name)
 		r.recorder.Event(dv, corev1.EventTypeWarning, ErrResourceMarkedForDeletion, msg)
-		return errors.Errorf(msg)
+		return errors.New(msg)
 	}
 	// If the PVC is not controlled by this DataVolume resource, we should log
 	// a warning to the event recorder and return
@@ -732,7 +732,7 @@ func (r *ReconcilerBase) validatePVC(dv *cdiv1.DataVolume, pvc *corev1.Persisten
 		} else {
 			msg := fmt.Sprintf(MessageResourceExists, pvc.Name)
 			r.recorder.Event(dv, corev1.EventTypeWarning, ErrResourceExists, msg)
-			return errors.Errorf(msg)
+			return errors.New(msg)
 		}
 	}
 	return nil

pkg/controller/populators/forklift-populator.go

@@ -321,7 +321,7 @@ func (r *ForkliftPopulatorReconciler) reconcileTargetPVC(pvc, pvcPrime *corev1.P
 		return reconcile.Result{}, nil
 	}
 
-	if pod.Status.ContainerStatuses == nil || len(pod.Status.ContainerStatuses) == 0 {
+	if len(pod.Status.ContainerStatuses) == 0 {
 		return reconcile.Result{RequeueAfter: 2 * time.Second}, nil
 	}
 

pkg/image/qemu.go

@@ -217,7 +217,7 @@ func (o *qemuOperations) Info(url *url.URL) (*ImgInfo, error) {
 		if nbdkitLog, err := os.ReadFile(common.NbdkitLogPath); err == nil {
 			errorMsg += " " + string(nbdkitLog)
 		}
-		return nil, errors.Errorf(errorMsg)
+		return nil, errors.New(errorMsg)
 	}
 	return checkOutputQemuImgInfo(output, url.String())
 }

pkg/importer/gcs-datasource.go

@@ -2,7 +2,6 @@ package importer
 
 import (
 	"context"
-	"fmt"
 	"io"
 	"net/url"
 	"path/filepath"
@@ -56,7 +55,7 @@ func NewGCSDataSource(endpoint, keyFile string) (*GCSDataSource, error) {
 	ep, err := ParseEndpoint(endpoint)
 
 	if err != nil {
-		return nil, errors.Wrapf(err, fmt.Sprintf("GCS Importer: unable to parse endpoint %q", endpoint))
+		return nil, errors.Wrapf(err, "GCS Importer: unable to parse endpoint %q", endpoint)
 	}
 
 	// Getting Context