check in gci fixes on flytectl

flyteorg · Jun 17, 2024 · f0166be · f0166be
1 parent c7cc3ba
commit f0166be
Show file tree

Hide file tree

Showing 118 changed files with 142 additions and 326 deletions.
diff --git a/flytectl/cmd/config/config.go b/flytectl/cmd/config/config.go
@@ -4,9 +4,8 @@ import (
 	"fmt"
 	"strings"
 
-	"github.com/flyteorg/flyte/flytestdlib/config"
-
 	"github.com/flyteorg/flyte/flytectl/pkg/printer"
+	"github.com/flyteorg/flyte/flytestdlib/config"
 )
 
 var (

diff --git a/flytectl/cmd/config/subcommand/clusterresourceattribute/file_config_test.go b/flytectl/cmd/config/subcommand/clusterresourceattribute/file_config_test.go
@@ -4,7 +4,6 @@ import (
 	"testing"
 
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
-
 	"github.com/stretchr/testify/assert"
 )
 

diff --git a/flytectl/cmd/config/subcommand/executionclusterlabel/file_config_test.go b/flytectl/cmd/config/subcommand/executionclusterlabel/file_config_test.go
@@ -4,7 +4,6 @@ import (
 	"testing"
 
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
-
 	"github.com/stretchr/testify/assert"
 )
 

diff --git a/flytectl/cmd/config/subcommand/executionqueueattribute/file_config_test.go b/flytectl/cmd/config/subcommand/executionqueueattribute/file_config_test.go
@@ -4,7 +4,6 @@ import (
 	"testing"
 
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
-
 	"github.com/stretchr/testify/assert"
 )
 

diff --git a/flytectl/cmd/config/subcommand/plugin_override/file_config_test.go b/flytectl/cmd/config/subcommand/plugin_override/file_config_test.go
@@ -4,7 +4,6 @@ import (
 	"testing"
 
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
-
 	"github.com/stretchr/testify/assert"
 )
 

diff --git a/flytectl/cmd/config/subcommand/project/project_config.go b/flytectl/cmd/config/subcommand/project/project_config.go
@@ -8,7 +8,6 @@ import (
 	"github.com/flyteorg/flyte/flytectl/cmd/config"
 	"github.com/flyteorg/flyte/flytectl/pkg/filters"
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
-
 	"gopkg.in/yaml.v3"
 )
 

diff --git a/flytectl/cmd/config/subcommand/project/project_config_test.go b/flytectl/cmd/config/subcommand/project/project_config_test.go
@@ -7,7 +7,6 @@ import (
 	"github.com/flyteorg/flyte/flytectl/clierrors"
 	"github.com/flyteorg/flyte/flytectl/cmd/config"
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
-
 	"github.com/stretchr/testify/assert"
 )
 

diff --git a/flytectl/cmd/config/subcommand/taskresourceattribute/file_config_test.go b/flytectl/cmd/config/subcommand/taskresourceattribute/file_config_test.go
@@ -4,7 +4,6 @@ import (
 	"testing"
 
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
-
 	"github.com/stretchr/testify/assert"
 )
 

diff --git a/flytectl/cmd/config/subcommand/workflowexecutionconfig/file_config_test.go b/flytectl/cmd/config/subcommand/workflowexecutionconfig/file_config_test.go
@@ -4,7 +4,6 @@ import (
 	"testing"
 
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
-
 	"github.com/stretchr/testify/assert"
 )
 

diff --git a/flytectl/cmd/configuration/configuration.go b/flytectl/cmd/configuration/configuration.go
@@ -8,13 +8,11 @@ import (
 	"os"
 	"strings"
 
-	"github.com/flyteorg/flyte/flytectl/pkg/util"
-
-	"github.com/flyteorg/flyte/flytectl/pkg/configutil"
-
 	initConfig "github.com/flyteorg/flyte/flytectl/cmd/config/subcommand/config"
 	cmdcore "github.com/flyteorg/flyte/flytectl/cmd/core"
 	cmdUtil "github.com/flyteorg/flyte/flytectl/pkg/commandutils"
+	"github.com/flyteorg/flyte/flytectl/pkg/configutil"
+	"github.com/flyteorg/flyte/flytectl/pkg/util"
 	"github.com/flyteorg/flyte/flytestdlib/config/viper"
 	"github.com/go-ozzo/ozzo-validation/v4/is"
 	"github.com/spf13/cobra"

diff --git a/flytectl/cmd/configuration/configuration_test.go b/flytectl/cmd/configuration/configuration_test.go
@@ -9,14 +9,11 @@ import (
 	"strings"
 	"testing"
 
-	admin2 "github.com/flyteorg/flyte/flyteidl/clients/go/admin"
-
-	"github.com/flyteorg/flyte/flytectl/pkg/configutil"
-
 	initConfig "github.com/flyteorg/flyte/flytectl/cmd/config/subcommand/config"
 	cmdCore "github.com/flyteorg/flyte/flytectl/cmd/core"
+	"github.com/flyteorg/flyte/flytectl/pkg/configutil"
 	"github.com/flyteorg/flyte/flytectl/pkg/util"
-
+	admin2 "github.com/flyteorg/flyte/flyteidl/clients/go/admin"
 	"github.com/stretchr/testify/assert"
 )
 

diff --git a/flytectl/cmd/core/cmd.go b/flytectl/cmd/core/cmd.go
@@ -4,16 +4,14 @@ import (
 	"context"
 	"fmt"
 
-	"github.com/pkg/errors"
-	"google.golang.org/grpc/codes"
-	"google.golang.org/grpc/status"
-
 	"github.com/flyteorg/flyte/flytectl/cmd/config"
 	"github.com/flyteorg/flyte/flytectl/pkg/pkce"
 	"github.com/flyteorg/flyte/flyteidl/clients/go/admin"
-
+	"github.com/pkg/errors"
 	"github.com/spf13/cobra"
 	"github.com/spf13/pflag"
+	"google.golang.org/grpc/codes"
+	"google.golang.org/grpc/status"
 )
 
 type PFlagProvider interface {

diff --git a/flytectl/cmd/core/cmd_ctx.go b/flytectl/cmd/core/cmd_ctx.go
@@ -3,9 +3,8 @@ package cmdcore
 import (
 	"io"
 
-	"github.com/flyteorg/flyte/flyteidl/clients/go/admin"
-
 	"github.com/flyteorg/flyte/flytectl/pkg/ext"
+	"github.com/flyteorg/flyte/flyteidl/clients/go/admin"
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/service"
 )
 

diff --git a/flytectl/cmd/core/cmd_test.go b/flytectl/cmd/core/cmd_test.go
@@ -7,7 +7,6 @@ import (
 
 	"github.com/flyteorg/flyte/flyteidl/clients/go/admin"
 	"github.com/flyteorg/flyte/flytestdlib/config"
-
 	"github.com/spf13/cobra"
 	"github.com/stretchr/testify/assert"
 )

diff --git a/flytectl/cmd/create/create.go b/flytectl/cmd/create/create.go
@@ -3,7 +3,6 @@ package create
 import (
 	"github.com/flyteorg/flyte/flytectl/cmd/config/subcommand/project"
 	cmdcore "github.com/flyteorg/flyte/flytectl/cmd/core"
-
 	"github.com/spf13/cobra"
 )
 

diff --git a/flytectl/cmd/create/create_test.go b/flytectl/cmd/create/create_test.go
@@ -4,9 +4,8 @@ import (
 	"sort"
 	"testing"
 
-	"github.com/stretchr/testify/assert"
-
 	"github.com/flyteorg/flyte/flytectl/cmd/testutils"
+	"github.com/stretchr/testify/assert"
 )
 
 const testDataFolder = "../testdata/"

diff --git a/flytectl/cmd/create/execution.go b/flytectl/cmd/create/execution.go
@@ -4,11 +4,10 @@ import (
 	"context"
 	"fmt"
 
-	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
-	"github.com/flyteorg/flyte/flytestdlib/logger"
-
 	"github.com/flyteorg/flyte/flytectl/cmd/config"
 	cmdCore "github.com/flyteorg/flyte/flytectl/cmd/core"
+	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
+	"github.com/flyteorg/flyte/flytestdlib/logger"
 )
 
 const (

diff --git a/flytectl/cmd/create/execution_test.go b/flytectl/cmd/create/execution_test.go
@@ -4,17 +4,15 @@ import (
 	"fmt"
 	"testing"
 
+	"github.com/flyteorg/flyte/flytectl/cmd/config"
+	cmdCore "github.com/flyteorg/flyte/flytectl/cmd/core"
+	"github.com/flyteorg/flyte/flytectl/cmd/testutils"
+	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
+	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/core"
 	"github.com/golang/protobuf/proto"
 	"github.com/stretchr/testify/mock"
 	"github.com/stretchr/testify/suite"
 	"google.golang.org/protobuf/types/known/timestamppb"
-
-	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
-	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/core"
-
-	"github.com/flyteorg/flyte/flytectl/cmd/config"
-	cmdCore "github.com/flyteorg/flyte/flytectl/cmd/core"
-	"github.com/flyteorg/flyte/flytectl/cmd/testutils"
 )
 
 type createSuite struct {

diff --git a/flytectl/cmd/create/execution_util.go b/flytectl/cmd/create/execution_util.go
@@ -6,14 +6,13 @@ import (
 	"io/ioutil"
 	"strings"
 
+	cmdCore "github.com/flyteorg/flyte/flytectl/cmd/core"
+	cmdGet "github.com/flyteorg/flyte/flytectl/cmd/get"
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/core"
 	"github.com/flyteorg/flyte/flytestdlib/logger"
 	"github.com/google/uuid"
 	"sigs.k8s.io/yaml"
-
-	cmdCore "github.com/flyteorg/flyte/flytectl/cmd/core"
-	cmdGet "github.com/flyteorg/flyte/flytectl/cmd/get"
 )
 
 func createExecutionRequestForWorkflow(ctx context.Context, workflowName, project, domain string,

diff --git a/flytectl/cmd/create/execution_util_test.go b/flytectl/cmd/create/execution_util_test.go
@@ -5,11 +5,9 @@ import (
 	"fmt"
 	"testing"
 
+	"github.com/flyteorg/flyte/flytectl/cmd/config"
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/core"
-
-	"github.com/flyteorg/flyte/flytectl/cmd/config"
-
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/mock"
 )

diff --git a/flytectl/cmd/create/project.go b/flytectl/cmd/create/project.go
@@ -7,9 +7,8 @@ import (
 	"github.com/flyteorg/flyte/flytectl/clierrors"
 	"github.com/flyteorg/flyte/flytectl/cmd/config"
 	"github.com/flyteorg/flyte/flytectl/cmd/config/subcommand/project"
-	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
-
 	cmdCore "github.com/flyteorg/flyte/flytectl/cmd/core"
+	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
 	"github.com/flyteorg/flyte/flytestdlib/logger"
 )
 

diff --git a/flytectl/cmd/create/project_test.go b/flytectl/cmd/create/project_test.go
@@ -6,12 +6,9 @@ import (
 	"testing"
 
 	"github.com/flyteorg/flyte/flytectl/clierrors"
-
 	"github.com/flyteorg/flyte/flytectl/cmd/config"
 	"github.com/flyteorg/flyte/flytectl/cmd/config/subcommand/project"
-
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
-
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/mock"
 )

diff --git a/flytectl/cmd/delete/delete.go b/flytectl/cmd/delete/delete.go
@@ -9,7 +9,6 @@ import (
 	"github.com/flyteorg/flyte/flytectl/cmd/config/subcommand/taskresourceattribute"
 	"github.com/flyteorg/flyte/flytectl/cmd/config/subcommand/workflowexecutionconfig"
 	cmdcore "github.com/flyteorg/flyte/flytectl/cmd/core"
-
 	"github.com/spf13/cobra"
 )
 

diff --git a/flytectl/cmd/delete/delete_test.go b/flytectl/cmd/delete/delete_test.go
@@ -4,9 +4,8 @@ import (
 	"sort"
 	"testing"
 
-	"github.com/stretchr/testify/assert"
-
 	"github.com/flyteorg/flyte/flytectl/cmd/testutils"
+	"github.com/stretchr/testify/assert"
 )
 
 const (

diff --git a/flytectl/cmd/delete/execution_test.go b/flytectl/cmd/delete/execution_test.go
@@ -7,7 +7,6 @@ import (
 	"github.com/flyteorg/flyte/flytectl/cmd/config"
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/core"
-
 	"github.com/stretchr/testify/assert"
 )
 

diff --git a/flytectl/cmd/delete/matchable_cluster_resource_attribute_test.go b/flytectl/cmd/delete/matchable_cluster_resource_attribute_test.go
@@ -7,7 +7,6 @@ import (
 	"github.com/flyteorg/flyte/flytectl/cmd/config"
 	"github.com/flyteorg/flyte/flytectl/cmd/config/subcommand/clusterresourceattribute"
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
-
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/mock"
 )

diff --git a/flytectl/cmd/delete/matchable_execution_cluster_label_test.go b/flytectl/cmd/delete/matchable_execution_cluster_label_test.go
@@ -7,7 +7,6 @@ import (
 	"github.com/flyteorg/flyte/flytectl/cmd/config"
 	"github.com/flyteorg/flyte/flytectl/cmd/config/subcommand/executionclusterlabel"
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
-
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/mock"
 )

diff --git a/flytectl/cmd/delete/matchable_execution_queue_attribute_test.go b/flytectl/cmd/delete/matchable_execution_queue_attribute_test.go
@@ -7,7 +7,6 @@ import (
 	"github.com/flyteorg/flyte/flytectl/cmd/config"
 	"github.com/flyteorg/flyte/flytectl/cmd/config/subcommand/executionqueueattribute"
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
-
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/mock"
 )

diff --git a/flytectl/cmd/delete/matchable_plugin_override_test.go b/flytectl/cmd/delete/matchable_plugin_override_test.go
@@ -7,7 +7,6 @@ import (
 	"github.com/flyteorg/flyte/flytectl/cmd/config"
 	pluginoverride "github.com/flyteorg/flyte/flytectl/cmd/config/subcommand/plugin_override"
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
-
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/mock"
 )

diff --git a/flytectl/cmd/delete/matchable_task_resource_attribute_test.go b/flytectl/cmd/delete/matchable_task_resource_attribute_test.go
@@ -7,7 +7,6 @@ import (
 	"github.com/flyteorg/flyte/flytectl/cmd/config"
 	"github.com/flyteorg/flyte/flytectl/cmd/config/subcommand/taskresourceattribute"
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
-
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/mock"
 )

diff --git a/flytectl/cmd/delete/matchable_workflow_execution_config.go b/flytectl/cmd/delete/matchable_workflow_execution_config.go
@@ -3,10 +3,9 @@ package delete
 import (
 	"context"
 
-	"github.com/flyteorg/flyte/flytectl/cmd/config/subcommand/workflowexecutionconfig"
-
 	"github.com/flyteorg/flyte/flytectl/cmd/config"
 	sconfig "github.com/flyteorg/flyte/flytectl/cmd/config/subcommand"
+	"github.com/flyteorg/flyte/flytectl/cmd/config/subcommand/workflowexecutionconfig"
 	cmdCore "github.com/flyteorg/flyte/flytectl/cmd/core"
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
 )

diff --git a/flytectl/cmd/delete/matchable_workflow_execution_config_test.go b/flytectl/cmd/delete/matchable_workflow_execution_config_test.go
@@ -4,11 +4,9 @@ import (
 	"fmt"
 	"testing"
 
-	"github.com/flyteorg/flyte/flytectl/cmd/config/subcommand/workflowexecutionconfig"
-
 	"github.com/flyteorg/flyte/flytectl/cmd/config"
+	"github.com/flyteorg/flyte/flytectl/cmd/config/subcommand/workflowexecutionconfig"
 	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
-
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/mock"
 )

diff --git a/flytectl/cmd/demo/exec_test.go b/flytectl/cmd/demo/exec_test.go
@@ -8,16 +8,13 @@ import (
 	"strings"
 	"testing"
 
-	"github.com/flyteorg/flyte/flytectl/cmd/testutils"
-
-	admin2 "github.com/flyteorg/flyte/flyteidl/clients/go/admin"
-
-	cmdCore "github.com/flyteorg/flyte/flytectl/cmd/core"
-	"github.com/stretchr/testify/assert"
-
 	"github.com/docker/docker/api/types"
+	cmdCore "github.com/flyteorg/flyte/flytectl/cmd/core"
+	"github.com/flyteorg/flyte/flytectl/cmd/testutils"
 	"github.com/flyteorg/flyte/flytectl/pkg/docker"
 	"github.com/flyteorg/flyte/flytectl/pkg/docker/mocks"
+	admin2 "github.com/flyteorg/flyte/flyteidl/clients/go/admin"
+	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/mock"
 )
 

diff --git a/flytectl/cmd/demo/start.go b/flytectl/cmd/demo/start.go
@@ -3,12 +3,10 @@ package demo
 import (
 	"context"
 
-	"github.com/flyteorg/flyte/flytectl/pkg/docker"
-
-	"github.com/flyteorg/flyte/flytectl/pkg/sandbox"
-
 	sandboxCmdConfig "github.com/flyteorg/flyte/flytectl/cmd/config/subcommand/sandbox"
 	cmdCore "github.com/flyteorg/flyte/flytectl/cmd/core"
+	"github.com/flyteorg/flyte/flytectl/pkg/docker"
+	"github.com/flyteorg/flyte/flytectl/pkg/sandbox"
 )
 
 const (

diff --git a/flytectl/cmd/demo/status.go b/flytectl/cmd/demo/status.go
@@ -3,10 +3,9 @@ package demo
 import (
 	"context"
 
-	"github.com/flyteorg/flyte/flytectl/pkg/sandbox"
-
 	cmdCore "github.com/flyteorg/flyte/flytectl/cmd/core"
 	"github.com/flyteorg/flyte/flytectl/pkg/docker"
+	"github.com/flyteorg/flyte/flytectl/pkg/sandbox"
 )
 
 const (
-Original file line number
+Diff line change
@@ Expand Up / @@ -4,7 +4,6 @@ import ( @@
     	"testing"
     	"github.com/flyteorg/flyte/flyteidl/gen/pb-go/flyteidl/admin"
     	"github.com/stretchr/testify/assert"
     )
@@ Expand Down @@