Fix lint problems

Signed-off-by: Stavros Foteinopoulos <stafot@gmail.com>

Author: stafot

Makefile

@@ -66,7 +66,7 @@ GOIMPORTS_VER := master
 GOIMPORTS_BIN := goimports
 GOIMPORTS := $(TOOLS_BIN_DIR)/$(GOIMPORTS_BIN)
 
-GOLANGCILINT_VER := v1.64.5
+GOLANGCILINT_VER := v2.0.2
 GOLANGCILINT_BIN := golangci-lint
 GOLANGCILINT := $(TOOLS_BIN_DIR)/$(GOLANGCILINT_BIN)
 
@@ -96,12 +96,12 @@ check-style: govet lint goformat goimports
 ## Runs lint against all packages.
 lint: $(GOLANGCILINT)
 	@echo Running golangci-lint
-	$(GOLANGCILINT) run
+	$(GOLANGCILINT) run --config .golangci.yml
 
 ## Runs lint against all packages for changes only
 lint-changes: $(GOLANGCILINT)
 	@echo Running golangci-lint over changes only
-	$(GOLANGCILINT) run -n
+	$(GOLANGCILINT) run -n --config .golangci.yml
 
 ## Runs govet against all packages.
 .PHONY: vet

cmd/cloud/clicontext/clicontext.go

@@ -121,7 +121,7 @@ func WriteContexts(contexts *Contexts) error {
 	if err != nil {
 		return err
 	}
-	defer file.Close()
+	defer func() { _ = file.Close() }()
 
 	encoder := json.NewEncoder(file)
 	err = encoder.Encode(contexts)

cmd/cloud/contexts.go

@@ -2,6 +2,7 @@ package main
 
 import (
 	"fmt"
+	log "github.com/sirupsen/logrus"
 	"reflect"
 
 	"github.com/mattermost/mattermost-cloud/cmd/cloud/clicontext"
@@ -90,7 +91,9 @@ type updateContextFlags struct {
 func (f *updateContextFlags) addFlags(command *cobra.Command) {
 	f.createContextFlags.addFlags(command)
 	command.Flags().StringVar(&f.Context, "context", "", "Name of the context to update")
-	command.MarkFlagRequired("context")
+	if err := command.MarkFlagRequired("context"); err != nil {
+		log.Fatalf("failed to mark flag required: %v", err)
+	}
 	command.Flags().BoolVar(&f.ClearAuth, "clear-auth", false, "Turns off authentication for this context")
 	command.Flags().BoolVar(&f.ConfirmationRequired, "confirmation-required", false, "Require confirmation for commands run in this context")
 }
@@ -244,8 +247,9 @@ func newCmdContextCreate() *cobra.Command {
 	}
 
 	flags.addFlags(cmd)
-	cmd.MarkFlagRequired("server-url")
-
+	if err := cmd.MarkFlagRequired("server-url"); err != nil {
+		log.Fatalf("failed to mark flag required: %v", err)
+	}
 	return cmd
 }
 
@@ -255,7 +259,9 @@ type deleteContextFlags struct {
 
 func (f *deleteContextFlags) addFlags(command *cobra.Command) {
 	command.Flags().StringVar(&f.contextName, "context", "", "Name of the context to delete")
-	command.MarkFlagRequired("context")
+	if err := command.MarkFlagRequired("context"); err != nil {
+		log.Fatalf("failed to mark flag required: %v", err)
+	}
 }
 
 func newCmdContextDelete() *cobra.Command {

cmd/cloud/installation.go

@@ -309,7 +309,7 @@ func newCmdInstallationUpdateDeletion() *cobra.Command {
 			client := createClient(command.Context(), flags.clusterFlags)
 
 			request := &model.PatchInstallationDeletionRequest{}
-			if flags.installationDeletionPatchRequestOptionsChanged.futureDeletionTimeChanged {
+			if flags.futureDeletionTimeChanged {
 				newExpiryTimeMillis := model.GetMillisAtTime(time.Now().Add(flags.futureDeletionTime))
 				request.DeletionPendingExpiry = &newExpiryTimeMillis
 			}

cmd/cloud/installation_flag.go

@@ -186,7 +186,10 @@ func (flags *installationUpdateFlags) GetPatchInstallationRequest() (*model.Patc
 
 	if flags.allowedIPRangesChanged {
 		allowedIPRanges := &model.AllowedIPRanges{}
-		allowedIPRanges.FromJSONString(flags.allowedIPRanges)
+		_, jsonErr := allowedIPRanges.FromJSONString(flags.allowedIPRanges)
+		if jsonErr != nil {
+			return nil, jsonErr
+		}
 		allowedIPRanges, err := allowedIPRanges.FromJSONString(flags.allowedIPRanges)
 		if err != nil {
 			return nil, err

cmd/cloud/logger_hooks.go

@@ -82,7 +82,11 @@ func (hook *ClusterLoggerHook) fileWrite(entry *logrus.Entry, kind string) error
 		log.Println("failed to open logfile:", path, err)
 		return err
 	}
-	defer fd.Close()
+	defer func() {
+		if err := fd.Close(); err != nil {
+			log.Printf("failed to close fd: %v", err)
+		}
+	}()
 
 	// use our formatter instead of entry.String()
 	msg, err := hook.formatter.Format(entry)
@@ -91,7 +95,10 @@ func (hook *ClusterLoggerHook) fileWrite(entry *logrus.Entry, kind string) error
 		log.Println("failed to generate string for entry:", err)
 		return err
 	}
-	fd.Write(msg)
+	if _, err := fd.Write(msg); err != nil {
+		log.Println("failed to write to logfile:", err)
+		return err
+	}
 	return nil
 }
 

cmd/cloud/server.go

@@ -413,7 +413,11 @@ func executeServerCmd(flags serverFlags) error {
 	}
 
 	standardSupervisor := supervisor.NewScheduler(multiDoer, time.Duration(flags.poll)*time.Second, logger)
-	defer standardSupervisor.Close()
+	defer func() {
+		if err := standardSupervisor.Close(); err != nil {
+			logrus.WithError(err).Error("failed to close standardSupervisor")
+		}
+	}()
 
 	if flags.slowPoll == 0 {
 		logger.WithField("slow-poll", flags.slowPoll).Info("Slow scheduler is disabled")
@@ -422,7 +426,11 @@ func executeServerCmd(flags serverFlags) error {
 		var slowMultiDoer supervisor.MultiDoer
 		slowMultiDoer = append(slowMultiDoer, supervisor.NewInstallationDeletionSupervisor(instanceID, flags.installationDeletionPendingTime, flags.installationDeletionMaxUpdating, sqlStore, eventsProducer, logger))
 		slowSupervisor := supervisor.NewScheduler(slowMultiDoer, time.Duration(flags.slowPoll)*time.Second, logger)
-		defer slowSupervisor.Close()
+		defer func() {
+			if err := slowSupervisor.Close(); err != nil {
+				logrus.WithError(err).Error("failed to close slowSupervisor")
+			}
+		}()
 	}
 
 	metricsRouter := mux.NewRouter()

cmd/provisioner-code-gen/main.go

@@ -43,8 +43,13 @@ func newRootCmd() *cobra.Command {
 
 // Binds all flags as viper values
 func bindFlags(cmd *cobra.Command) {
-	viper.BindPFlags(cmd.PersistentFlags())
-	viper.BindPFlags(cmd.Flags())
+	if err := viper.BindPFlags(cmd.PersistentFlags()); err != nil {
+		logrus.WithError(err).Fatal("failed to bind persistent flags")
+	}
+	if err := viper.BindPFlags(cmd.Flags()); err != nil {
+		logrus.WithError(err).Fatal("failed to bind command flags")
+	}
+
 	for _, c := range cmd.Commands() {
 		bindFlags(c)
 	}

cmd/tools/cloudburst/cloudburst.go

@@ -345,7 +345,9 @@ func (b *Blaster) cleanupInstallations(installations map[string]*cloud.Installat
 				fallthrough
 			case cloud.InstallationStateDeletionInProgress:
 			default:
-				b.client.DeleteInstallation(fetched.ID)
+				if err := b.client.DeleteInstallation(fetched.ID); err != nil {
+					log.WithError(err).Error("failed to delete installation")
+				}
 			}
 		}
 	}

cmd/tools/ctest/test.go

@@ -7,6 +7,7 @@ package main
 import (
 	"encoding/json"
 	"fmt"
+	log "github.com/sirupsen/logrus"
 	"io"
 	"net/http"
 
@@ -36,7 +37,11 @@ func runInstallationLifecycleTest(request *model.CreateInstallationRequest, clie
 	if err != nil {
 		return errors.Wrap(err, "failed to run enhanced ping test")
 	}
-	defer resp.Body.Close()
+	defer func() {
+		if err := resp.Body.Close(); err != nil {
+			log.WithError(err).Error("failed to close resp.Body")
+		}
+	}()
 
 	if resp.StatusCode != http.StatusOK {
 		var body []byte