show indented json for machine readable output

girishramnani · Oct 9, 2019 · 4e93514 · 4e93514
1 parent 2bf3f38
commit 4e93514
Show file tree

Hide file tree

Showing 10 changed files with 27 additions and 22 deletions.
diff --git a/pkg/machineoutput/types.go b/pkg/machineoutput/types.go
@@ -32,7 +32,7 @@ type GenericSuccess struct {
 
 // OutputSuccess outputs a "successful" machine-readable output format in json
 func OutputSuccess(machineOutput interface{}) {
-	printableOutput, err := json.Marshal(machineOutput)
+	printableOutput, err := MarshalJSONIndented(machineOutput)
 
 	// If we error out... there's no way to output it (since we disable logging when using -o json)
 	if err != nil {
@@ -44,7 +44,7 @@ func OutputSuccess(machineOutput interface{}) {
 
 // OutputError outputs a "successful" machine-readable output format in json
 func OutputError(machineOutput interface{}) {
-	printableOutput, err := json.Marshal(machineOutput)
+	printableOutput, err := MarshalJSONIndented(machineOutput)
 
 	// If we error out... there's no way to output it (since we disable logging when using -o json)
 	if err != nil {
@@ -53,3 +53,8 @@ func OutputError(machineOutput interface{}) {
 		fmt.Fprintf(log.GetStderr(), "%s\n", string(printableOutput))
 	}
 }
+
+// MarshalJSONIndented returns indented json representation of obj
+func MarshalJSONIndented(obj interface{}) ([]byte, error) {
+	return json.MarshalIndent(obj, "", "    ")
+}
diff --git a/pkg/odo/cli/application/describe.go b/pkg/odo/cli/application/describe.go
@@ -1,12 +1,12 @@
 package application
 
 import (
-	"encoding/json"
 	"fmt"
 
 	"github.com/openshift/odo/pkg/application"
 	"github.com/openshift/odo/pkg/component"
 	"github.com/openshift/odo/pkg/log"
+	"github.com/openshift/odo/pkg/machineoutput"
 	"github.com/openshift/odo/pkg/odo/cli/project"
 	"github.com/openshift/odo/pkg/odo/genericclioptions"
 	"github.com/openshift/odo/pkg/odo/util"
@@ -65,7 +65,7 @@ func (o *DescribeOptions) Validate() (err error) {
 func (o *DescribeOptions) Run() (err error) {
 	if log.IsJSON() {
 		appDef := application.GetMachineReadableFormat(o.Client, o.appName, o.Project)
-		out, err := json.Marshal(appDef)
+		out, err := machineoutput.MarshalJSONIndented(appDef)
 		if err != nil {
 			return err
 		}

diff --git a/pkg/odo/cli/application/list.go b/pkg/odo/cli/application/list.go
@@ -1,13 +1,13 @@
 package application
 
 import (
-	"encoding/json"
 	"fmt"
 	"os"
 	"text/tabwriter"
 
 	"github.com/openshift/odo/pkg/application"
 	"github.com/openshift/odo/pkg/log"
+	"github.com/openshift/odo/pkg/machineoutput"
 	"github.com/openshift/odo/pkg/odo/cli/project"
 	"github.com/openshift/odo/pkg/odo/genericclioptions"
 	"github.com/openshift/odo/pkg/odo/util"
@@ -68,7 +68,7 @@ func (o *ListOptions) Run() (err error) {
 			}
 
 			appListDef := application.GetMachineReadableFormatForList(appList)
-			out, err := json.Marshal(appListDef)
+			out, err := machineoutput.MarshalJSONIndented(appListDef)
 			if err != nil {
 				return err
 			}
@@ -91,7 +91,7 @@ func (o *ListOptions) Run() (err error) {
 		}
 	} else {
 		if log.IsJSON() {
-			out, err := json.Marshal(application.GetMachineReadableFormatForList([]application.App{}))
+			out, err := machineoutput.MarshalJSONIndented(application.GetMachineReadableFormatForList([]application.App{}))
 			if err != nil {
 				return err
 			}

diff --git a/pkg/odo/cli/component/describe.go b/pkg/odo/cli/component/describe.go
@@ -1,11 +1,11 @@
 package component
 
 import (
-	"encoding/json"
 	"fmt"
 
 	"github.com/openshift/odo/pkg/config"
 	"github.com/openshift/odo/pkg/log"
+	"github.com/openshift/odo/pkg/machineoutput"
 	"github.com/openshift/odo/pkg/odo/genericclioptions"
 
 	"github.com/openshift/odo/pkg/component"
@@ -70,7 +70,7 @@ func (do *DescribeOptions) Run() (err error) {
 	}
 	if log.IsJSON() {
 		componentDesc.Spec.Ports = do.localConfigInfo.GetPorts()
-		out, err := json.Marshal(componentDesc)
+		out, err := machineoutput.MarshalJSONIndented(componentDesc)
 		if err != nil {
 			return err
 		}

diff --git a/pkg/odo/cli/component/list.go b/pkg/odo/cli/component/list.go
@@ -1,14 +1,14 @@
 package component
 
 import (
-	"encoding/json"
 	"fmt"
 	"os"
 	"path/filepath"
 	"text/tabwriter"
 
 	"github.com/golang/glog"
 	"github.com/openshift/odo/pkg/application"
+	"github.com/openshift/odo/pkg/machineoutput"
 	"github.com/pkg/errors"
 	"github.com/spf13/cobra"
 
@@ -67,7 +67,7 @@ func (lo *ListOptions) Run() (err error) {
 			return err
 		}
 		if log.IsJSON() {
-			out, err := json.Marshal(components)
+			out, err := machineoutput.MarshalJSONIndented(components)
 			if err != nil {
 				return err
 			}
@@ -123,7 +123,7 @@ func (lo *ListOptions) Run() (err error) {
 
 	if log.IsJSON() {
 
-		out, err := json.Marshal(components)
+		out, err := machineoutput.MarshalJSONIndented(components)
 		if err != nil {
 			return err
 		}

diff --git a/pkg/odo/cli/project/list.go b/pkg/odo/cli/project/list.go
@@ -1,12 +1,12 @@
 package project
 
 import (
-	"encoding/json"
 	"fmt"
 	"os"
 	"text/tabwriter"
 
 	"github.com/openshift/odo/pkg/log"
+	"github.com/openshift/odo/pkg/machineoutput"
 	"github.com/openshift/odo/pkg/odo/genericclioptions"
 	"github.com/openshift/odo/pkg/project"
 	"github.com/spf13/cobra"
@@ -53,7 +53,7 @@ func (plo *ProjectListOptions) Run() (err error) {
 		return err
 	}
 	if log.IsJSON() {
-		out, err := json.Marshal(projects)
+		out, err := machineoutput.MarshalJSONIndented(projects)
 		if err != nil {
 			return err
 		}

diff --git a/pkg/odo/cli/storage/create.go b/pkg/odo/cli/storage/create.go
@@ -1,11 +1,11 @@
 package storage
 
 import (
-	"encoding/json"
 	"fmt"
 
 	"github.com/openshift/odo/pkg/config"
 	"github.com/openshift/odo/pkg/log"
+	"github.com/openshift/odo/pkg/machineoutput"
 	"github.com/openshift/odo/pkg/odo/genericclioptions"
 	"github.com/openshift/odo/pkg/odo/util/completion"
 	"github.com/openshift/odo/pkg/storage"
@@ -70,7 +70,7 @@ func (o *StorageCreateOptions) Run() (err error) {
 	storageResultMachineReadable := storage.GetMachineReadableFormat(storageResult.Name, storageResult.Size, storageResult.Path)
 
 	if log.IsJSON() {
-		out, err := json.Marshal(storageResultMachineReadable)
+		out, err := machineoutput.MarshalJSONIndented(storageResultMachineReadable)
 		if err != nil {
 			return err
 		}

diff --git a/pkg/odo/cli/storage/list.go b/pkg/odo/cli/storage/list.go
@@ -1,11 +1,11 @@
 package storage
 
 import (
-	"encoding/json"
 	"fmt"
 	"os"
 
 	"github.com/openshift/odo/pkg/config"
+	"github.com/openshift/odo/pkg/machineoutput"
 	"github.com/openshift/odo/pkg/odo/util/completion"
 	"github.com/openshift/odo/pkg/storage"
 
@@ -70,7 +70,7 @@ func (o *StorageListOptions) Run() (err error) {
 	storageListResultMachineReadable := storage.GetMachineReadableFormatForList(storageListMachineReadable)
 
 	if log.IsJSON() {
-		out, err := json.Marshal(storageListResultMachineReadable)
+		out, err := machineoutput.MarshalJSONIndented(storageListResultMachineReadable)
 		if err != nil {
 			return err
 		}

diff --git a/pkg/odo/cli/url/list.go b/pkg/odo/cli/url/list.go
@@ -1,13 +1,13 @@
 package url
 
 import (
-	"encoding/json"
 	"fmt"
 	"os"
 	"text/tabwriter"
 
 	"github.com/openshift/odo/pkg/config"
 	"github.com/openshift/odo/pkg/log"
+	"github.com/openshift/odo/pkg/machineoutput"
 	"github.com/openshift/odo/pkg/odo/genericclioptions"
 	"github.com/openshift/odo/pkg/odo/util"
 	"github.com/openshift/odo/pkg/odo/util/completion"
@@ -63,7 +63,7 @@ func (o *URLListOptions) Run() (err error) {
 	}
 
 	if log.IsJSON() {
-		out, err := json.Marshal(urls)
+		out, err := machineoutput.MarshalJSONIndented(urls)
 		if err != nil {
 			return err
 		}

diff --git a/pkg/util/util.go b/pkg/util/util.go
@@ -2,7 +2,6 @@ package util
 
 import (
 	"bufio"
-	"encoding/json"
 	"fmt"
 	"io"
 	"math/rand"
@@ -21,6 +20,7 @@ import (
 
 	"github.com/gobwas/glob"
 	"github.com/golang/glog"
+	"github.com/openshift/odo/pkg/machineoutput"
 	"github.com/pkg/errors"
 	corev1 "k8s.io/api/core/v1"
 	"k8s.io/apimachinery/pkg/api/resource"
@@ -590,7 +590,7 @@ func MachineOutput(outputFlag string, resource interface{}) (string, error) {
 	switch outputFlag {
 	case "json":
 		// If `-o json` is provided
-		out, err = json.Marshal(resource)
+		out, err = machineoutput.MarshalJSONIndented(resource)
 	}
 
 	return string(out), err