Move sparkctl to cmd directory

Makefile

@@ -173,12 +173,12 @@ override LDFLAGS += \
 .PHONY: build-operator
 build-operator: ## Build Spark operator.
 	echo "Building spark-operator binary..."
-	go build -o $(SPARK_OPERATOR) -ldflags '${LDFLAGS}' cmd/main.go
+	go build -o $(SPARK_OPERATOR) -ldflags '${LDFLAGS}' cmd/operator/main.go
 
 .PHONY: build-sparkctl
 build-sparkctl: ## Build sparkctl binary.
 	echo "Building sparkctl binary..."
-	CGO_ENABLED=0 go build -o $(SPARKCTL) -buildvcs=false sparkctl/main.go
+	CGO_ENABLED=0 go build -o $(SPARKCTL) -buildvcs=false cmd/sparkctl/main.go
 
 .PHONY: install-sparkctl
 install-sparkctl: build-sparkctl ## Install sparkctl binary.
@@ -191,7 +191,7 @@ clean: ## Clean spark-operator and sparktcl binaries.
 	rm -f $(SPARKCTL)
 
 .PHONY: build-api-docs
-build-api-docs: gen-crd-api-reference-docs ## Build api documentaion.
+build-api-docs: gen-crd-api-reference-docs ## Build api documentation.
 	$(GEN_CRD_API_REFERENCE_DOCS) \
 	-config hack/api-docs/config.json \
 	-api-dir github.com/kubeflow/spark-operator/api/v1beta2 \

cmd/operator/root.go → cmd/operator/app/root.go

@@ -14,14 +14,14 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package operator
+package app
 
 import (
 	"github.com/spf13/cobra"
 
-	"github.com/kubeflow/spark-operator/cmd/operator/controller"
-	"github.com/kubeflow/spark-operator/cmd/operator/version"
-	"github.com/kubeflow/spark-operator/cmd/operator/webhook"
+	"github.com/kubeflow/spark-operator/cmd/operator/app/controller"
+	"github.com/kubeflow/spark-operator/cmd/operator/app/version"
+	"github.com/kubeflow/spark-operator/cmd/operator/app/webhook"
 )
 
 func NewCommand() *cobra.Command {

cmd/main.go → cmd/operator/main.go

@@ -20,11 +20,11 @@ import (
 	"fmt"
 	"os"
 
-	"github.com/kubeflow/spark-operator/cmd/operator"
+	"github.com/kubeflow/spark-operator/cmd/operator/app"
 )
 
 func main() {
-	if err := operator.NewCommand().Execute(); err != nil {
+	if err := app.NewCommand().Execute(); err != nil {
 		fmt.Fprintf(os.Stderr, "%v\n", err)
 		os.Exit(1)
 	}

sparkctl/cmd/client.go → cmd/sparkctl/app/client.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package cmd
+package app
 
 import (
 	"context"

sparkctl/cmd/create.go → cmd/sparkctl/app/create.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package cmd
+package app
 
 import (
 	"context"

sparkctl/cmd/create_test.go → cmd/sparkctl/app/create_test.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package cmd
+package app
 
 import (
 	"strings"

sparkctl/cmd/delete.go → cmd/sparkctl/app/delete.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package cmd
+package app
 
 import (
 	"context"

sparkctl/cmd/event.go → cmd/sparkctl/app/event.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package cmd
+package app
 
 import (
 	"context"

sparkctl/cmd/forward.go → cmd/sparkctl/app/forward.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package cmd
+package app
 
 import (
 	"context"

sparkctl/cmd/gcs.go → cmd/sparkctl/app/gcs.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package cmd
+package app
 
 import (
 	"fmt"

sparkctl/cmd/list.go → cmd/sparkctl/app/list.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package cmd
+package app
 
 import (
 	"context"

sparkctl/cmd/log.go → cmd/sparkctl/app/log.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package cmd
+package app
 
 import (
 	"context"

sparkctl/cmd/root.go → cmd/sparkctl/app/root.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package cmd
+package app
 
 import (
 	"fmt"

sparkctl/cmd/s3.go → cmd/sparkctl/app/s3.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package cmd
+package app
 
 import (
 	"context"

sparkctl/cmd/status.go → cmd/sparkctl/app/status.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package cmd
+package app
 
 import (
 	"fmt"

sparkctl/cmd/utils.go → cmd/sparkctl/app/utils.go

@@ -14,7 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-package cmd
+package app
 
 import (
 	"time"

sparkctl/main.go → cmd/sparkctl/main.go

@@ -19,9 +19,9 @@ package main
 import (
 	_ "k8s.io/client-go/plugin/pkg/client/auth"
 
-	"github.com/kubeflow/spark-operator/sparkctl/cmd"
+	"github.com/kubeflow/spark-operator/cmd/sparkctl/app"
 )
 
 func main() {
-	cmd.Execute()
+	app.Execute()
 }