Deprecating sparkctl (#2484)

* Added deprecation warning in cmd/sparkctl/main.go

Signed-off-by: Vikas Saxena <[email protected]>

* added deprecation warning in Makefile

Signed-off-by: Vikas Saxena <[email protected]>

* added recommendation to use kubectl in Makefile

Signed-off-by: Vikas Saxena <[email protected]>

* updated readme for sparkctl to show its being deprecated

Signed-off-by: Vikas Saxena <[email protected]>

* adding deprecated comment as well to main.go

Signed-off-by: Vikas Saxena <[email protected]>

---------

Signed-off-by: Vikas Saxena <[email protected]>

Author: vikas-saxena02

Makefile

@@ -177,6 +177,7 @@ build-operator: ## Build Spark operator.
 
 .PHONY: build-sparkctl
 build-sparkctl: ## Build sparkctl binary.
+	echo "⚠️ Warning: sparkctl is deprecated and no longer maintained. It will be removed in a future release. Please use kubectl instead."
 	echo "Building sparkctl binary..."
 	CGO_ENABLED=0 go build -o $(SPARKCTL) -buildvcs=false cmd/sparkctl/main.go
 

cmd/sparkctl/README.md

@@ -1,4 +1,6 @@
 # sparkctl
+> ⚠️ **Deprecated:** `sparkctl` is no longer actively maintained and may be removed in a future release.  
+> We recommend using `kubectl` or other Kubernetes-native tools to manage `SparkApplication` resources.
 
 `sparkctl` is a command-line tool of the Spark Operator for creating, listing, checking status of, getting logs of, and deleting `SparkApplication`s. It can also do port forwarding from a local port to the Spark web UI port for accessing the Spark web UI on the driver. Each function is implemented as a sub-command of `sparkctl`.
 

cmd/sparkctl/main.go

@@ -17,11 +17,16 @@ limitations under the License.
 package main
 
 import (
+	"fmt"
+	"os"
+
 	_ "k8s.io/client-go/plugin/pkg/client/auth"
 
 	"github.com/kubeflow/spark-operator/cmd/sparkctl/app"
 )
 
+// Deprecated: sparkctl is no longer maintained. Use kubectl or other tools instead.
 func main() {
+	fmt.Fprintln(os.Stderr, "⚠️ Warning: sparkctl is deprecated and no longer maintained. Use kubectl instead.")
 	app.Execute()
 }