Update to Go 1.21

.github/workflows/test_workflow.yaml

@@ -1,6 +1,8 @@
 name: Build and run tests
 # This workflow is triggered on pushes to the repository.
 on: [push, pull_request]
+env:
+  GOTOOLCHAIN: local
 
 jobs:
   lint:
@@ -13,7 +15,7 @@ jobs:
       - name: Set up Go
         uses: actions/setup-go@v4
         with:
-          go-version: '1.20'
+          go-version: '1.21'
           check-latest: true
 
       - name: golangci-lint
@@ -32,7 +34,7 @@ jobs:
       - name: Set up Go
         uses: actions/setup-go@v4
         with:
-          go-version: '1.20'
+          go-version: '1.21'
           check-latest: true
 
       - name: Install govulncheck
@@ -51,7 +53,7 @@ jobs:
       - name: Set up Go
         uses: actions/setup-go@v4
         with:
-          go-version: '1.20'
+          go-version: '1.21'
           check-latest: true
 
       - name: Cache Go modules

Dockerfile

@@ -10,7 +10,7 @@ COPY go.sum go.sum
 RUN go mod download
 
 COPY ./ /workspace/
-RUN CGO_ENABLED=0 GOOS=linux GOARCH=amd64 GO111MODULE=on go build -a -o manager main.go
+RUN CGO_ENABLED=0 GOOS=linux GOARCH=amd64 GO111MODULE=on GOTOOLCHAIN=local go build -a -o manager main.go
 
 # Use distroless as minimal base image to package the manager binary
 # Refer to https://github.com/GoogleContainerTools/distroless for more details

controllers/node_handler.go

@@ -20,10 +20,10 @@
 package controllers
 
 import (
+	"errors"
 	"fmt"
 	"time"
 
-	"github.com/sapcc/maintenance-controller/common"
 	"github.com/sapcc/maintenance-controller/constants"
 	"github.com/sapcc/maintenance-controller/metrics"
 	"github.com/sapcc/maintenance-controller/plugin"
@@ -105,7 +105,7 @@ func ApplyProfiles(params reconcileParameters, data *state.DataV2) error {
 		Labels:   filterNodeLabels(params.node.Labels, params.config.DashboardLabelFilter),
 	})
 	if len(errs) > 0 {
-		return fmt.Errorf("failed to apply current state: %s", common.ConcatErrors(errs))
+		return fmt.Errorf("failed to apply current state: %s", errors.Join(errs...))
 	}
 	for i, ps := range profileStates {
 		result := profileResults[i]

state/state.go

@@ -21,12 +21,12 @@ package state
 
 import (
 	"encoding/json"
+	"errors"
 	"fmt"
 	"strings"
 	"time"
 
 	"github.com/go-logr/logr"
-	"github.com/sapcc/maintenance-controller/common"
 	"github.com/sapcc/maintenance-controller/constants"
 	"github.com/sapcc/maintenance-controller/plugin"
 	v1 "k8s.io/api/core/v1"
@@ -322,7 +322,7 @@ func transitionDefault(params plugin.Parameters, current NodeStateLabel, ts []Tr
 		Infos: results,
 	}
 	if len(errs) > 0 {
-		return final, fmt.Errorf("had failed transition checks: %s", common.ConcatErrors(errs))
+		return final, fmt.Errorf("had failed transition checks: %s", errors.Join(errs...))
 	}
 	for i, result := range results {
 		if !result.Passed {