github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-28 22:17:14 +00:00
Code Issues Projects Releases Wiki Activity

Assign options.Err in "set image"

Browse Source 
There is a usage of options.Err in a Printf, but this option is never set.
This patch passes the stderr into the command and assigns the option correctly.
This commit is contained in:
Angus Salkeld 2016-10-10 17:01:27 +10:00
parent 01189549f3
commit ea1063a263
3 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
pkg/kubectl/cmd/cmd.go
View File

@ -249,7 +249,7 @@ Find more information at https://github.com/kubernetes/kubernetes.`,
NewCmdCreate(f, out),
NewCmdExposeService(f, out),
NewCmdRun(f, in, out, err),
set.NewCmdSet(f, out),
set.NewCmdSet(f, out, err),
},
},
{

4
pkg/kubectl/cmd/set/set.go
View File

@ -32,7 +32,7 @@ var (
set_example = dedent.Dedent(``)
)
func NewCmdSet(f cmdutil.Factory, out io.Writer) *cobra.Command {
func NewCmdSet(f cmdutil.Factory, out, err io.Writer) *cobra.Command {
cmd := &cobra.Command{
Use: "set SUBCOMMAND",
@ -45,7 +45,7 @@ func NewCmdSet(f cmdutil.Factory, out io.Writer) *cobra.Command {
}
// add subcommands
cmd.AddCommand(NewCmdImage(f, out))
cmd.AddCommand(NewCmdImage(f, out, err))
return cmd
}

3
pkg/kubectl/cmd/set/set_image.go
View File

@ -78,9 +78,10 @@ var (
kubectl set image -f path/to/file.yaml nginx=nginx:1.9.1 --local -o yaml`)
)
func NewCmdImage(f cmdutil.Factory, out io.Writer) *cobra.Command {
func NewCmdImage(f cmdutil.Factory, out, err io.Writer) *cobra.Command {
options := &ImageOptions{
Out: out,
Err: err,
}
cmd := &cobra.Command{