improve

Author: darox

.golangci.yaml

@@ -0,0 +1,29 @@
+run:
+  concurrency: 4
+  timeout: 5m
+  issues-exit-code: 1
+  tests: true
+  modules-download-mode: readonly
+
+linters-settings:
+  govet:
+    enable-all: true
+    disable:
+      - shadow
+      - fieldalignment
+  gosec:
+    includes: []
+
+linters:
+  disable-all: true
+  enable:
+    - err113
+    - gofmt
+    - goimports
+    - govet
+    - ineffassign
+    - misspell
+    - staticcheck
+    - unused
+    - goheader
+    - gosec

Makefile

@@ -6,11 +6,15 @@ DOCKER_TAG := latest
 # Go build flags
 GO_BUILD_FLAGS := -ldflags="-s -w"
 
-.PHONY: all build docker clean
+.PHONY: all build docker clean lint
 
-all: build docker push
+all: lint build docker push
 
-build:
+lint:
+	@echo "Running golangci-lint..."
+	golangci-lint run ./...
+
+build: lint
 	@echo "Building Go binary..."
 	go build $(GO_BUILD_FLAGS) -o $(BINARY_NAME) ./cmd/main.go
 

pkg/iperf/test.go

@@ -2,6 +2,7 @@ package iperf
 
 import (
 	"context"
+	"errors"
 	"fmt"
 	"os"
 	"os/signal"
@@ -16,6 +17,9 @@ import (
 	"k8s.io/client-go/kubernetes"
 )
 
+// At the package level, add this static error:
+var errIperf3ClientPodFailed = errors.New("iperf3 client pod failed")
+
 // TestConfig holds the configuration for the iperf3 test
 type TestConfig struct {
 	Client     *kubernetes.Clientset
@@ -247,7 +251,7 @@ PodCompleted:
 	fmt.Println(string(logs))
 
 	if podPhase == corev1.PodFailed {
-		return fmt.Errorf("iperf3 client pod failed")
+		return fmt.Errorf("iperf3 client execution: %w", errIperf3ClientPodFailed)
 	}
 
 	return nil