From 65ac310f711fa0baee38a6385a330797191e1e51 Mon Sep 17 00:00:00 2001 From: Egor Kovetskiy Date: Wed, 13 Mar 2019 08:46:59 +0300 Subject: [PATCH] rework debug printings --- cmd/tubectl/kube.go | 9 ++------- cmd/tubectl/main.go | 15 ++++++++++++--- cmd/tubectl/task.go | 5 +---- 3 files changed, 15 insertions(+), 14 deletions(-) diff --git a/cmd/tubectl/kube.go b/cmd/tubectl/kube.go index 57130e3..d78a365 100644 --- a/cmd/tubectl/kube.go +++ b/cmd/tubectl/kube.go @@ -3,7 +3,6 @@ package main import ( "encoding/json" "fmt" - "log" "os" "os/exec" "sort" @@ -43,9 +42,7 @@ func requestNamespaces(ctlPath string, params *Params) ([]string, error) { "get", "namespaces", "-o", "json", ) - if debug { - log.Printf(":: %q", args) - } + debugcmd(args) ctx := karma.Describe( "cmdline", @@ -85,9 +82,7 @@ func requestResources(ctlPath string, params *Params) ([]Resource, error) { "get", params.Match.Resource, "-o", "json", ) - if debug { - log.Printf(":: %q", args) - } + debugcmd(args) ctx := karma.Describe( "cmdline", diff --git a/cmd/tubectl/main.go b/cmd/tubectl/main.go index aea2d80..d65a506 100644 --- a/cmd/tubectl/main.go +++ b/cmd/tubectl/main.go @@ -124,9 +124,7 @@ func syscallExec(ctlPath string, params *Params) { args = append(args, arg) } - if debug { - log.Printf(":: %q", args) - } + debugcmd(args) syscall.Exec( ctlPath, @@ -191,3 +189,14 @@ func complete(items []string, query string) string { return partial } + +func debugcmd(args []string) { + if debug { + values := []string{} + for _, arg := range args { + values = append(values, fmt.Sprintf("%q", arg)) + } + + log.Printf("%s", strings.Join(values, " ")) + } +} diff --git a/cmd/tubectl/task.go b/cmd/tubectl/task.go index a5a9c84..dd5b2ef 100644 --- a/cmd/tubectl/task.go +++ b/cmd/tubectl/task.go @@ -3,7 +3,6 @@ package main import ( "fmt" "io" - "log" "os" "os/exec" "sync" @@ -77,9 +76,7 @@ func parallelize(tasks []Task) { } func run(ctlPath string, args []string, writer io.Writer) error { - if debug { - log.Printf(":: %q", append([]string{ctlPath}, args...)) - } + debugcmd(append([]string{ctlPath}, args...)) cmd := exec.Command(ctlPath, args...) cmd.Stdout = writer