Fix goimports violations

.golangci.yml

@@ -18,7 +18,6 @@ linters:
         - lll
 
         # TODO: fixme
-        - goimports
         - gochecknoglobals
         - gochecknoinits
         - unparam

cmd/bank-vaults/init.go

@@ -15,9 +15,10 @@
 package main
 
 import (
-	"github.com/banzaicloud/bank-vaults/pkg/sdk/vault"
 	"github.com/sirupsen/logrus"
 	"github.com/spf13/cobra"
+
+	"github.com/banzaicloud/bank-vaults/pkg/sdk/vault"
 )
 
 const cfgInitRootToken = "init-root-token"

cmd/bank-vaults/metrics.go

@@ -15,11 +15,12 @@
 package main
 
 import (
-	"github.com/banzaicloud/bank-vaults/pkg/sdk/vault"
 	"github.com/gin-gonic/gin"
 	"github.com/prometheus/client_golang/prometheus"
 	"github.com/prometheus/client_golang/prometheus/promhttp"
 	"github.com/sirupsen/logrus"
+
+	"github.com/banzaicloud/bank-vaults/pkg/sdk/vault"
 )
 
 const prometheusNS = "vault"

cmd/bank-vaults/unseal.go

@@ -19,9 +19,10 @@ import (
 	"strings"
 	"time"
 
-	"github.com/banzaicloud/bank-vaults/pkg/sdk/vault"
 	"github.com/sirupsen/logrus"
 	"github.com/spf13/cobra"
+
+	"github.com/banzaicloud/bank-vaults/pkg/sdk/vault"
 )
 
 const cfgUnsealPeriod = "unseal-period"

cmd/bank-vaults/util.go

@@ -17,6 +17,8 @@ package main
 import (
 	"fmt"
 
+	"github.com/spf13/viper"
+
 	"github.com/banzaicloud/bank-vaults/pkg/kv"
 	"github.com/banzaicloud/bank-vaults/pkg/kv/alibabakms"
 	"github.com/banzaicloud/bank-vaults/pkg/kv/alibabaoss"
@@ -31,7 +33,6 @@ import (
 	"github.com/banzaicloud/bank-vaults/pkg/kv/s3"
 	kvvault "github.com/banzaicloud/bank-vaults/pkg/kv/vault"
 	"github.com/banzaicloud/bank-vaults/pkg/sdk/vault"
-	"github.com/spf13/viper"
 )
 
 func vaultConfigForConfig(cfg *viper.Viper) (vault.Config, error) {

cmd/template/main.go

@@ -20,8 +20,9 @@ import (
 	"os"
 	"strings"
 
-	"github.com/banzaicloud/bank-vaults/internal/configuration"
 	log "github.com/sirupsen/logrus"
+
+	"github.com/banzaicloud/bank-vaults/internal/configuration"
 )
 
 type arrayFlags []string

cmd/vault-secrets-webhook/configmap.go

@@ -19,9 +19,10 @@ import (
 	"fmt"
 	"strings"
 
+	corev1 "k8s.io/api/core/v1"
+
 	internal "github.com/banzaicloud/bank-vaults/internal/configuration"
 	"github.com/banzaicloud/bank-vaults/pkg/sdk/vault"
-	corev1 "k8s.io/api/core/v1"
 )
 
 func configMapNeedsMutation(configMap *corev1.ConfigMap) bool {

cmd/vault-secrets-webhook/main.go

@@ -23,8 +23,6 @@ import (
 
 	internal "github.com/banzaicloud/bank-vaults/internal/configuration"
 
-	"github.com/banzaicloud/bank-vaults/cmd/vault-secrets-webhook/registry"
-	"github.com/banzaicloud/bank-vaults/pkg/sdk/vault"
 	vaultapi "github.com/hashicorp/vault/api"
 	"github.com/prometheus/client_golang/prometheus"
 	"github.com/prometheus/client_golang/prometheus/promhttp"
@@ -42,6 +40,9 @@ import (
 	kubeVer "k8s.io/apimachinery/pkg/version"
 	"k8s.io/client-go/kubernetes"
 	kubernetesConfig "sigs.k8s.io/controller-runtime/pkg/client/config"
+
+	"github.com/banzaicloud/bank-vaults/cmd/vault-secrets-webhook/registry"
+	"github.com/banzaicloud/bank-vaults/pkg/sdk/vault"
 )
 
 var vaultAgentConfig = `

cmd/vault-secrets-webhook/main_test.go

@@ -23,9 +23,10 @@ import (
 	"k8s.io/client-go/kubernetes"
 	fake "k8s.io/client-go/kubernetes/fake"
 
+	imagev1 "github.com/opencontainers/image-spec/specs-go/v1"
+
 	"github.com/banzaicloud/bank-vaults/cmd/vault-secrets-webhook/registry"
 	internal "github.com/banzaicloud/bank-vaults/internal/configuration"
-	imagev1 "github.com/opencontainers/image-spec/specs-go/v1"
 )
 
 var vaultConfig = internal.VaultConfig{

cmd/vault-secrets-webhook/secret.go

@@ -21,12 +21,13 @@ import (
 	"fmt"
 	"strings"
 
-	"github.com/banzaicloud/bank-vaults/cmd/vault-secrets-webhook/registry"
-	internal "github.com/banzaicloud/bank-vaults/internal/configuration"
-	"github.com/banzaicloud/bank-vaults/pkg/sdk/vault"
 	dockerTypes "github.com/docker/docker/api/types"
 	"github.com/spf13/cast"
 	corev1 "k8s.io/api/core/v1"
+
+	"github.com/banzaicloud/bank-vaults/cmd/vault-secrets-webhook/registry"
+	internal "github.com/banzaicloud/bank-vaults/internal/configuration"
+	"github.com/banzaicloud/bank-vaults/pkg/sdk/vault"
 )
 
 func secretNeedsMutation(secret *corev1.Secret) bool {

pkg/kv/alibabakms/kms.go

@@ -17,6 +17,7 @@ package alibabakms
 import (
 	"github.com/aliyun/alibaba-cloud-sdk-go/sdk/requests"
 	"github.com/aliyun/alibaba-cloud-sdk-go/services/kms"
+
 	"github.com/banzaicloud/bank-vaults/pkg/kv"
 )
 

pkg/kv/alibabaoss/oss.go

@@ -20,6 +20,7 @@ import (
 	"io/ioutil"
 
 	"github.com/aliyun/aliyun-oss-go-sdk/oss"
+
 	"github.com/banzaicloud/bank-vaults/pkg/kv"
 )
 

pkg/kv/awskms/awskms.go

@@ -20,6 +20,7 @@ import (
 	"github.com/aws/aws-sdk-go/aws"
 	"github.com/aws/aws-sdk-go/aws/session"
 	"github.com/aws/aws-sdk-go/service/kms"
+
 	"github.com/banzaicloud/bank-vaults/pkg/kv"
 )
 

pkg/kv/azurekv/keyvault.go

@@ -22,6 +22,7 @@ import (
 	"github.com/Azure/azure-sdk-for-go/services/keyvault/2016-10-01/keyvault"
 	"github.com/Azure/go-autorest/autorest"
 	"github.com/Azure/go-autorest/autorest/azure"
+
 	"github.com/banzaicloud/bank-vaults/pkg/kv"
 )
 

pkg/kv/gckms/cloudkms.go

@@ -19,9 +19,10 @@ import (
 	"encoding/base64"
 	"fmt"
 
-	"github.com/banzaicloud/bank-vaults/pkg/kv"
 	"golang.org/x/oauth2/google"
 	cloudkms "google.golang.org/api/cloudkms/v1"
+
+	"github.com/banzaicloud/bank-vaults/pkg/kv"
 )
 
 // googleKms is an implementation of the kv.Service interface, that encrypts

pkg/kv/gcs/gcs.go

@@ -20,6 +20,7 @@ import (
 	"io/ioutil"
 
 	"cloud.google.com/go/storage"
+
 	"github.com/banzaicloud/bank-vaults/pkg/kv"
 )
 

pkg/kv/k8s/k8s.go

@@ -19,13 +19,14 @@ import (
 	"fmt"
 	"os"
 
-	"github.com/banzaicloud/bank-vaults/pkg/kv"
 	v1 "k8s.io/api/core/v1"
 	"k8s.io/apimachinery/pkg/api/errors"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/client-go/kubernetes"
 	"k8s.io/client-go/rest"
 	"k8s.io/client-go/tools/clientcmd"
+
+	"github.com/banzaicloud/bank-vaults/pkg/kv"
 )
 
 // EnvK8SOwnerReference holds the environment variable name for passing in K8S owner refs

pkg/kv/multi/multi.go

@@ -16,8 +16,9 @@ package multi
 
 import (
 	"emperror.dev/errors"
-	"github.com/banzaicloud/bank-vaults/pkg/kv"
 	"github.com/sirupsen/logrus"
+
+	"github.com/banzaicloud/bank-vaults/pkg/kv"
 )
 
 type multi struct {

pkg/kv/vault/vault.go

@@ -18,9 +18,10 @@ import (
 	"encoding/base64"
 	"fmt"
 
+	"github.com/spf13/cast"
+
 	"github.com/banzaicloud/bank-vaults/pkg/kv"
 	"github.com/banzaicloud/bank-vaults/pkg/sdk/vault"
-	"github.com/spf13/cast"
 )
 
 type VaultStorage struct {