This is an automated email from the ASF dual-hosted git repository.

nferraro pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel-k.git

commit f9d78ffaaa2cc35c19ee9265af02cb48749a1237
Author: nferraro <ni.ferr...@gmail.com>
AuthorDate: Fri Jan 4 17:29:19 2019 +0100

    Fix #237: fix linter
---
 .golangci.yml                                      |  2 +-
 cmd/manager/main.go                                |  4 +--
 pkg/apis/camel/v1alpha1/common_types.go            |  2 +-
 pkg/apis/camel/v1alpha1/common_types_support.go    |  1 +
 .../camel/v1alpha1/integration_types_support.go    |  5 +--
 .../camel/v1alpha1/integrationplatform_types.go    |  2 --
 .../v1alpha1/integrationplatform_types_support.go  |  5 +--
 pkg/builder/builder.go                             |  3 +-
 pkg/builder/builder_types.go                       | 10 +++---
 pkg/builder/s2i/publisher.go                       |  2 +-
 pkg/client/client.go                               | 15 ++++----
 pkg/cmd/delete.go                                  |  2 +-
 pkg/cmd/install.go                                 |  2 +-
 pkg/cmd/log.go                                     |  2 +-
 pkg/cmd/reset.go                                   |  5 ++-
 pkg/cmd/root.go                                    | 40 ++++++++++++----------
 pkg/cmd/run.go                                     | 11 +++---
 pkg/cmd/util.go                                    |  5 ++-
 pkg/controller/integration/action.go               |  1 +
 pkg/controller/integration/deploy.go               |  1 +
 pkg/controller/integration/initialize.go           |  1 +
 .../integration/integration_controller.go          |  2 +-
 pkg/controller/integration/monitor.go              |  1 +
 pkg/controller/integration/util.go                 |  3 +-
 pkg/controller/integrationcontext/action.go        |  3 +-
 pkg/controller/integrationcontext/initialize.go    |  1 +
 pkg/controller/integrationcontext/monitor.go       |  1 +
 pkg/controller/integrationplatform/action.go       |  1 +
 pkg/controller/integrationplatform/create.go       |  1 +
 pkg/controller/integrationplatform/initialize.go   |  1 +
 .../integrationplatform_controller.go              |  3 +-
 pkg/controller/integrationplatform/start.go        |  1 +
 pkg/install/cluster.go                             |  5 +--
 pkg/install/common.go                              |  3 +-
 pkg/platform/platform.go                           |  2 +-
 pkg/trait/catalog.go                               |  5 ++-
 pkg/trait/environment_test.go                      |  2 +-
 pkg/trait/knative.go                               |  7 ++--
 pkg/trait/knative_test.go                          |  2 +-
 pkg/trait/trait.go                                 |  3 +-
 pkg/trait/trait_test.go                            |  2 +-
 pkg/trait/types.go                                 |  1 +
 pkg/util/knative/knative.go                        |  1 +
 pkg/util/kubernetes/customclient/customclient.go   |  6 ++--
 pkg/util/kubernetes/loader.go                      |  1 +
 pkg/util/kubernetes/replace.go                     |  1 +
 pkg/util/kubernetes/util.go                        |  2 +-
 pkg/util/log/annotation_scraper.go                 |  2 +-
 pkg/util/log/pod_scraper.go                        |  5 ++-
 pkg/util/log/util.go                               |  2 +-
 pkg/util/minishift/minishift.go                    |  4 +--
 pkg/util/watch/watch.go                            |  1 +
 52 files changed, 104 insertions(+), 92 deletions(-)

diff --git a/.golangci.yml b/.golangci.yml
index d4ece64..a67fb8d 100644
--- a/.golangci.yml
+++ b/.golangci.yml
@@ -1,6 +1,6 @@
 linters-settings:
   lll:
-    line-length: 150
+    line-length: 170
 linters:
   enable-all: true
   disable:
diff --git a/cmd/manager/main.go b/cmd/manager/main.go
index 41dd81f..19aed31 100644
--- a/cmd/manager/main.go
+++ b/cmd/manager/main.go
@@ -75,7 +75,7 @@ func main() {
        }
 
        // Become the leader before proceeding
-       leader.Become(context.TODO(), "camel-k-lock")
+       leader.Become(context.TODO(), "camel-k-lock") // nolint: errcheck
 
        r := ready.NewFileReady()
        err = r.Set()
@@ -83,7 +83,7 @@ func main() {
                log.Error(err, "")
                os.Exit(1)
        }
-       defer r.Unset()
+       defer r.Unset() // nolint: errcheck
 
        // Create a new Cmd to provide shared dependencies and start components
        mgr, err := manager.New(cfg, manager.Options{Namespace: namespace})
diff --git a/pkg/apis/camel/v1alpha1/common_types.go 
b/pkg/apis/camel/v1alpha1/common_types.go
index 669239a..2d0aac7 100644
--- a/pkg/apis/camel/v1alpha1/common_types.go
+++ b/pkg/apis/camel/v1alpha1/common_types.go
@@ -42,4 +42,4 @@ type Flows []Flow
 type Step struct {
        Kind string `json:"kind"`
        URI  string `json:"uri"`
-}
\ No newline at end of file
+}
diff --git a/pkg/apis/camel/v1alpha1/common_types_support.go 
b/pkg/apis/camel/v1alpha1/common_types_support.go
index 0dd8844..84ff4f0 100644
--- a/pkg/apis/camel/v1alpha1/common_types_support.go
+++ b/pkg/apis/camel/v1alpha1/common_types_support.go
@@ -19,6 +19,7 @@ package v1alpha1
 
 import (
        "fmt"
+
        "gopkg.in/yaml.v2"
 )
 
diff --git a/pkg/apis/camel/v1alpha1/integration_types_support.go 
b/pkg/apis/camel/v1alpha1/integration_types_support.go
index 4400394..bdade9f 100644
--- a/pkg/apis/camel/v1alpha1/integration_types_support.go
+++ b/pkg/apis/camel/v1alpha1/integration_types_support.go
@@ -18,10 +18,11 @@ limitations under the License.
 package v1alpha1
 
 import (
+       "strings"
+
        "github.com/apache/camel-k/pkg/util"
        "github.com/mitchellh/mapstructure"
        metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
-       "strings"
 )
 
 // NewIntegrationList --
@@ -121,4 +122,4 @@ func (s SourceSpec) InferLanguage() Language {
                }
        }
        return ""
-}
\ No newline at end of file
+}
diff --git a/pkg/apis/camel/v1alpha1/integrationplatform_types.go 
b/pkg/apis/camel/v1alpha1/integrationplatform_types.go
index 88fd5df..d13ea63 100644
--- a/pkg/apis/camel/v1alpha1/integrationplatform_types.go
+++ b/pkg/apis/camel/v1alpha1/integrationplatform_types.go
@@ -18,7 +18,6 @@ type IntegrationPlatformStatus struct {
        Phase IntegrationPlatformPhase `json:"phase,omitempty"`
 }
 
-
 // +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object
 
 // IntegrationPlatform is the Schema for the integrationplatforms API
@@ -86,7 +85,6 @@ const (
        IntegrationPlatformBuildPublishStrategyKaniko = "Kaniko"
 )
 
-
 // IntegrationPlatformPhase --
 type IntegrationPlatformPhase string
 
diff --git a/pkg/apis/camel/v1alpha1/integrationplatform_types_support.go 
b/pkg/apis/camel/v1alpha1/integrationplatform_types_support.go
index 08f2ab3..11800d3 100644
--- a/pkg/apis/camel/v1alpha1/integrationplatform_types_support.go
+++ b/pkg/apis/camel/v1alpha1/integrationplatform_types_support.go
@@ -18,8 +18,9 @@ limitations under the License.
 package v1alpha1
 
 import (
-       metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
        "strings"
+
+       metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 )
 
 // NewIntegrationPlatformList --
@@ -54,4 +55,4 @@ func TraitProfileByName(name string) TraitProfile {
                }
        }
        return ""
-}
\ No newline at end of file
+}
diff --git a/pkg/builder/builder.go b/pkg/builder/builder.go
index 2760f11..9becccb 100644
--- a/pkg/builder/builder.go
+++ b/pkg/builder/builder.go
@@ -22,14 +22,13 @@ import (
        "errors"
        "io/ioutil"
        "os"
-       "github.com/apache/camel-k/pkg/client"
        "sort"
        "sync"
        "sync/atomic"
        "time"
 
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
-
+       "github.com/apache/camel-k/pkg/client"
        "github.com/sirupsen/logrus"
 )
 
diff --git a/pkg/builder/builder_types.go b/pkg/builder/builder_types.go
index 662d068..6666bda 100644
--- a/pkg/builder/builder_types.go
+++ b/pkg/builder/builder_types.go
@@ -21,14 +21,12 @@ import (
        "context"
        "fmt"
        "math"
-       "github.com/apache/camel-k/pkg/client"
        "time"
 
-       "k8s.io/apimachinery/pkg/apis/meta/v1"
-
-       "github.com/apache/camel-k/pkg/util/maven"
-
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
+       "github.com/apache/camel-k/pkg/client"
+       "github.com/apache/camel-k/pkg/util/maven"
+       "k8s.io/apimachinery/pkg/apis/meta/v1"
 )
 
 const (
@@ -137,7 +135,7 @@ type Result struct {
 
 // Context --
 type Context struct {
-       C                 context.Context
+       C context.Context
        client.Client
        Request           Request
        Image             string
diff --git a/pkg/builder/s2i/publisher.go b/pkg/builder/s2i/publisher.go
index 757d1b3..b02a7ef 100644
--- a/pkg/builder/s2i/publisher.go
+++ b/pkg/builder/s2i/publisher.go
@@ -135,7 +135,7 @@ func Publisher(ctx *builder.Context) error {
        }
 
        ocbuild := buildv1.Build{}
-       json.Unmarshal(data, &ocbuild)
+       err = json.Unmarshal(data, &ocbuild)
        if err != nil {
                return errors.Wrap(err, "cannot unmarshal instantiated binary 
response")
        }
diff --git a/pkg/client/client.go b/pkg/client/client.go
index 756ac28..7a0c73a 100644
--- a/pkg/client/client.go
+++ b/pkg/client/client.go
@@ -18,19 +18,20 @@ limitations under the License.
 package client
 
 import (
+       "io/ioutil"
+       "os"
+       "os/user"
+       "path/filepath"
+
        "github.com/apache/camel-k/pkg/apis"
        "github.com/operator-framework/operator-sdk/pkg/k8sutil"
        "github.com/pkg/errors"
-       "io/ioutil"
        "k8s.io/apimachinery/pkg/runtime"
        "k8s.io/apimachinery/pkg/runtime/schema"
        "k8s.io/client-go/kubernetes"
        "k8s.io/client-go/tools/clientcmd"
        clientcmdapi "k8s.io/client-go/tools/clientcmd/api"
        clientcmdlatest "k8s.io/client-go/tools/clientcmd/api/latest"
-       "os"
-       "os/user"
-       "path/filepath"
        controller "sigs.k8s.io/controller-runtime/pkg/client"
        "sigs.k8s.io/controller-runtime/pkg/client/config"
        "sigs.k8s.io/controller-runtime/pkg/manager"
@@ -73,6 +74,9 @@ func NewOutOfClusterClient(kubeconfig string) (Client, error) 
{
 
        // Create a new Cmd to provide shared dependencies and start components
        mgr, err := manager.New(cfg, options)
+       if err != nil {
+               return nil, err
+       }
 
        // Setup Scheme for all resources
        if err := apis.AddToScheme(mgr.GetScheme()); err != nil {
@@ -116,12 +120,11 @@ func FromManager(manager manager.Manager) (Client, error) 
{
 }
 
 // init initialize the k8s client for usage outside the cluster
-func initialize(kubeconfig string) error {
+func initialize(kubeconfig string) {
        if kubeconfig == "" {
                kubeconfig = getDefaultKubeConfigFile()
        }
        os.Setenv(k8sutil.KubeConfigEnvVar, kubeconfig)
-       return nil
 }
 
 func getDefaultKubeConfigFile() string {
diff --git a/pkg/cmd/delete.go b/pkg/cmd/delete.go
index 22a9280..15378d7 100644
--- a/pkg/cmd/delete.go
+++ b/pkg/cmd/delete.go
@@ -22,11 +22,11 @@ import (
        "fmt"
        "strconv"
 
-       k8sclient "sigs.k8s.io/controller-runtime/pkg/client"
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        "github.com/spf13/cobra"
        k8errors "k8s.io/apimachinery/pkg/api/errors"
        metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
+       k8sclient "sigs.k8s.io/controller-runtime/pkg/client"
 )
 
 // newCmdDelete --
diff --git a/pkg/cmd/install.go b/pkg/cmd/install.go
index fceebdf..c097cb6 100644
--- a/pkg/cmd/install.go
+++ b/pkg/cmd/install.go
@@ -19,13 +19,13 @@ package cmd
 
 import (
        "fmt"
+       "strings"
 
        "github.com/apache/camel-k/pkg/install"
        "github.com/apache/camel-k/pkg/util/kubernetes"
        "github.com/pkg/errors"
        "github.com/spf13/cobra"
        k8serrors "k8s.io/apimachinery/pkg/api/errors"
-       "strings"
 )
 
 func newCmdInstall(rootCmdOptions *RootCmdOptions) *cobra.Command {
diff --git a/pkg/cmd/log.go b/pkg/cmd/log.go
index 8e92ebc..b847c49 100644
--- a/pkg/cmd/log.go
+++ b/pkg/cmd/log.go
@@ -19,11 +19,11 @@ package cmd
 
 import (
        "fmt"
-       metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        "github.com/apache/camel-k/pkg/util/log"
        "github.com/spf13/cobra"
+       metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
        k8sclient "sigs.k8s.io/controller-runtime/pkg/client"
 )
 
diff --git a/pkg/cmd/reset.go b/pkg/cmd/reset.go
index 652964f..6946f7e 100644
--- a/pkg/cmd/reset.go
+++ b/pkg/cmd/reset.go
@@ -19,13 +19,12 @@ package cmd
 
 import (
        "fmt"
-       "github.com/apache/camel-k/pkg/client"
-
-       k8sclient "sigs.k8s.io/controller-runtime/pkg/client"
 
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
+       "github.com/apache/camel-k/pkg/client"
        "github.com/pkg/errors"
        "github.com/spf13/cobra"
+       k8sclient "sigs.k8s.io/controller-runtime/pkg/client"
 )
 
 func newCmdReset(rootCmdOptions *RootCmdOptions) *cobra.Command {
diff --git a/pkg/cmd/root.go b/pkg/cmd/root.go
index 584489e..c718d94 100644
--- a/pkg/cmd/root.go
+++ b/pkg/cmd/root.go
@@ -19,15 +19,15 @@ package cmd
 
 import (
        "context"
+
        "github.com/apache/camel-k/pkg/client"
        "github.com/pkg/errors"
        "github.com/spf13/cobra"
 )
 
-const kamelCommandLongDescription = `
-Long:  "Apache Camel K (a.k.a. Kamel) is a lightweight integration framework
+const kamelCommandLongDescription = `Apache Camel K is a lightweight 
integration framework
 built from Apache Camel that runs natively on Kubernetes and is
-specifically designed for serverless and microservice architectures.",,        
+specifically designed for serverless and microservice architectures.
 `
 
 // RootCmdOptions --
@@ -44,23 +44,11 @@ func NewKamelCommand(ctx context.Context) (*cobra.Command, 
error) {
                Context: ctx,
        }
        var cmd = cobra.Command{
-               Use:   "kamel",
-               Short: "Kamel is a awesome client tool for running Apache Camel 
integrations natively on Kubernetes",
-               Long:  kamelCommandLongDescription,
                BashCompletionFunction: bashCompletionFunction,
-               PersistentPreRunE: func(cmd *cobra.Command, args []string) 
error {
-                       if options.Namespace == "" {
-                               current, err := 
client.GetCurrentNamespace(options.KubeConfig)
-                               if err != nil {
-                                       return errors.Wrap(err, "cannot get 
current namespace")
-                               }
-                               err = cmd.Flag("namespace").Value.Set(current)
-                               if err != nil {
-                                       return err
-                               }
-                       }
-                       return nil
-               },
+               PersistentPreRunE:      options.preRun,
+               Use:                    "kamel",
+               Short:                  "Kamel is a awesome client tool for 
running Apache Camel integrations natively on Kubernetes",
+               Long:                   kamelCommandLongDescription,
        }
 
        cmd.PersistentFlags().StringVar(&options.KubeConfig, "config", "", 
"Path to the config file to use for CLI requests")
@@ -79,6 +67,20 @@ func NewKamelCommand(ctx context.Context) (*cobra.Command, 
error) {
        return &cmd, nil
 }
 
+func (command *RootCmdOptions) preRun(cmd *cobra.Command, args []string) error 
{
+       if command.Namespace == "" {
+               current, err := client.GetCurrentNamespace(command.KubeConfig)
+               if err != nil {
+                       return errors.Wrap(err, "cannot get current namespace")
+               }
+               err = cmd.Flag("namespace").Value.Set(current)
+               if err != nil {
+                       return err
+               }
+       }
+       return nil
+}
+
 // GetCmdClient returns a client that can be used from command line tools
 func (command *RootCmdOptions) GetCmdClient() (client.Client, error) {
        // Get the pre-computed client
diff --git a/pkg/cmd/run.go b/pkg/cmd/run.go
index 26def41..f7addb2 100644
--- a/pkg/cmd/run.go
+++ b/pkg/cmd/run.go
@@ -21,7 +21,6 @@ import (
        "bytes"
        "encoding/base64"
        "fmt"
-       "github.com/apache/camel-k/pkg/client"
        "io/ioutil"
        "net/http"
        "os"
@@ -32,19 +31,17 @@ import (
        "strings"
        "syscall"
 
+       "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
+       "github.com/apache/camel-k/pkg/client"
        "github.com/apache/camel-k/pkg/gzip"
-
        "github.com/apache/camel-k/pkg/trait"
        "github.com/apache/camel-k/pkg/util"
-
-       "github.com/pkg/errors"
-       "github.com/sirupsen/logrus"
-
-       "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        "github.com/apache/camel-k/pkg/util/kubernetes"
        "github.com/apache/camel-k/pkg/util/log"
        "github.com/apache/camel-k/pkg/util/sync"
        "github.com/apache/camel-k/pkg/util/watch"
+       "github.com/pkg/errors"
+       "github.com/sirupsen/logrus"
        "github.com/spf13/cobra"
        k8serrors "k8s.io/apimachinery/pkg/api/errors"
        "k8s.io/apimachinery/pkg/apis/meta/v1"
diff --git a/pkg/cmd/util.go b/pkg/cmd/util.go
index 70dd208..d6cb4ee 100644
--- a/pkg/cmd/util.go
+++ b/pkg/cmd/util.go
@@ -19,9 +19,10 @@ package cmd
 
 import (
        "context"
+
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
-       metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
        "github.com/apache/camel-k/pkg/client"
+       metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 )
 
 // DeleteIntegration --
@@ -38,5 +39,3 @@ func DeleteIntegration(ctx context.Context, c client.Client, 
name string, namesp
        }
        return c.Delete(ctx, &integration)
 }
-
-
diff --git a/pkg/controller/integration/action.go 
b/pkg/controller/integration/action.go
index 6a66e06..eebadef 100644
--- a/pkg/controller/integration/action.go
+++ b/pkg/controller/integration/action.go
@@ -19,6 +19,7 @@ package integration
 
 import (
        "context"
+
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        "github.com/apache/camel-k/pkg/client"
 )
diff --git a/pkg/controller/integration/deploy.go 
b/pkg/controller/integration/deploy.go
index 1ea1096..a8ac087 100644
--- a/pkg/controller/integration/deploy.go
+++ b/pkg/controller/integration/deploy.go
@@ -19,6 +19,7 @@ package integration
 
 import (
        "context"
+
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        "github.com/apache/camel-k/pkg/trait"
        "github.com/apache/camel-k/pkg/util/kubernetes"
diff --git a/pkg/controller/integration/initialize.go 
b/pkg/controller/integration/initialize.go
index d99c8c5..aa824af 100644
--- a/pkg/controller/integration/initialize.go
+++ b/pkg/controller/integration/initialize.go
@@ -19,6 +19,7 @@ package integration
 
 import (
        "context"
+
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        "github.com/apache/camel-k/pkg/platform"
        "github.com/apache/camel-k/pkg/trait"
diff --git a/pkg/controller/integration/integration_controller.go 
b/pkg/controller/integration/integration_controller.go
index ddbe9a7..eb8e33a 100644
--- a/pkg/controller/integration/integration_controller.go
+++ b/pkg/controller/integration/integration_controller.go
@@ -2,11 +2,11 @@ package integration
 
 import (
        "context"
-       "github.com/sirupsen/logrus"
        "time"
 
        camelv1alpha1 "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        "github.com/apache/camel-k/pkg/client"
+       "github.com/sirupsen/logrus"
        appsv1 "k8s.io/api/apps/v1"
        "k8s.io/apimachinery/pkg/api/errors"
        "k8s.io/apimachinery/pkg/runtime"
diff --git a/pkg/controller/integration/monitor.go 
b/pkg/controller/integration/monitor.go
index e45cb1b..3f0d6a5 100644
--- a/pkg/controller/integration/monitor.go
+++ b/pkg/controller/integration/monitor.go
@@ -19,6 +19,7 @@ package integration
 
 import (
        "context"
+
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        "github.com/apache/camel-k/pkg/util/digest"
        "github.com/sirupsen/logrus"
diff --git a/pkg/controller/integration/util.go 
b/pkg/controller/integration/util.go
index 2581cdc..9c31ec8 100644
--- a/pkg/controller/integration/util.go
+++ b/pkg/controller/integration/util.go
@@ -20,7 +20,6 @@ package integration
 import (
        "context"
 
-       "github.com/apache/camel-k/pkg/client"
        "github.com/apache/camel-k/pkg/util"
        k8sclient "sigs.k8s.io/controller-runtime/pkg/client"
 
@@ -29,7 +28,7 @@ import (
 )
 
 // LookupContextForIntegration --
-func LookupContextForIntegration(ctx context.Context, c client.Client, 
integration *v1alpha1.Integration) (*v1alpha1.IntegrationContext, error) {
+func LookupContextForIntegration(ctx context.Context, c k8sclient.Reader, 
integration *v1alpha1.Integration) (*v1alpha1.IntegrationContext, error) {
        if integration.Status.Context != "" {
                name := integration.Status.Context
                ictx := v1alpha1.NewIntegrationContext(integration.Namespace, 
name)
diff --git a/pkg/controller/integrationcontext/action.go 
b/pkg/controller/integrationcontext/action.go
index 55e72c8..6d1369a 100644
--- a/pkg/controller/integrationcontext/action.go
+++ b/pkg/controller/integrationcontext/action.go
@@ -19,6 +19,7 @@ package integrationcontext
 
 import (
        "context"
+
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        "github.com/apache/camel-k/pkg/client"
 )
@@ -43,4 +44,4 @@ type baseAction struct {
 
 func (action *baseAction) InjectClient(client client.Client) {
        action.client = client
-}
\ No newline at end of file
+}
diff --git a/pkg/controller/integrationcontext/initialize.go 
b/pkg/controller/integrationcontext/initialize.go
index 0cb363c..20302ac 100644
--- a/pkg/controller/integrationcontext/initialize.go
+++ b/pkg/controller/integrationcontext/initialize.go
@@ -19,6 +19,7 @@ package integrationcontext
 
 import (
        "context"
+
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        "github.com/apache/camel-k/pkg/platform"
        "github.com/apache/camel-k/pkg/util/digest"
diff --git a/pkg/controller/integrationcontext/monitor.go 
b/pkg/controller/integrationcontext/monitor.go
index 3570e33..5265bca 100644
--- a/pkg/controller/integrationcontext/monitor.go
+++ b/pkg/controller/integrationcontext/monitor.go
@@ -19,6 +19,7 @@ package integrationcontext
 
 import (
        "context"
+
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        "github.com/apache/camel-k/pkg/util/digest"
        "github.com/sirupsen/logrus"
diff --git a/pkg/controller/integrationplatform/action.go 
b/pkg/controller/integrationplatform/action.go
index b332a74..fccc36c 100644
--- a/pkg/controller/integrationplatform/action.go
+++ b/pkg/controller/integrationplatform/action.go
@@ -19,6 +19,7 @@ package integrationplatform
 
 import (
        "context"
+
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        "github.com/apache/camel-k/pkg/client"
 )
diff --git a/pkg/controller/integrationplatform/create.go 
b/pkg/controller/integrationplatform/create.go
index 36c8c7b..82cfcd3 100644
--- a/pkg/controller/integrationplatform/create.go
+++ b/pkg/controller/integrationplatform/create.go
@@ -19,6 +19,7 @@ package integrationplatform
 
 import (
        "context"
+
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        "github.com/apache/camel-k/pkg/install"
        "github.com/sirupsen/logrus"
diff --git a/pkg/controller/integrationplatform/initialize.go 
b/pkg/controller/integrationplatform/initialize.go
index 79fff73..a508a2c 100644
--- a/pkg/controller/integrationplatform/initialize.go
+++ b/pkg/controller/integrationplatform/initialize.go
@@ -20,6 +20,7 @@ package integrationplatform
 import (
        "context"
        "errors"
+
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        platformutils "github.com/apache/camel-k/pkg/platform"
        "github.com/apache/camel-k/pkg/util/openshift"
diff --git 
a/pkg/controller/integrationplatform/integrationplatform_controller.go 
b/pkg/controller/integrationplatform/integrationplatform_controller.go
index 2a943d4..0859f95 100644
--- a/pkg/controller/integrationplatform/integrationplatform_controller.go
+++ b/pkg/controller/integrationplatform/integrationplatform_controller.go
@@ -2,6 +2,8 @@ package integrationplatform
 
 import (
        "context"
+       "time"
+
        camelv1alpha1 "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        "github.com/apache/camel-k/pkg/client"
        "github.com/sirupsen/logrus"
@@ -13,7 +15,6 @@ import (
        "sigs.k8s.io/controller-runtime/pkg/reconcile"
        logf "sigs.k8s.io/controller-runtime/pkg/runtime/log"
        "sigs.k8s.io/controller-runtime/pkg/source"
-       "time"
 )
 
 var log = logf.Log.WithName("controller_integrationplatform")
diff --git a/pkg/controller/integrationplatform/start.go 
b/pkg/controller/integrationplatform/start.go
index 536c9d0..6d69bf6 100644
--- a/pkg/controller/integrationplatform/start.go
+++ b/pkg/controller/integrationplatform/start.go
@@ -19,6 +19,7 @@ package integrationplatform
 
 import (
        "context"
+
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        "github.com/sirupsen/logrus"
        "k8s.io/apimachinery/pkg/labels"
diff --git a/pkg/install/cluster.go b/pkg/install/cluster.go
index 4c78f5c..7adf008 100644
--- a/pkg/install/cluster.go
+++ b/pkg/install/cluster.go
@@ -19,6 +19,7 @@ package install
 
 import (
        "context"
+
        "github.com/apache/camel-k/deploy"
        "github.com/apache/camel-k/pkg/client"
        "github.com/apache/camel-k/pkg/util/kubernetes"
@@ -31,7 +32,7 @@ import (
 )
 
 // SetupClusterwideResources --
-func SetupClusterwideResources(ctx context.Context, c client.Client, ) error {
+func SetupClusterwideResources(ctx context.Context, c client.Client) error {
        return SetupClusterwideResourcesOrCollect(ctx, c, nil)
 }
 
@@ -127,7 +128,7 @@ func installCRD(ctx context.Context, c client.Client, kind 
string, resourceName
 }
 
 // IsClusterRoleInstalled check if cluster role camel-k:edit is installed
-func IsClusterRoleInstalled(ctx context.Context, c client.Client, ) (bool, 
error) {
+func IsClusterRoleInstalled(ctx context.Context, c client.Client) (bool, 
error) {
        clusterRole := v1.ClusterRole{
                TypeMeta: metav1.TypeMeta{
                        Kind:       "ClusterRole",
diff --git a/pkg/install/common.go b/pkg/install/common.go
index fe18638..a2edac4 100644
--- a/pkg/install/common.go
+++ b/pkg/install/common.go
@@ -19,13 +19,14 @@ package install
 
 import (
        "context"
+
        "github.com/apache/camel-k/deploy"
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
+       "github.com/apache/camel-k/pkg/client"
        "github.com/apache/camel-k/pkg/util/kubernetes"
        "k8s.io/apimachinery/pkg/api/errors"
        metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
        "k8s.io/apimachinery/pkg/runtime"
-       "github.com/apache/camel-k/pkg/client"
 )
 
 // Resources installs named resources from the project resource directory
diff --git a/pkg/platform/platform.go b/pkg/platform/platform.go
index df8fab6..48cffcb 100644
--- a/pkg/platform/platform.go
+++ b/pkg/platform/platform.go
@@ -20,10 +20,10 @@ package platform
 import (
        "context"
        "errors"
-       "github.com/apache/camel-k/pkg/client"
 
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        "github.com/apache/camel-k/pkg/builder"
+       "github.com/apache/camel-k/pkg/client"
        k8sclient "sigs.k8s.io/controller-runtime/pkg/client"
 )
 
diff --git a/pkg/trait/catalog.go b/pkg/trait/catalog.go
index facab15..47decb0 100644
--- a/pkg/trait/catalog.go
+++ b/pkg/trait/catalog.go
@@ -20,13 +20,12 @@ package trait
 import (
        "context"
        "reflect"
-       "github.com/apache/camel-k/pkg/client"
        "strings"
 
-       "github.com/sirupsen/logrus"
-
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
+       "github.com/apache/camel-k/pkg/client"
        "github.com/fatih/structs"
+       "github.com/sirupsen/logrus"
 )
 
 // Catalog collects all information about traits in one place
diff --git a/pkg/trait/environment_test.go b/pkg/trait/environment_test.go
index 955eca3..0fd35f1 100644
--- a/pkg/trait/environment_test.go
+++ b/pkg/trait/environment_test.go
@@ -191,4 +191,4 @@ func TestDisabledContainerMetaDataEnvVars(t *testing.T) {
 
 func NewEnvironmentTestCatalog() *Catalog {
        return NewCatalog(context.TODO(), nil)
-}
\ No newline at end of file
+}
diff --git a/pkg/trait/knative.go b/pkg/trait/knative.go
index ac33930..cc6e990 100644
--- a/pkg/trait/knative.go
+++ b/pkg/trait/knative.go
@@ -19,20 +19,17 @@ package trait
 
 import (
        "fmt"
-       "github.com/apache/camel-k/pkg/util/envvar"
-
        "strconv"
        "strings"
 
-       "github.com/pkg/errors"
-
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
-
        knativeapi "github.com/apache/camel-k/pkg/apis/camel/v1alpha1/knative"
        "github.com/apache/camel-k/pkg/metadata"
+       "github.com/apache/camel-k/pkg/util/envvar"
        knativeutil "github.com/apache/camel-k/pkg/util/knative"
        eventing "github.com/knative/eventing/pkg/apis/eventing/v1alpha1"
        serving "github.com/knative/serving/pkg/apis/serving/v1alpha1"
+       "github.com/pkg/errors"
        corev1 "k8s.io/api/core/v1"
        metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
        k8sclient "sigs.k8s.io/controller-runtime/pkg/client"
diff --git a/pkg/trait/knative_test.go b/pkg/trait/knative_test.go
index cb1818d..73406e2 100644
--- a/pkg/trait/knative_test.go
+++ b/pkg/trait/knative_test.go
@@ -135,4 +135,4 @@ func TestKnativeTraitWithCompressedSources(t *testing.T) {
 
 func NewKnativeTestCatalog() *Catalog {
        return NewCatalog(context.TODO(), nil)
-}
\ No newline at end of file
+}
diff --git a/pkg/trait/trait.go b/pkg/trait/trait.go
index df773e4..8c28086 100644
--- a/pkg/trait/trait.go
+++ b/pkg/trait/trait.go
@@ -19,12 +19,13 @@ package trait
 
 import (
        "context"
+
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
+       "github.com/apache/camel-k/pkg/client"
        "github.com/apache/camel-k/pkg/platform"
        "github.com/apache/camel-k/pkg/util/kubernetes"
        "github.com/pkg/errors"
        "k8s.io/api/core/v1"
-       "github.com/apache/camel-k/pkg/client"
 )
 
 // True --
diff --git a/pkg/trait/trait_test.go b/pkg/trait/trait_test.go
index bb206d3..3ea9b7d 100644
--- a/pkg/trait/trait_test.go
+++ b/pkg/trait/trait_test.go
@@ -203,4 +203,4 @@ func createTestEnv(cluster 
v1alpha1.IntegrationPlatformCluster, script string) *
 
 func NewTraitTestCatalog() *Catalog {
        return NewCatalog(context.TODO(), nil)
-}
\ No newline at end of file
+}
diff --git a/pkg/trait/types.go b/pkg/trait/types.go
index 8b48645..b9d892d 100644
--- a/pkg/trait/types.go
+++ b/pkg/trait/types.go
@@ -19,6 +19,7 @@ package trait
 
 import (
        "context"
+
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        "github.com/apache/camel-k/pkg/builder"
        "github.com/apache/camel-k/pkg/client"
diff --git a/pkg/util/knative/knative.go b/pkg/util/knative/knative.go
index 19cd091..7f86ba9 100644
--- a/pkg/util/knative/knative.go
+++ b/pkg/util/knative/knative.go
@@ -19,6 +19,7 @@ package knative
 
 import (
        "context"
+
        "k8s.io/apimachinery/pkg/api/errors"
        "k8s.io/client-go/kubernetes"
 )
diff --git a/pkg/util/kubernetes/customclient/customclient.go 
b/pkg/util/kubernetes/customclient/customclient.go
index 866d508..345be1d 100644
--- a/pkg/util/kubernetes/customclient/customclient.go
+++ b/pkg/util/kubernetes/customclient/customclient.go
@@ -60,8 +60,8 @@ func GetDynamicClientFor(group string, version string, kind 
string, namespace st
                return nil, err
        }
        return dynamicClient.Resource(schema.GroupVersionResource{
-               Group: group,
-               Version: version,
+               Group:    group,
+               Version:  version,
                Resource: kind,
        }).Namespace(namespace), nil
-}
\ No newline at end of file
+}
diff --git a/pkg/util/kubernetes/loader.go b/pkg/util/kubernetes/loader.go
index fb6b909..aba9700 100644
--- a/pkg/util/kubernetes/loader.go
+++ b/pkg/util/kubernetes/loader.go
@@ -20,6 +20,7 @@ package kubernetes
 import (
        "encoding/json"
        "fmt"
+
        "k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
        "k8s.io/apimachinery/pkg/runtime"
        "k8s.io/apimachinery/pkg/runtime/serializer"
diff --git a/pkg/util/kubernetes/replace.go b/pkg/util/kubernetes/replace.go
index 619eb9e..ba7b347 100644
--- a/pkg/util/kubernetes/replace.go
+++ b/pkg/util/kubernetes/replace.go
@@ -19,6 +19,7 @@ package kubernetes
 
 import (
        "context"
+
        "github.com/apache/camel-k/pkg/client"
        eventing "github.com/knative/eventing/pkg/apis/eventing/v1alpha1"
        routev1 "github.com/openshift/api/route/v1"
diff --git a/pkg/util/kubernetes/util.go b/pkg/util/kubernetes/util.go
index d07940c..6da33ad 100644
--- a/pkg/util/kubernetes/util.go
+++ b/pkg/util/kubernetes/util.go
@@ -19,10 +19,10 @@ package kubernetes
 
 import (
        "fmt"
-       "k8s.io/apimachinery/pkg/util/json"
 
        "gopkg.in/yaml.v2"
        "k8s.io/apimachinery/pkg/runtime"
+       "k8s.io/apimachinery/pkg/util/json"
 )
 
 // ToJSON --
diff --git a/pkg/util/log/annotation_scraper.go 
b/pkg/util/log/annotation_scraper.go
index 52be8dc..31aaba7 100644
--- a/pkg/util/log/annotation_scraper.go
+++ b/pkg/util/log/annotation_scraper.go
@@ -21,7 +21,6 @@ import (
        "bufio"
        "context"
        "io"
-       "k8s.io/client-go/kubernetes"
        "strconv"
        "sync"
        "sync/atomic"
@@ -30,6 +29,7 @@ import (
        "github.com/sirupsen/logrus"
        "k8s.io/api/core/v1"
        metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
+       "k8s.io/client-go/kubernetes"
 )
 
 // SelectorScraper scrapes all pods with a given selector
diff --git a/pkg/util/log/pod_scraper.go b/pkg/util/log/pod_scraper.go
index 6d74eb6..a0c3c16 100644
--- a/pkg/util/log/pod_scraper.go
+++ b/pkg/util/log/pod_scraper.go
@@ -21,8 +21,6 @@ import (
        "bufio"
        "context"
        "io"
-       "k8s.io/apimachinery/pkg/util/json"
-       "k8s.io/client-go/kubernetes"
        "time"
 
        "github.com/pkg/errors"
@@ -31,7 +29,9 @@ import (
        metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
        "k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
        "k8s.io/apimachinery/pkg/runtime"
+       "k8s.io/apimachinery/pkg/util/json"
        "k8s.io/apimachinery/pkg/watch"
+       "k8s.io/client-go/kubernetes"
 )
 
 var commonUserContainerNames = map[string]bool{
@@ -176,7 +176,6 @@ func (s *PodScraper) waitForPodRunning(ctx context.Context, 
namespace string, na
                                                return "", err
                                        }
 
-
                                } else if gotPod, ok := e.Object.(*v1.Pod); ok {
                                        recvPod = gotPod
                                }
diff --git a/pkg/util/log/util.go b/pkg/util/log/util.go
index d4c4254..eb20eb9 100644
--- a/pkg/util/log/util.go
+++ b/pkg/util/log/util.go
@@ -22,10 +22,10 @@ import (
        "fmt"
        "io"
        "io/ioutil"
-       "k8s.io/client-go/kubernetes"
        "os"
 
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
+       "k8s.io/client-go/kubernetes"
 )
 
 // Print prints integrations logs to the stdout
diff --git a/pkg/util/minishift/minishift.go b/pkg/util/minishift/minishift.go
index ae6ebdc..1a106e2 100644
--- a/pkg/util/minishift/minishift.go
+++ b/pkg/util/minishift/minishift.go
@@ -20,12 +20,12 @@ package minishift
 
 import (
        "context"
-       "github.com/apache/camel-k/pkg/client"
-       "k8s.io/apimachinery/pkg/labels"
        "strconv"
 
+       "github.com/apache/camel-k/pkg/client"
        "k8s.io/api/core/v1"
        metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
+       "k8s.io/apimachinery/pkg/labels"
        k8sclient "sigs.k8s.io/controller-runtime/pkg/client"
 )
 
diff --git a/pkg/util/watch/watch.go b/pkg/util/watch/watch.go
index b9e56b4..dcf35ae 100644
--- a/pkg/util/watch/watch.go
+++ b/pkg/util/watch/watch.go
@@ -19,6 +19,7 @@ package watch
 
 import (
        "context"
+
        "github.com/apache/camel-k/pkg/apis/camel/v1alpha1"
        "github.com/apache/camel-k/pkg/util/kubernetes/customclient"
        "github.com/sirupsen/logrus"

Reply via email to