delete require profile/machine name

This changeset makes it necessary to pass a profile/machine name in order
to delete. This would help avoid accidental deletion of machine by just
delete command.

cmd/minikube/cmd/delete.go

@@ -21,18 +21,27 @@ import (
 	"os"
 
 	"github.com/spf13/cobra"
+	"github.com/spf13/viper"
 	cmdUtil "k8s.io/minikube/cmd/util"
 	"k8s.io/minikube/pkg/minikube/cluster"
+	"k8s.io/minikube/pkg/minikube/config"
 	"k8s.io/minikube/pkg/minikube/machine"
 )
 
 // deleteCmd represents the delete command
 var deleteCmd = &cobra.Command{
-	Use:   "delete",
-	Short: "Deletes a local kubernetes cluster",
-	Long: `Deletes a local kubernetes cluster. This command deletes the VM, and removes all
-associated files.`,
+	Use:   "delete MINIKUBE_PROFILE_NAME",
+	Short: "Deletes a local kubernetes cluster under a given profile",
+	Long: `Deletes a local kubernetes cluster under a given profile. This command deletes
+the VM, and removes all associated files.`,
 	Run: func(cmd *cobra.Command, args []string) {
+		if len(args) != 1 {
+			fmt.Fprintln(os.Stderr, "usage: minikube delete MINIKUBE_PROFILE_NAME")
+			os.Exit(1)
+		}
+
+		viper.Set(config.MachineProfile, args[0])
+
 		fmt.Println("Deleting local Kubernetes cluster...")
 		api, err := machine.NewAPIClient()
 		if err != nil {

test/integration/start_stop_delete_test.go

@@ -35,7 +35,7 @@ func TestStartStop(t *testing.T) {
 		Args:       *args,
 		BinaryPath: *binaryPath,
 		T:          t}
-	runner.RunCommand("delete", false)
+	runner.RunCommand("delete minikube", false)
 	runner.CheckStatus(state.None.String())
 
 	runner.Start()
@@ -63,6 +63,6 @@ func TestStartStop(t *testing.T) {
 	runner.Start()
 	runner.CheckStatus(state.Running.String())
 
-	runner.RunCommand("delete", true)
+	runner.RunCommand("delete minikube", true)
 	runner.CheckStatus(state.None.String())
 }