BUILD/MAJOR: go: upgrade to 1.25

Author: mjuraga

.gitlab-ci.yml

@@ -1,7 +1,7 @@
 ---
 variables:
   DOCKER_HOST: tcp://docker:2375
-  GO_VERSION: "1.24"
+  GO_VERSION: "1.25"
   DOCKER_VERSION: "26.0"
 stages:
   - bots
@@ -216,35 +216,7 @@ test-equal:
     - docker image tag $CI_REGISTRY_GO/haproxy-alpine:$HAPROXY_VERSION haproxytech/haproxy-alpine:$HAPROXY_VERSION
   script:
     - DOCKER_HAPROXY_VERSION=$HAPROXY_VERSION make e2e-docker
-HAProxy-2.2:
+HAProxy-3.3:
   extends: .e2e
   variables:
-    HAPROXY_VERSION: "2.2"
-HAProxy-2.4:
-  extends: .e2e
-  variables:
-    HAPROXY_VERSION: "2.4"
-HAProxy-2.6:
-  extends: .e2e
-  variables:
-    HAPROXY_VERSION: "2.6"
-HAProxy-2.8:
-  extends: .e2e
-  variables:
-    HAPROXY_VERSION: "2.8"
-HAProxy-2.9:
-  extends: .e2e
-  variables:
-    HAPROXY_VERSION: "2.9"
-HAProxy-3.0:
-  extends: .e2e
-  variables:
-    HAPROXY_VERSION: "3.0"
-HAProxy-3.1:
-  extends: .e2e
-  variables:
-    HAPROXY_VERSION: "3.1"
-HAProxy-3.2:
-  extends: .e2e
-  variables:
-    HAPROXY_VERSION: "3.2"
+    HAPROXY_VERSION: "3.3"

.golangci.yml

@@ -1,51 +1,66 @@
----
-linters-settings:
-  golint:
-    min-confidence: 0
-  gocyclo:
-    min-complexity: 42
-  cyclop:
-    max-complexity: 42
-  maligned:
-    suggest-new: true
-  dupl:
-    threshold: 200
-  goconst:
-    min-len: 2
-    min-occurrences: 3
-  gci:
-    prefix: github.com/haproxytech/client-native/v6
-
+version: "2"
 linters:
-  enable-all: true
+  settings:
+    cyclop:
+      max-complexity: 42
+    gocyclo:
+      min-complexity: 42
+  default: all
   disable:
-    - mnd
+    - depguard
+    - dupword
+    - dupl
     - err113
-    - wsl
-    - wrapcheck
-    - godot
-    - varnamelen
-    - nlreturn
-    - ireturn
-    - interfacebloat
-    - forcetypeassert
     - exhaustruct
-    - dupword
-    - nestif
-    - lll
-    - depguard
+    - forcetypeassert
+    - funcorder
     - funlen
-    - gci
     - goconst
+    - godoclint
+    - godot
+    - interfacebloat
+    - ireturn
+    - lll
+    - mnd
+    - nestif
+    - nlreturn
+    - noinlineerr
     - recvcheck
-    - tenv # deprecated
-
-issues:
-  exclude-files:
-    - ".*_test\\.go$"
-  exclude-dirs:
-    - test
-  exclude:
-    - "unused-parameter: parameter 'comment' seems to be unused"
-    - "unused-parameter: parameter 'parts' seems to be unused"
-    - "unused-parameter: parameter 'parserType' seems to be unused"
+    - revive
+    - varnamelen
+    - wrapcheck
+    - wsl
+    - wsl_v5
+  exclusions:
+    generated: lax
+    presets:
+      - comments
+      - common-false-positives
+      - legacy
+      - std-error-handling
+    rules:
+      - path: (.+)\.go$
+        text: 'unused-parameter: parameter ''comment'' seems to be unused'
+      - path: (.+)\.go$
+        text: 'unused-parameter: parameter ''parts'' seems to be unused'
+      - path: (.+)\.go$
+        text: 'unused-parameter: parameter ''parserType'' seems to be unused'
+    paths:
+      - .*_test\.go$
+      - test
+      - third_party$
+      - builtin$
+      - examples$
+formatters:
+  enable:
+    - gofmt
+    - gofumpt
+    - goimports
+  exclusions:
+    generated: lax
+    paths:
+      - .*_test\.go$
+      - test
+      - third_party$
+      - builtin$
+      - examples$

Makefile

@@ -1,8 +1,8 @@
 PROJECT_PATH=${PWD}
-DOCKER_HAPROXY_VERSION?=3.2
+DOCKER_HAPROXY_VERSION?=3.3
 SWAGGER_VERSION=v0.32.3
 GO_VERSION:=${shell go mod edit -json | jq -r .Go}
-GOLANGCI_LINT_VERSION=1.64.5
+GOLANGCI_LINT_VERSION=2.8.0
 CHECK_COMMIT=5.2.0
 
 .PHONY: test

cmd/defaults-setter/main.go

@@ -28,9 +28,9 @@ type Definition struct {
 
 // Property represents a property of a model in the OpenAPI specification
 type Property struct {
-	Default interface{} `yaml:"default"`
-	Type    string      `yaml:"type"`
-	GoType  string      `yaml:"x-go-name"` //nolint:tagliatelle
+	Default any    `yaml:"default"`
+	Type    string `yaml:"type"`
+	GoType  string `yaml:"x-go-name"` //nolint:tagliatelle
 }
 
 type FieldInfo struct {
@@ -288,7 +288,7 @@ func insertCode(filepath, structName string, assignments []string, hasPointers b
 			}
 		}
 
-		var newAssignments []string
+		var newAssignments []string //nolint: prealloc
 		newAssignments = append(newAssignments, assignments...)
 		newAssignments = append(newAssignments, "")
 

cmd/gitlab-mr-checker/main.go

@@ -90,8 +90,9 @@ type MergeRequest struct {
 
 var baseURL string //nolint:gochecknoglobals
 
-const LABEL_COLOR = "#8fbc8f" //nolint:stylecheck,revive
+const LABEL_COLOR = "#8fbc8f" //nolint:revive
 
+//nolint:modernize,perfsprint
 func main() {
 	_ = godotenv.Overload()
 	fmt.Print(hello) //nolint:forbidigo
@@ -125,7 +126,7 @@ func main() {
 		slog.Error(err.Error())
 		os.Exit(1)
 	}
-	var versions []*semver.Version //nolint:prealloc
+	var versions []*semver.Version
 	for _, r := range docs {
 		v, err := semver.NewVersion(r)
 		if err != nil {
@@ -142,18 +143,18 @@ func main() {
 
 	gitlabToken := os.Getenv("GITLAB_TOKEN")
 
-	CI_MERGE_REQUEST_IID_STR := os.Getenv("CI_MERGE_REQUEST_IID") //nolint:stylecheck,revive
+	CI_MERGE_REQUEST_IID_STR := os.Getenv("CI_MERGE_REQUEST_IID") //nolint:revive
 	if CI_MERGE_REQUEST_IID_STR == "" {
 		slog.Error("CI_MERGE_REQUEST_IID not set")
 		os.Exit(1)
 	}
-	CI_MERGE_REQUEST_IID, err := strconv.Atoi(CI_MERGE_REQUEST_IID_STR) //nolint:stylecheck,revive
+	CI_MERGE_REQUEST_IID, err := strconv.Atoi(CI_MERGE_REQUEST_IID_STR) //nolint:revive
 	if err != nil {
 		slog.Error(err.Error())
 		os.Exit(1)
 	}
 
-	CI_PROJECT_ID := os.Getenv("CI_PROJECT_ID") //nolint:stylecheck,revive
+	CI_PROJECT_ID := os.Getenv("CI_PROJECT_ID") //nolint:revive
 	if CI_PROJECT_ID == "" {
 		slog.Error("CI_PROJECT_ID not set")
 		os.Exit(1)
@@ -204,7 +205,7 @@ func main() {
 
 func startThreadOnMergeRequest(baseURL, token, projectID string, mergeRequestIID int, threadBody string) {
 	client := &http.Client{}
-	threadData := map[string]interface{}{
+	threadData := map[string]any{
 		"body": threadBody,
 	}
 	threadDataBytes, err := json.Marshal(threadData)

cmd/kubebuilder_marker_generator/main.go

@@ -58,12 +58,12 @@ func generate(fileName string) error { //nolint:gocognit
 									// We must keep empty strings:
 									// For example in Globals HttpclientSslVerify: // Enum: [ none required]
 									// from swagger: enum: ["", "none", "required"]
-									for _, enum := range strings.Split(comment, ",") {
+									for enum := range strings.SplitSeq(comment, ",") {
 										enum = strings.Trim(enum, "\"")
 										if enum == "" {
 											newComment += `""`
 										}
-										newComment += enum
+										newComment += enum //nolint: perfsprint
 										newComment += ";"
 									}
 									field.Decorations().Start.Append(newComment)

cmd/server_params_runtime/main.go

@@ -64,7 +64,7 @@ func main() {
 		doNotSendDisabledFields := listEmptyDisabledFields(allEnumFields)
 		doNotSendEnabledFields := listEmtpyEnabledFields(allEnumFields)
 
-		errTmpl = tmpl.Execute(file, map[string]interface{}{
+		errTmpl = tmpl.Execute(file, map[string]any{
 			"PrepareFieldsForRuntimeAddServer": FuncPrepareFieldsForRuntimeAddServer,
 			"DoNotSendDisabledFields":          doNotSendDisabledFields,
 			"DoNotSendDisabledFieldsFunc":      FuncDoNotSendDisabledFields,

cmd/specification/build.go

@@ -17,7 +17,7 @@ import (
 	"gopkg.in/yaml.v3"
 )
 
-var cache map[string]interface{} //nolint:gochecknoglobals
+var cache map[string]any //nolint:gochecknoglobals
 
 func errorExit(msg string) {
 	fmt.Fprintf(os.Stderr, "ERROR: %v\n", msg)
@@ -39,7 +39,7 @@ func expandRef(refValue string, absPath string, prefix string) string {
 	filePath := path.Join(absPath, words[0])
 	keyPath := words[1]
 
-	m, ok := cache[filePath].(map[string]interface{})
+	m, ok := cache[filePath].(map[string]any)
 	if !ok {
 		fileHandle, err := os.Open(filePath)
 		if err != nil {
@@ -48,11 +48,12 @@ func expandRef(refValue string, absPath string, prefix string) string {
 		defer fileHandle.Close()
 
 		fileScanner := bufio.NewScanner(fileHandle)
-		value := ""
+		sb := strings.Builder{}
 		for fileScanner.Scan() {
-			value += fileScanner.Text() + "\n"
+			sb.WriteString(fileScanner.Text())
+			sb.WriteString("\n")
 		}
-
+		value := sb.String()
 		err = yaml.Unmarshal([]byte(value), &m)
 		if err != nil {
 			fmt.Println(refValue)         //nolint:forbidigo
@@ -61,9 +62,9 @@ func expandRef(refValue string, absPath string, prefix string) string {
 		}
 		cache[filePath] = m
 	}
-	retVal := make(map[string]interface{})
+	retVal := make(map[string]any)
 	if m[keyPath[1:]] != nil {
-		retVal = m[keyPath[1:]].(map[string]interface{})
+		retVal = m[keyPath[1:]].(map[string]any)
 	} else {
 		fmt.Println(refValue)       //nolint:forbidigo
 		fmt.Println(keyPath)        //nolint:forbidigo
@@ -83,17 +84,18 @@ func expandRef(refValue string, absPath string, prefix string) string {
 
 	var indentedRetValStr string
 	var indentedLine string
-	for _, line := range strings.Split(retValStr, "\n") {
+	for line := range strings.SplitSeq(retValStr, "\n") {
 		if strings.TrimSpace(line) != "" {
 			indentedLine = prefix + "" + line + "\n"
-			indentedRetValStr += indentedLine
+			indentedRetValStr += indentedLine //nolint: perfsprint
 		}
 	}
 
 	return indentedRetValStr[:len(indentedRetValStr)-1]
 }
 
-func main() { //nolint:gocognit
+//nolint:gocognit,modernize,perfsprint
+func main() {
 	inputFilePtr := flag.String("file", "", "Source file")
 
 	flag.Parse()
@@ -106,7 +108,7 @@ func main() { //nolint:gocognit
 		errorExit("File " + *inputFilePtr + " does not exist")
 	}
 
-	cache = make(map[string]interface{})
+	cache = make(map[string]any)
 
 	absPath := filepath.Dir(*inputFilePtr)
 	fileHandle, err := os.Open(*inputFilePtr)
@@ -166,7 +168,7 @@ func main() { //nolint:gocognit
 
 					b, _ := yaml.Marshal(&ts)
 
-					for _, line := range strings.Split(strings.TrimRight(string(b), "\n"), "\n") {
+					for line := range strings.SplitSeq(strings.TrimRight(string(b), "\n"), "\n") {
 						result.WriteString("  " + line + "\n")
 					}
 					result.WriteString("security:")

cmd/struct_equal_generator/generate.go

@@ -39,7 +39,7 @@ func generateCompareTests(opt generateEqualAndDiffOptions) error {
 			break
 		}
 	}
-	err = tmpl.Execute(opt.FileTest, map[string]interface{}{
+	err = tmpl.Execute(opt.FileTest, map[string]any{
 		"TestType":    []string{"Equal", "Diff"},
 		"Name":        opt.Name,
 		"HasIndex":    hasIndex,

cmd/struct_equal_generator/main.go

@@ -101,7 +101,7 @@ func generate(fileName string, args Args) (string, error) { //nolint:gocognit
 		return packageName, err
 	}
 
-	err = tmpl2.Execute(fileTest, map[string]interface{}{
+	err = tmpl2.Execute(fileTest, map[string]any{
 		"Package": node.Name.String(),
 		"License": args.License,
 	})