fix linters

.golangci.yaml

@@ -35,12 +35,60 @@ linters:
     - whitespace
 
 linters-settings:
+  gocritic:
+    enabled-all: true
+    disabled-checks:
+      - commentFormatting
+  godot:
+    scope: all
+  gofumpt:
+    extra-rules: true
+  goconst:
+    min-len: 2
+    min-occurrences: 5
+  golint:
+    min-confidence: 0
+  gomnd:
+    settings:
+      mnd:
+        # don't include the "operation" and "assign"
+        checks: [argument,case,condition,return]
+  govet:
+    # shadow is marked as experimental feature, skip it for now.
+    check-shadowing: false
+    settings:
+      printf:
+        funcs:
+          - (github.com/golangci/golangci-lint/pkg/logutils.Log).Infof
+          - (github.com/golangci/golangci-lint/pkg/logutils.Log).Warnf
+          - (github.com/golangci/golangci-lint/pkg/logutils.Log).Errorf
+          - (github.com/golangci/golangci-lint/pkg/logutils.Log).Fatalf
+  lll:
+    line-length: 200
+  maligned:
+    suggest-new: true
+  misspell:
+    locale: US
+  revive:
+    rules:
+      - name: redefines-builtin-id
+        disabled: true
+
+      # Allow code like:
+      #      Items: binpacking.Items{
+      #        {
+      #        },
+      #      }
+      - name: nested-structs
+        disabled: true
   gci:
     sections:
       - standard
       - default
       - prefix(cloud-proxy)
-
+issues:
+  fix: true
 run:
   skip-dirs:
     - mock
+    - internal/castai

cmd/proxy/main.go

@@ -34,7 +34,7 @@ func main() {
 
 	dialOpts := make([]grpc.DialOption, 0)
 	if cfg.CastAI.DisableGRPCTLS {
-		// ONLY For testing purposes
+		// ONLY For testing purposes.
 		dialOpts = append(dialOpts, grpc.WithTransportCredentials(insecure.NewCredentials()))
 	} else {
 		dialOpts = append(dialOpts, grpc.WithTransportCredentials(credentials.NewTLS(nil)))
@@ -72,11 +72,11 @@ func main() {
 	}(conn)
 
 	ctx := metadata.NewOutgoingContext(context.Background(), metadata.Pairs(
-		"authorization", fmt.Sprintf("Token %s", cfg.CastAI.ApiKey),
+		"authorization", fmt.Sprintf("Token %s", cfg.CastAI.APIKey),
 	))
 
 	client := proxy.New(conn, gcp.New(gcpauth.NewCredentialsSource(), http.DefaultClient), logger,
-		cfg.ClusterID, GetVersion(), cfg.KeepAlive, cfg.KeepAliveTimeout)
+		cfg.GetPodName(), cfg.ClusterID, GetVersion(), cfg.KeepAlive, cfg.KeepAliveTimeout)
 	err = client.Run(ctx)
 	if err != nil {
 		logger.Panicf("Failed to run client: %v", err)
@@ -93,7 +93,7 @@ func setupLogger(cfg config.Config) *logrus.Logger {
 	logger.SetLevel(logrus.Level(cfg.Log.Level))
 	logger.SetReportCaller(true)
 	logger.Formatter = &logrus.TextFormatter{
-		CallerPrettyfier: func(f *runtime.Frame) (function string, file string) {
+		CallerPrettyfier: func(f *runtime.Frame) (function, file string) {
 			filename := path.Base(f.File)
 			return fmt.Sprintf("%s()", f.Function), fmt.Sprintf("%s:%d", filename, f.Line)
 		},