chore(*): enable nilerr, unparam linter

.golangci.yaml

@@ -40,22 +40,22 @@ linters:
     # - gomoddirectives
     # - gosec
     - nakedret
-    # - nilerr
+    - nilerr
     # - nilnil
     # - noctx
     - nolintlint
     - prealloc
     - predeclared
     - promlinter
-    # - reassign
+    - reassign
     - rowserrcheck
     - sqlclosecheck
     - tenv
     # - testableexamples
     # - thelper
     - tparallel
     - unconvert
-    # - unparam
+    - unparam
     - usestdlibvars
     - wastedassign
 
@@ -66,4 +66,8 @@ linters-settings:
     ignore-tests: true
 
 issues:
-  fix: true
+  fix: true
+  exclude-rules:
+    - path: _test\.go
+      linters:
+        - unparam

common/logger.go

@@ -57,6 +57,7 @@ func ParseLogLevel(levelStr string) (slog.Level, error) {
 
 func ConfigureLogger() {
 	zerolog.TimeFieldFormat = time.RFC3339Nano
+	//nolint
 	zerolog.ErrorStackMarshaler = pkgerrors.MarshalStack
 
 	protoMarshal := protojson.MarshalOptions{

coordinator/impl/coordinator.go

@@ -443,6 +443,7 @@ func (c *coordinator) handleClusterConfigUpdated() error {
 	return nil
 }
 
+//nolint:unparam
 func (c *coordinator) rebalanceCluster() error {
 	c.Lock()
 	actions := rebalanceCluster(c.ClusterConfig.Servers, c.clusterStatus)

coordinator/impl/k8s_client_test.go

@@ -38,6 +38,7 @@ func K8SResourceVersionSupport(tracker testing.ObjectTracker) testing.ReactionFu
 			objMeta := accessor(action.GetObject())
 			existing, err := tracker.Get(gvr, ns, objMeta.GetName())
 			if err != nil {
+				//nolint:nilerr
 				return false, action.GetObject(), nil
 			}
 			existingObjMeta := accessor(existing)

coordinator/impl/metadata_test.go

@@ -31,13 +31,13 @@ var (
 		return f
 	}()
 	metadataProviders = map[string]func(t *testing.T) MetadataProvider{
-		"memory": func(t *testing.T) MetadataProvider {
+		"memory": func(_ *testing.T) MetadataProvider {
 			return NewMetadataProviderMemory()
 		},
 		"file": func(t *testing.T) MetadataProvider {
 			return NewMetadataProviderFile(filepath.Join(t.TempDir(), "metadata"))
 		},
-		"configmap": func(t *testing.T) MetadataProvider {
+		"configmap": func(_ *testing.T) MetadataProvider {
 			return NewMetadataProviderConfigMap(_fake, "ns", "n")
 		},
 	}

server/wal/wal_rw_segment.go

@@ -148,6 +148,7 @@ func (ms *readWriteSegment) Flush() error {
 	return ms.txnMappedFile.Flush()
 }
 
+//nolint:unparam
 func (ms *readWriteSegment) rebuildIdx() error {
 	// Scan the mapped file and rebuild the index