[release-4.13] test: upgrade to golint 2.10

.github/workflows/static.yaml

@@ -13,10 +13,10 @@ jobs:
                 go-version: ^1.19
             - uses: actions/checkout@master
             - name: Run linter
-              uses: golangci/golangci-lint-action@v6
+              uses: golangci/golangci-lint-action@v7
               with:
-                  version: v1.64
-                  args: -E=gofmt,unused,ineffassign,revive,misspell,copyloopvar,asciicheck,bodyclose,contextcheck --timeout=30m0s
+                  version: v2.10
+                  args: -E=errcheck,govet,unused,ineffassign,staticcheck,revive,misspell,asciicheck,bodyclose,dogsled,durationcheck,errname,forbidigo --timeout=30m0s
     verify-helm:
         name: Verify Helm
         runs-on: ubuntu-latest

.github/workflows/trivy.yaml

@@ -15,7 +15,7 @@ jobs:
       - name: Install go
         uses: actions/setup-go@v6
         with:
-          go-version: 1.24.13
+          go-version: 1.25.7
 
       - name: Build an image from Dockerfile
         run: |

.golangci.yml

@@ -1,12 +1,41 @@
-linters-settings:
-  depguard:
+version: "2"
+
+linters:
+  default: none
+  enable:
+    - staticcheck
+  settings:
+    staticcheck:
+      checks:
+        - "S*"
+    depguard:
+      rules:
+        main:
+          files:
+            - $all
+            - "!$test"
+          allow:
+            - $gostd
+            - k8s.io
+            - sigs.k8s.io
+            - github.com
+  exclusions:
+    generated: lax
+    presets:
+      - comments
+      - std-error-handling
+      - common-false-positives
+      - legacy
     rules:
-      main:
-        files:
-          - $all
-          - "!$test"
-        allow:
-          - $gostd
-          - k8s.io
-          - sigs.k8s.io
-          - github.com
+      # unused-parameter is a new revive rule not present in v1's bundled revive.
+      - linters:
+          - revive
+        text: "^unused-parameter:"
+      # var-naming package name checks are new in v2's revive (v1 only checked variable names).
+      - linters:
+          - revive
+        text: "^var-naming: avoid"
+
+formatters:
+  enable:
+    - gofmt

go.mod

@@ -1,6 +1,6 @@
 module github.com/kubernetes-csi/csi-driver-nfs
 
-go 1.24.0
+go 1.25.0
 
 require (
 	github.com/container-storage-interface/spec v1.11.0

pkg/nfs/utils.go

@@ -201,7 +201,7 @@ func setKeyValueInMap(m map[string]string, key, value string) {
 
 func waitForPathNotExistWithTimeout(path string, timeout time.Duration) error {
 	// Loop until the path no longer exists or the timeout is reached
-	timeoutTime := time.Now().Add(time.Duration(timeout) * time.Second)
+	timeoutTime := time.Now().Add(timeout)
 	for {
 		if _, err := os.Lstat(path); err != nil {
 			if os.IsNotExist(err) {