Fixup flytectl (flyteorg#5309)

* Fix up flytectl Signed-off-by: Eduardo Apolinario <[email protected]> * Rename module and fix references Signed-off-by: Eduardo Apolinario <[email protected]> * Hook up linting and unit tests Signed-off-by: Eduardo Apolinario <[email protected]> * Remove boilerplate directory Signed-off-by: Eduardo Apolinario <[email protected]> * Include flytectl in the go-checks gh job Signed-off-by: Eduardo Apolinario <[email protected]> * Fix a few unit tests Signed-off-by: Eduardo Apolinario <[email protected]> * Fix more tests Signed-off-by: Eduardo Apolinario <[email protected]> * Fix more tests Signed-off-by: Eduardo Apolinario <[email protected]> * Fix graphviz test Signed-off-by: Eduardo Apolinario <[email protected]> * Reorder test in graphviz_test Signed-off-by: Eduardo Apolinario <[email protected]> * Remove -race from unit tests temporarily Signed-off-by: Eduardo Apolinario <[email protected]> --------- Signed-off-by: Eduardo Apolinario <[email protected]> Co-authored-by: Eduardo Apolinario <[email protected]>
austin362667 · May 7, 2024 · 24e62dc · 24e62dc
1 parent 0aec80c
commit 24e62dc
Show file tree

Hide file tree

Showing 197 changed files with 811 additions and 2,859 deletions.
diff --git a/.github/workflows/checks.yml b/.github/workflows/checks.yml
@@ -34,6 +34,7 @@ jobs:
           - flyteadmin
           # TODO(monorepo): Enable lint flytecopilot
           # - flytecopilot
+          - flytectl
           - flyteidl
           - flyteplugins
           - flytepropeller
@@ -53,6 +54,7 @@ jobs:
           - datacatalog
           - flyteadmin
           - flytecopilot
+          - flytectl
           - flyteplugins
           - flytepropeller
           - flytestdlib
@@ -74,7 +76,7 @@ jobs:
     name: Docker Build Images
     uses: ./.github/workflows/component_docker_build.yml
     with:
-       component: ${{ matrix.component }}
+      component: ${{ matrix.component }}
 
   # TODO(monorepo): these tests are broken. They never test an actual change.
   # endtoend:
@@ -113,6 +115,7 @@ jobs:
           - datacatalog
           - flyteadmin
           - flytecopilot
+          - flytectl
           - flytepropeller
     uses: ./.github/workflows/go_generate.yml
     with:

diff --git a/flytectl/.golangci.yml b/flytectl/.golangci.yml
@@ -28,3 +28,9 @@ linters:
     - unparam
     - unused
     - varcheck
+linters-settings:
+  gci:
+    skip-generated: true
+issues:
+  exclude:
+    - copylocks
diff --git a/flytectl/CODE_OF_CONDUCT.md b/flytectl/CODE_OF_CONDUCT.md
diff --git a/flytectl/Makefile b/flytectl/Makefile
@@ -1,6 +1,8 @@
 export REPOSITORY=flytectl
-include boilerplate/flyte/golang_test_targets/Makefile
-include boilerplate/flyte/precommit/Makefile
+export REPO_ROOT=..
+include ../boilerplate/flyte/docker_build/Makefile
+include ../boilerplate/flyte/golang_test_targets/Makefile
+include ../boilerplate/flyte/end2end/Makefile
 
 GIT_VERSION := $(shell git describe --always --tags)
 GIT_HASH := $(shell git rev-parse --short HEAD)
@@ -38,3 +40,8 @@ test_unit_without_flag:
 	cat coverage.temp.txt  | grep -v "_flags.go" > coverage.txt
 	rm coverage.temp.txt
 	curl -s https://codecov.io/bash > codecov_bash.sh && bash codecov_bash.sh
+
+# TODO - Add the -race flag back
+.PHONY: test_unit
+test_unit:
+	go test -cover ./...
diff --git a/flytectl/README.md b/flytectl/README.md
@@ -16,12 +16,12 @@
 </html>
 
 [![Docs](https://readthedocs.org/projects/flytectl/badge/?version=latest&style=plastic)](https://flytectl.rtfd.io)
-[![Current Release](https://img.shields.io/github/release/flyteorg/flytectl.svg)](https://github.com/flyteorg/flytectl/releases/latest)
-![Master](https://github.com/flyteorg/flytectl/workflows/Master/badge.svg)
-[![GoDoc](https://godoc.org/github.com/flyteorg/flytectl?status.svg)](https://pkg.go.dev/mod/github.com/flyteorg/flytectl)
+[![Current Release](https://img.shields.io/github/release/flyteorg/flyte/flytectl.svg)](https://github.com/flyteorg/flytectl/releases/latest)
+![Master](https://github.com/flyteorg/flyte/flytectl/workflows/Master/badge.svg)
+[![GoDoc](https://godoc.org/github.com/flyteorg/flyte/flytectl?status.svg)](https://pkg.go.dev/mod/github.com/flyteorg/flytectl)
 [![License](https://img.shields.io/badge/LICENSE-Apache2.0-ff69b4.svg)](http://www.apache.org/licenses/LICENSE-2.0.html)
-[![CodeCoverage](https://img.shields.io/codecov/c/github/flyteorg/flytectl.svg)](https://codecov.io/gh/flyteorg/flytectl)
-[![Go Report Card](https://goreportcard.com/badge/github.com/flyteorg/flytectl)](https://goreportcard.com/report/github.com/flyteorg/flytectl)
+[![CodeCoverage](https://img.shields.io/codecov/c/github/flyteorg/flyte/flytectl.svg)](https://codecov.io/gh/flyteorg/flytectl)
+[![Go Report Card](https://goreportcard.com/badge/github.com/flyteorg/flyte/flytectl)](https://goreportcard.com/report/github.com/flyteorg/flytectl)
 ![Commit activity](https://img.shields.io/github/commit-activity/w/lyft/flytectl.svg?style=plastic)
 ![Commit since last release](https://img.shields.io/github/commits-since/lyft/flytectl/latest.svg?style=plastic)
 [![Slack](https://img.shields.io/badge/slack-join_chat-white.svg?logo=slack&style=social)](https://slack.flyte.org)

diff --git a/flytectl/boilerplate/flyte/code_of_conduct/CODE_OF_CONDUCT.md b/flytectl/boilerplate/flyte/code_of_conduct/CODE_OF_CONDUCT.md
diff --git a/flytectl/boilerplate/flyte/code_of_conduct/README.rst b/flytectl/boilerplate/flyte/code_of_conduct/README.rst
diff --git a/flytectl/boilerplate/flyte/code_of_conduct/update.sh b/flytectl/boilerplate/flyte/code_of_conduct/update.sh
diff --git a/flytectl/boilerplate/flyte/golang_support_tools/go.mod b/flytectl/boilerplate/flyte/golang_support_tools/go.mod