Update Go version in go.mod to 1.22 (Go 1.22: 2/3)

.ci-operator.yaml

@@ -1,4 +1,4 @@
 build_root_image:
   namespace: openshift
   name: release
-  tag: golang-1.21
+  tag: golang-1.22

Dockerfiles/Dockerfile

@@ -1,5 +1,5 @@
 # Build the manager binary
-ARG GOLANG_VERSION=1.21
+ARG GOLANG_VERSION=1.22
 
 ################################################################################
 FROM registry.access.redhat.com/ubi8/toolbox as manifests

Dockerfiles/toolbox.Dockerfile

@@ -1,6 +1,6 @@
 FROM registry.fedoraproject.org/fedora-toolbox:38
 
-ARG GOLANG_VERSION=1.21
+ARG GOLANG_VERSION=1.22.10
 ARG OPERATOR_SDK_VERSION=1.31.0
 
 ENV GOLANG_VERSION=$GOLANG_VERSION \

Makefile

@@ -357,7 +357,7 @@ catalog-build: opm ## Build a catalog image.
 catalog-push: ## Push a catalog image.
 	$(MAKE) image-push IMG=$(CATALOG_IMG)
 
-TOOLBOX_GOLANG_VERSION := 1.21
+TOOLBOX_GOLANG_VERSION := 1.22.10
 
 # Generate a Toolbox container for locally testing changes easily
 .PHONY: toolbox

README.md

@@ -77,7 +77,7 @@ Additionally installing `Authorino operator` & `Service Mesh operator` enhances
 
 #### Pre-requisites
 
-- Go version **go1.21**
+- Go version **go1.22**
 - operator-sdk version can be updated to **v1.31.1**
 
 #### Download manifests

controllers/secretgenerator/secret.go

@@ -90,7 +90,7 @@ func generateSecretValue(secret *Secret) error {
 	switch secret.Type {
 	case "random":
 		randomValue := make([]byte, secret.Complexity)
-		for i := 0; i < secret.Complexity; i++ {
+		for i := 0; i < secret.Complexity; i++ { //nolint: intrange
 			num, err := rand.Int(rand.Reader, big.NewInt(int64(len(letterRunes))))
 			if err != nil {
 				return err

controllers/secretgenerator/secret_test.go

@@ -92,7 +92,7 @@ func TestNewSecret(t *testing.T) {
 	}
 
 	for name, tc := range cases {
-		tc := tc
+		tc := tc //nolint: copyloopvar
 		t.Run(name, func(t *testing.T) {
 			secret, err := secretgenerator.NewSecretFrom(tc.annotations)
 			if err != nil {

go.mod

@@ -1,6 +1,6 @@
 module github.com/opendatahub-io/opendatahub-operator/v2
 
-go 1.21
+go 1.22
 
 require (
 	github.com/blang/semver/v4 v4.0.0

pkg/controller/actions/render/kustomize/action_render_manifests_test.go

@@ -204,7 +204,7 @@ func TestRenderResourcesWithCacheAction(t *testing.T) {
 
 	render.RenderedResourcesTotal.Reset()
 
-	for i := int64(0); i < 3; i++ {
+	for i := int64(0); i < 3; i++ { //nolint: intrange
 		d := componentApi.Dashboard{}
 
 		if i >= 1 {

pkg/controller/actions/render/template/action_render_templates_test.go

@@ -157,7 +157,7 @@ func TestRenderTemplateWithCache(t *testing.T) {
 		},
 	}
 
-	for i := int64(0); i < 3; i++ {
+	for i := int64(0); i < 3; i++ { //nolint: intrange
 		d := componentApi.Dashboard{
 			ObjectMeta: metav1.ObjectMeta{
 				Name: ns,

pkg/upgrade/uninstallation.go

@@ -48,7 +48,7 @@
 	}
 
 	for _, namespace := range generatedNamespaces.Items {
-		namespace := namespace
+		namespace := namespace //nolint: copyloopvar
 		if namespace.Status.Phase == corev1.NamespaceActive {
 			if err := cli.Delete(ctx, &namespace); err != nil {
 				return fmt.Errorf("error deleting namespace %v: %w", namespace.Name, err)
@@ -95,7 +95,7 @@
 
 	var multiErr *multierror.Error
 	for _, dsciInstance := range instanceList.Items {
-		dsciInstance := dsciInstance
+		dsciInstance := dsciInstance //nolint: copyloopvar
 		if err := cli.Delete(ctx, &dsciInstance); !k8serr.IsNotFound(err) {
 			multiErr = multierror.Append(multiErr, err)
 		}

pkg/upgrade/upgrade.go

@@ -313,7 +313,7 @@
 	}
 
 	for _, item := range list.Items {
-		item := item
+		item := item //nolint: copyloopvar
 		v, ok, err := unstructured.NestedString(item.Object, res.Path...)
 		if err != nil {
 			return fmt.Errorf("failed to get field %v for %s %s/%s: %w", res.Path, res.Gvk.Kind, res.Namespace, item.GetName(), err)
@@ -345,7 +345,7 @@
 		multiErr = multierror.Append(multiErr, err)
 	}
 	items := reflect.ValueOf(resourceType).Elem().FieldByName("Items")
-	for i := 0; i < items.Len(); i++ {
+	for i := 0; i < items.Len(); i++ { //nolint: intrange
 		item := items.Index(i).Addr().Interface().(client.Object) //nolint:errcheck,forcetypeassert
 		for _, name := range resourceList {
 			if name == item.GetName() {
@@ -376,7 +376,7 @@
 	}
 
 	for _, servicemonitor := range servicemonitors.Items {
-		servicemonitor := servicemonitor
+		servicemonitor := servicemonitor //nolint: copyloopvar
 		for _, name := range resourceList {
 			if name == servicemonitor.Name {
 				log.Info("Attempting to delete " + servicemonitor.Name + " in namespace " + namespace)

tests/e2e/helper_test.go

@@ -256,7 +256,7 @@ func getCSV(ctx context.Context, cli client.Client, name string, namespace strin
 	}
 
 	// do not use range Items to avoid pointer to the loop variable
-	for i := 0; i < len(csvList.Items); i++ {
+	for i := 0; i < len(csvList.Items); i++ { //nolint: intrange
 		csv := &csvList.Items[i]
 		if isMatched(csv, name) {
 			return csv, nil
@@ -416,15 +416,15 @@ func ensureServicemeshOperators(t *testing.T, tc *testContext) error { //nolint:
 	c := make(chan error)
 
 	for _, op := range ops {
-		op := op // to avoid loop variable in the closures
+		op := op //nolint: copyloopvar
 		t.Logf("Ensuring %s is installed", op)
 		go func(op string) {
 			err := ensureOperator(tc, op, servicemeshNamespace)
 			c <- err
 		}(op)
 	}
 
-	for i := 0; i < len(ops); i++ {
+	for i := 0; i < len(ops); i++ { //nolint: intrange
 		err := <-c
 		errors = multierror.Append(errors, err)
 	}

tests/envtestutil/cleaner.go

@@ -43,7 +43,7 @@ func CreateCleaner(c client.Client, config *rest.Config, timeout, interval time.
 
 func (c *Cleaner) DeleteAll(ctx context.Context, objects ...client.Object) {
 	for _, obj := range objects {
-		obj := obj
+		obj := obj //nolint: copyloopvar
 		Expect(client.IgnoreNotFound(c.client.Delete(ctx, obj))).Should(Succeed())
 
 		if ns, ok := obj.(*corev1.Namespace); ok {