make --output description better, make json the default flag value

Author: jmccormick2001

hugo/content/getting-started/_index.adoc

@@ -57,8 +57,8 @@ $ pgo status [FLAGS]
 |=========================================================
 |Name |Shorthand |Input |Usage
 
-|`--output` |-o |String |
-The output format. Currently, JSON is supported.
+|`--output=json` |-o json|String |
+The output format. Currently, *json* is the only supported value.
 |=========================================================
 
 === Operator Configuration
@@ -405,8 +405,8 @@ $ pgo show cluster NAME|all [FLAGS]
 |=========================================================
 |Name |Shorthand |Input |Usage
 
-|`--output` |-o |String |
-The output format. Currently, JSON is supported.
+|`--output=json` |-o json |String |
+The output format. Currently, *json* is the only supported value.
 
 |`--selector` |-s |String |
 The selector to use for cluster filtering.
@@ -482,8 +482,8 @@ $ pgo test NAME|all [FLAGS]
 |=========================================================
 |Name |Shorthand |Input |Usage
 
-|`--output` |-o |String |
-The output format. Currently, JSON is supported.
+|`--output=json` |-o json |String |
+The output format. Currently, *json* is the only supported value.
 
 |`--selector` |-s |String |
 The selector to use for cluster filtering.

pgo/cmd/show.go

@@ -97,7 +97,7 @@ func init() {
 
 	ShowBackupCmd.Flags().StringVarP(&BackupType, "backup-type", "", util.LABEL_BACKUP_TYPE_BASEBACKUP, "The backup type output to list. Valid choices are pgbasebackup (default) or pgbackrest.")
 	ShowClusterCmd.Flags().StringVarP(&CCPImageTag, "ccp-image-tag", "", "", "Filter the results based on the image tag of the cluster.")
-	ShowClusterCmd.Flags().StringVarP(&OutputFormat, "output", "o", "", "The output format. Currently, JSON is supported.")
+	ShowClusterCmd.Flags().StringVarP(&OutputFormat, "output", "o", "json", "The output format. Currently, json is the only supported value.")
 	ShowClusterCmd.Flags().StringVarP(&Selector, "selector", "s", "", "The selector to use for cluster filtering.")
 	ShowPVCCmd.Flags().StringVarP(&PVCRoot, "pvc-root", "", "", "The PVC directory to list.")
 	ShowScheduleCmd.Flags().StringVarP(&Selector, "selector", "s", "", "The selector to use for cluster filtering.")

pgo/cmd/status.go

@@ -43,7 +43,7 @@ var Summary bool
 func init() {
 	RootCmd.AddCommand(statusCmd)
 
-	statusCmd.Flags().StringVarP(&OutputFormat, "output", "o", "", "The output format. Currently, JSON is supported.")
+	statusCmd.Flags().StringVarP(&OutputFormat, "output", "o", "json", "The output format. Currently, json is the only supported value.")
 
 }
 

pgo/cmd/test.go

@@ -39,7 +39,7 @@ var testCmd = &cobra.Command{
 			fmt.Println(`Error: You must specify the name of the clusters to test.`)
 		} else {
 			if OutputFormat != "" && OutputFormat != "json" {
-				fmt.Println("Error: Only JSON is currently supported for the --output flag value.")
+				fmt.Println("Error: Only 'json' is currently supported for the --output flag value.")
 				os.Exit(2)
 			}
 			showTest(args)
@@ -50,7 +50,7 @@ var testCmd = &cobra.Command{
 func init() {
 	RootCmd.AddCommand(testCmd)
 	testCmd.Flags().StringVarP(&Selector, "selector", "s", "", "The selector to use for cluster filtering.")
-	testCmd.Flags().StringVarP(&OutputFormat, "output", "o", "", "The output format. Currently, JSON is supported.")
+	testCmd.Flags().StringVarP(&OutputFormat, "output", "o", "json", "The output format. Currently, json is the only supported value.")
 }
 
 func showTest(args []string) {