From 110ab79f05e031effaa8563d371cbfde4a3f4f78 Mon Sep 17 00:00:00 2001
From: Jerzy Szczepkowski <jsz@google.com>
Date: Tue, 10 Mar 2015 13:21:49 +0100
Subject: [PATCH] Added support for labels to "kubectl stop".

Added support for labels and --all to "kubectl stop". Fixes #5178.
---
 docs/kubectl-delete.md       |  2 +-
 docs/kubectl-stop.md         |  7 ++++++-
 docs/man/man1/kubectl-stop.1 | 11 +++++++++++
 pkg/kubectl/cmd/delete.go    |  2 +-
 pkg/kubectl/cmd/stop.go      |  9 ++++++++-
 5 files changed, 27 insertions(+), 4 deletions(-)

diff --git a/docs/kubectl-delete.md b/docs/kubectl-delete.md
index 470e042dd4aa2..7fa55933b1486 100644
--- a/docs/kubectl-delete.md
+++ b/docs/kubectl-delete.md
@@ -17,7 +17,7 @@ submits an update to a resource right when you submit a delete, their update
 will be lost along with the rest of the resource.
 ```
 
-kubectl delete ([-f filename] | (<resource> [(<id> | -l <label> | --all)]
+kubectl delete (-f filename | <resource> (<id> | -l <label> | --all))
 
 ### Examples
 
diff --git a/docs/kubectl-stop.md b/docs/kubectl-stop.md
index 8ca896d74d54d..05ebc70c701f4 100644
--- a/docs/kubectl-stop.md
+++ b/docs/kubectl-stop.md
@@ -11,7 +11,7 @@ Attempts to shut down and delete a resource that supports graceful termination.
 If the resource is resizable it will be resized to 0 before deletion.
 ```
 
-kubectl stop (<resource> <id>|-f filename)
+kubectl stop (-f filename | <resource> (<id> | -l <label> | --all))
 
 ### Examples
 
@@ -19,6 +19,9 @@ kubectl stop (<resource> <id>|-f filename)
 // Shut down foo.
 $ kubectl stop replicationcontroller foo
 
+// Stop pods and services with label name=myLabel.
+$ kubectl stop pods,services -l name=myLabel
+
 // Shut down the service defined in service.json
 $ kubectl stop -f service.json
 
@@ -29,7 +32,9 @@ $ kubectl stop -f path/to/resources
 ### Options
 
 ```
+      --all=false: [-all] to select all the specified resources
   -f, --filename=[]: Filename, directory, or URL to file of resource(s) to be stopped
+  -l, --selector="": Selector (label query) to filter on
 ```
 
 ### Options inherrited from parent commands
diff --git a/docs/man/man1/kubectl-stop.1 b/docs/man/man1/kubectl-stop.1
index caa979d25fca0..9ab05bbff1b7a 100644
--- a/docs/man/man1/kubectl-stop.1
+++ b/docs/man/man1/kubectl-stop.1
@@ -21,10 +21,18 @@ If the resource is resizable it will be resized to 0 before deletion.
 
 
 .SH OPTIONS
+.PP
+\fB\-\-all\fP=false
+    [\-all] to select all the specified resources
+
 .PP
 \fB\-f\fP, \fB\-\-filename\fP=[]
     Filename, directory, or URL to file of resource(s) to be stopped
 
+.PP
+\fB\-l\fP, \fB\-\-selector\fP=""
+    Selector (label query) to filter on
+
 
 .SH OPTIONS INHERITED FROM PARENT COMMANDS
 .PP
@@ -140,6 +148,9 @@ If the resource is resizable it will be resized to 0 before deletion.
 // Shut down foo.
 $ kubectl stop replicationcontroller foo
 
+// Stop pods and services with label name=myLabel.
+$ kubectl stop pods,services \-l name=myLabel
+
 // Shut down the service defined in service.json
 $ kubectl stop \-f service.json
 
diff --git a/pkg/kubectl/cmd/delete.go b/pkg/kubectl/cmd/delete.go
index 1ed5331db097a..1f75e243bdc32 100644
--- a/pkg/kubectl/cmd/delete.go
+++ b/pkg/kubectl/cmd/delete.go
@@ -60,7 +60,7 @@ func (f *Factory) NewCmdDelete(out io.Writer) *cobra.Command {
 		Filenames util.StringList
 	}{}
 	cmd := &cobra.Command{
-		Use:     "delete ([-f filename] | (<resource> [(<id> | -l <label> | --all)]",
+		Use:     "delete (-f filename | <resource> (<id> | -l <label> | --all))",
 		Short:   "Delete a resource by filename, stdin, or resource and ID.",
 		Long:    delete_long,
 		Example: delete_example,
diff --git a/pkg/kubectl/cmd/stop.go b/pkg/kubectl/cmd/stop.go
index b8b17a11bcfee..72959f865bc7d 100644
--- a/pkg/kubectl/cmd/stop.go
+++ b/pkg/kubectl/cmd/stop.go
@@ -34,6 +34,9 @@ If the resource is resizable it will be resized to 0 before deletion.`
 	stop_example = `// Shut down foo.
 $ kubectl stop replicationcontroller foo
 
+// Stop pods and services with label name=myLabel.
+$ kubectl stop pods,services -l name=myLabel
+
 // Shut down the service defined in service.json
 $ kubectl stop -f service.json
 
@@ -46,7 +49,7 @@ func (f *Factory) NewCmdStop(out io.Writer) *cobra.Command {
 		Filenames util.StringList
 	}{}
 	cmd := &cobra.Command{
-		Use:     "stop (<resource> <id>|-f filename)",
+		Use:     "stop (-f filename | <resource> (<id> | -l <label> | --all))",
 		Short:   "Gracefully shut down a resource by id or filename.",
 		Long:    stop_long,
 		Example: stop_example,
@@ -59,6 +62,8 @@ func (f *Factory) NewCmdStop(out io.Writer) *cobra.Command {
 				NamespaceParam(cmdNamespace).RequireNamespace().
 				ResourceTypeOrNameArgs(false, args...).
 				FilenameParam(flags.Filenames...).
+				SelectorParam(cmdutil.GetFlagString(cmd, "selector")).
+				SelectAllParam(cmdutil.GetFlagBool(cmd, "all")).
 				Flatten().
 				Do()
 			cmdutil.CheckErr(r.Err())
@@ -76,5 +81,7 @@ func (f *Factory) NewCmdStop(out io.Writer) *cobra.Command {
 		},
 	}
 	cmd.Flags().VarP(&flags.Filenames, "filename", "f", "Filename, directory, or URL to file of resource(s) to be stopped")
+	cmd.Flags().StringP("selector", "l", "", "Selector (label query) to filter on")
+	cmd.Flags().Bool("all", false, "[-all] to select all the specified resources")
 	return cmd
 }