diff --git a/contrib/completions/bash/kubectl b/contrib/completions/bash/kubectl index 77b23f6af05..5e54cb4155c 100644 --- a/contrib/completions/bash/kubectl +++ b/contrib/completions/bash/kubectl @@ -371,9 +371,6 @@ _kubectl_get() flags+=("--template=") flags_with_completion+=("--template") flags_completion+=("_filedir") - two_word_flags+=("-t") - flags_with_completion+=("-t") - flags_completion+=("_filedir") flags+=("--watch") flags+=("-w") flags+=("--watch-only") @@ -617,9 +614,6 @@ _kubectl_create_namespace() flags+=("--template=") flags_with_completion+=("--template") flags_completion+=("_filedir") - two_word_flags+=("-t") - flags_with_completion+=("-t") - flags_completion+=("_filedir") flags+=("--validate") flags+=("--alsologtostderr") flags+=("--api-version=") @@ -686,9 +680,6 @@ _kubectl_create_secret_docker-registry() flags+=("--template=") flags_with_completion+=("--template") flags_completion+=("_filedir") - two_word_flags+=("-t") - flags_with_completion+=("-t") - flags_completion+=("_filedir") flags+=("--validate") flags+=("--alsologtostderr") flags+=("--api-version=") @@ -755,9 +746,6 @@ _kubectl_create_secret_generic() flags+=("--template=") flags_with_completion+=("--template") flags_completion+=("_filedir") - two_word_flags+=("-t") - flags_with_completion+=("-t") - flags_completion+=("_filedir") flags+=("--type=") flags+=("--validate") flags+=("--alsologtostderr") @@ -867,9 +855,6 @@ _kubectl_create_configmap() flags+=("--template=") flags_with_completion+=("--template") flags_completion+=("_filedir") - two_word_flags+=("-t") - flags_with_completion+=("-t") - flags_completion+=("_filedir") flags+=("--validate") flags+=("--alsologtostderr") flags+=("--api-version=") @@ -932,9 +917,6 @@ _kubectl_create_serviceaccount() flags+=("--template=") flags_with_completion+=("--template") flags_completion+=("_filedir") - two_word_flags+=("-t") - flags_with_completion+=("-t") - flags_completion+=("_filedir") flags+=("--validate") flags+=("--alsologtostderr") flags+=("--api-version=") @@ -1685,9 +1667,6 @@ _kubectl_rolling-update() flags+=("--template=") flags_with_completion+=("--template") flags_completion+=("_filedir") - two_word_flags+=("-t") - flags_with_completion+=("-t") - flags_completion+=("_filedir") flags+=("--timeout=") flags+=("--update-period=") flags+=("--validate") @@ -2169,10 +2148,8 @@ _kubectl_run() flags+=("--template=") flags_with_completion+=("--template") flags_completion+=("_filedir") - two_word_flags+=("-t") - flags_with_completion+=("-t") - flags_completion+=("_filedir") flags+=("--tty") + flags+=("-t") flags+=("--alsologtostderr") flags+=("--api-version=") flags+=("--as=") @@ -2253,9 +2230,6 @@ _kubectl_expose() flags+=("--template=") flags_with_completion+=("--template") flags_completion+=("_filedir") - two_word_flags+=("-t") - flags_with_completion+=("-t") - flags_completion+=("_filedir") flags+=("--type=") flags+=("--alsologtostderr") flags+=("--api-version=") @@ -2342,9 +2316,6 @@ _kubectl_autoscale() flags+=("--template=") flags_with_completion+=("--template") flags_completion+=("_filedir") - two_word_flags+=("-t") - flags_with_completion+=("-t") - flags_completion+=("_filedir") flags+=("--alsologtostderr") flags+=("--api-version=") flags+=("--as=") @@ -2669,9 +2640,6 @@ _kubectl_label() flags+=("--template=") flags_with_completion+=("--template") flags_completion+=("_filedir") - two_word_flags+=("-t") - flags_with_completion+=("-t") - flags_completion+=("_filedir") flags+=("--alsologtostderr") flags+=("--api-version=") flags+=("--as=") @@ -2810,9 +2778,6 @@ _kubectl_annotate() flags+=("--template=") flags_with_completion+=("--template") flags_completion+=("_filedir") - two_word_flags+=("-t") - flags_with_completion+=("-t") - flags_completion+=("_filedir") flags+=("--alsologtostderr") flags+=("--api-version=") flags+=("--as=") @@ -2906,9 +2871,6 @@ _kubectl_config_view() flags+=("--template=") flags_with_completion+=("--template") flags_completion+=("_filedir") - two_word_flags+=("-t") - flags_with_completion+=("-t") - flags_completion+=("_filedir") flags+=("--alsologtostderr") flags+=("--api-version=") flags+=("--as=") @@ -3511,9 +3473,6 @@ _kubectl_convert() flags+=("--template=") flags_with_completion+=("--template") flags_completion+=("_filedir") - two_word_flags+=("-t") - flags_with_completion+=("-t") - flags_completion+=("_filedir") flags+=("--validate") flags+=("--alsologtostderr") flags+=("--api-version=") diff --git a/docs/man/man1/kubectl-run.1 b/docs/man/man1/kubectl-run.1 index 2b2664d447c..5845e549a30 100644 --- a/docs/man/man1/kubectl-run.1 +++ b/docs/man/man1/kubectl-run.1 @@ -142,8 +142,8 @@ Creates a deployment or job to manage the created container(s). \[la]http://golang.org/pkg/text/template/#pkg-overview\[ra]]. .PP -\fB\-\-tty\fP=false - Allocated a TTY for each container in the pod. Because \-t is currently shorthand for \-\-template, \-t is not supported for \-\-tty. This shorthand is deprecated and we expect to adopt \-t for \-\-tty soon. +\fB\-t\fP, \fB\-\-tty\fP=false + Allocated a TTY for each container in the pod. .SH OPTIONS INHERITED FROM PARENT COMMANDS @@ -268,7 +268,7 @@ kubectl run nginx \-\-image=nginx \-\-dry\-run kubectl run nginx \-\-image=nginx \-\-overrides='{ "apiVersion": "v1", "spec": { ... } }' # Start a single instance of busybox and keep it in the foreground, don't restart it if it exits. -kubectl run \-i \-\-tty busybox \-\-image=busybox \-\-restart=Never +kubectl run \-i \-t busybox \-\-image=busybox \-\-restart=Never # Start the nginx container using the default command, but use custom arguments (arg1 .. argN) for that command. kubectl run nginx \-\-image=nginx \-\- ... diff --git a/docs/user-guide/kubectl/kubectl_run.md b/docs/user-guide/kubectl/kubectl_run.md index fb424ca01e8..9d2e3de569e 100644 --- a/docs/user-guide/kubectl/kubectl_run.md +++ b/docs/user-guide/kubectl/kubectl_run.md @@ -68,7 +68,7 @@ kubectl run nginx --image=nginx --dry-run kubectl run nginx --image=nginx --overrides='{ "apiVersion": "v1", "spec": { ... } }' # Start a single instance of busybox and keep it in the foreground, don't restart it if it exits. -kubectl run -i --tty busybox --image=busybox --restart=Never +kubectl run -i -t busybox --image=busybox --restart=Never # Start the nginx container using the default command, but use custom arguments (arg1 .. argN) for that command. kubectl run nginx --image=nginx -- ... @@ -113,7 +113,7 @@ kubectl run pi --image=perl --restart=OnFailure -- perl -Mbignum=bpi -wle 'print --sort-by="": If non-empty, sort list types using this field specification. The field specification is expressed as a JSONPath expression (e.g. '{.metadata.name}'). The field in the API resource specified by this JSONPath expression must be an integer or a string. -i, --stdin[=false]: Keep stdin open on the container(s) in the pod, even if nothing is attached. --template="": Template string or path to template file to use when -o=go-template, -o=go-template-file. The template format is golang templates [http://golang.org/pkg/text/template/#pkg-overview]. - --tty[=false]: Allocated a TTY for each container in the pod. Because -t is currently shorthand for --template, -t is not supported for --tty. This shorthand is deprecated and we expect to adopt -t for --tty soon. + -t, --tty[=false]: Allocated a TTY for each container in the pod. ``` ### Options inherited from parent commands @@ -148,7 +148,7 @@ kubectl run pi --image=perl --restart=OnFailure -- perl -Mbignum=bpi -wle 'print * [kubectl](kubectl.md) - kubectl controls the Kubernetes cluster manager -###### Auto generated by spf13/cobra on 5-Apr-2016 +###### Auto generated by spf13/cobra on 15-Apr-2016 [![Analytics](https://kubernetes-site.appspot.com/UA-36037335-10/GitHub/docs/user-guide/kubectl/kubectl_run.md?pixel)]() diff --git a/docs/yaml/kubectl/kubectl_run.yaml b/docs/yaml/kubectl/kubectl_run.yaml index 10dd4ab78bf..86df995b541 100644 --- a/docs/yaml/kubectl/kubectl_run.yaml +++ b/docs/yaml/kubectl/kubectl_run.yaml @@ -114,9 +114,9 @@ options: usage: | Template string or path to template file to use when -o=go-template, -o=go-template-file. The template format is golang templates [http://golang.org/pkg/text/template/#pkg-overview]. - name: tty + shorthand: t default_value: "false" - usage: | - Allocated a TTY for each container in the pod. Because -t is currently shorthand for --template, -t is not supported for --tty. This shorthand is deprecated and we expect to adopt -t for --tty soon. + usage: Allocated a TTY for each container in the pod. inherited_options: - name: alsologtostderr default_value: "false" @@ -198,7 +198,7 @@ example: |- kubectl run nginx --image=nginx --overrides='{ "apiVersion": "v1", "spec": { ... } }' # Start a single instance of busybox and keep it in the foreground, don't restart it if it exits. - kubectl run -i --tty busybox --image=busybox --restart=Never + kubectl run -i -t busybox --image=busybox --restart=Never # Start the nginx container using the default command, but use custom arguments (arg1 .. argN) for that command. kubectl run nginx --image=nginx -- ... diff --git a/pkg/kubectl/cmd/run.go b/pkg/kubectl/cmd/run.go index 82b6167ca39..010164bdaa2 100644 --- a/pkg/kubectl/cmd/run.go +++ b/pkg/kubectl/cmd/run.go @@ -57,7 +57,7 @@ kubectl run nginx --image=nginx --dry-run kubectl run nginx --image=nginx --overrides='{ "apiVersion": "v1", "spec": { ... } }' # Start a single instance of busybox and keep it in the foreground, don't restart it if it exits. -kubectl run -i --tty busybox --image=busybox --restart=Never +kubectl run -i -t busybox --image=busybox --restart=Never # Start the nginx container using the default command, but use custom arguments (arg1 .. argN) for that command. kubectl run nginx --image=nginx -- ... @@ -104,7 +104,7 @@ func addRunFlags(cmd *cobra.Command) { cmd.Flags().Int("hostport", -1, "The host port mapping for the container port. To demonstrate a single-machine container.") cmd.Flags().StringP("labels", "l", "", "Labels to apply to the pod(s).") cmd.Flags().BoolP("stdin", "i", false, "Keep stdin open on the container(s) in the pod, even if nothing is attached.") - cmd.Flags().Bool("tty", false, "Allocated a TTY for each container in the pod. Because -t is currently shorthand for --template, -t is not supported for --tty. This shorthand is deprecated and we expect to adopt -t for --tty soon.") + cmd.Flags().BoolP("tty", "t", false, "Allocated a TTY for each container in the pod.") cmd.Flags().Bool("attach", false, "If true, wait for the Pod to start running, and then attach to the Pod as if 'kubectl attach ...' were called. Default false, unless '-i/--interactive' is set, in which case the default is true.") cmd.Flags().Bool("leave-stdin-open", false, "If the pod is started in interactive mode or with stdin, leave stdin open after the first attach completes. By default, stdin will be closed after the first attach completes.") cmd.Flags().String("restart", "Always", "The restart policy for this Pod. Legal values [Always, OnFailure, Never]. If set to 'Always' a deployment is created for this pod, if set to OnFailure or Never, a job is created for this pod and --replicas must be 1. Default 'Always'") @@ -129,7 +129,7 @@ func Run(f *cmdutil.Factory, cmdIn io.Reader, cmdOut, cmdErr io.Writer, cmd *cob interactive := cmdutil.GetFlagBool(cmd, "stdin") tty := cmdutil.GetFlagBool(cmd, "tty") if tty && !interactive { - return cmdutil.UsageError(cmd, "-i/--stdin is required for containers with --tty=true") + return cmdutil.UsageError(cmd, "-i/--stdin is required for containers with -t/--tty=true") } replicas := cmdutil.GetFlagInt(cmd, "replicas") if interactive && replicas != 1 { diff --git a/pkg/kubectl/cmd/util/printing.go b/pkg/kubectl/cmd/util/printing.go index 1e1f943d7b3..f1153b6cd61 100644 --- a/pkg/kubectl/cmd/util/printing.go +++ b/pkg/kubectl/cmd/util/printing.go @@ -34,11 +34,8 @@ func AddPrinterFlags(cmd *cobra.Command) { cmd.Flags().String("output-version", "", "Output the formatted object with the given group version (for ex: 'extensions/v1beta1').") cmd.Flags().Bool("no-headers", false, "When using the default output, don't print headers.") cmd.Flags().Bool("show-labels", false, "When printing, show all labels as the last column (default hide labels column)") - // template shorthand -t is deprecated to support -t for --tty - // TODO: remove template flag shorthand -t - cmd.Flags().StringP("template", "t", "", "Template string or path to template file to use when -o=go-template, -o=go-template-file. The template format is golang templates [http://golang.org/pkg/text/template/#pkg-overview].") + cmd.Flags().String("template", "", "Template string or path to template file to use when -o=go-template, -o=go-template-file. The template format is golang templates [http://golang.org/pkg/text/template/#pkg-overview].") cmd.MarkFlagFilename("template") - cmd.Flags().MarkShorthandDeprecated("template", "please use --template instead") cmd.Flags().String("sort-by", "", "If non-empty, sort list types using this field specification. The field specification is expressed as a JSONPath expression (e.g. '{.metadata.name}'). The field in the API resource specified by this JSONPath expression must be an integer or a string.") cmd.Flags().BoolP("show-all", "a", false, "When printing, show all resources (default hide terminated pods.)") }