Merge pull request #47 from srinandan/nov8

improve docs for cmds
srinandan · Nov 8, 2022 · 16a9dd3 · 16a9dd3
2 parents 0f40c15 + 0fa866c
commit 16a9dd3
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 7 deletions.
diff --git a/cmd/connectors/setadmin.go b/cmd/connectors/setadmin.go
@@ -41,7 +41,7 @@ func init() {
 	SetAdminCmd.Flags().StringVarP(&memberName, "member", "m",
 		"", "Member Name, example Service Account Name")
 	SetAdminCmd.Flags().StringVarP(&memberType, "memberType", "p",
-		"serviceAccount", "memberType must be serviceAccount, user or group")
+		"serviceAccount", "memberType must be serviceAccount, user, or group (default serviceAccount)")
 
 	_ = SetAdminCmd.MarkFlagRequired("name")
 }
diff --git a/cmd/connectors/setcustom.go b/cmd/connectors/setcustom.go
@@ -43,7 +43,7 @@ func init() {
 	SetCustomCmd.Flags().StringVarP(&role, "role", "r",
 		"", "Custom IAM role in the format projects/{project-id}/roles/{role}")
 	SetCustomCmd.Flags().StringVarP(&memberType, "memberType", "p",
-		"serviceAccount", "memberType must be serviceAccount, user or group")
+		"serviceAccount", "memberType must be serviceAccount, user, or group (default serviceAccount)")
 
 	_ = SetCustomCmd.MarkFlagRequired("name")
 	_ = SetCustomCmd.MarkFlagRequired("role")

diff --git a/cmd/connectors/setinvoke.go b/cmd/connectors/setinvoke.go
@@ -22,7 +22,7 @@ import (
 
 // SetInvokeCmd to set admin role
 var SetInvokeCmd = &cobra.Command{
-	Use:   "setadmin",
+	Use:   "setinvoke",
 	Short: "Set Connection Invoke IAM policy on a Connection",
 	Long:  "Set Connection Invoke IAM policy on a Connection",
 	Args: func(cmd *cobra.Command, args []string) (err error) {
@@ -41,7 +41,7 @@ func init() {
 	SetInvokeCmd.Flags().StringVarP(&memberName, "member", "m",
 		"", "Member Name, example Service Account Name")
 	SetInvokeCmd.Flags().StringVarP(&memberType, "memberType", "p",
-		"serviceAccount", "memberType must be serviceAccount, user or group")
+		"serviceAccount", "memberType must be serviceAccount, user, or group (default serviceAccount)")
 
 	_ = SetInvokeCmd.MarkFlagRequired("name")
 }
diff --git a/cmd/connectors/setviewer.go b/cmd/connectors/setviewer.go
@@ -22,7 +22,7 @@ import (
 
 // SetViewerCmd to set viewer role
 var SetViewerCmd = &cobra.Command{
-	Use:   "setadmin",
+	Use:   "setviewer",
 	Short: "Set Connection Viewer IAM policy on a Connection",
 	Long:  "Set Connection Viewer IAM policy on a Connection",
 	Args: func(cmd *cobra.Command, args []string) (err error) {
@@ -41,7 +41,7 @@ func init() {
 	SetViewerCmd.Flags().StringVarP(&memberName, "member", "m",
 		"", "Member Name, example Service Account Name")
 	SetViewerCmd.Flags().StringVarP(&memberType, "memberType", "p",
-		"serviceAccount", "memberType must be serviceAccount, user or group")
+		"serviceAccount", "memberType must be serviceAccount, user, or group (default serviceAccount)")
 
 	_ = SetViewerCmd.MarkFlagRequired("name")
 }
diff --git a/cmd/integrations/execute.go b/cmd/integrations/execute.go
@@ -58,7 +58,8 @@ func init() {
 	ExecuteCmd.Flags().StringVarP(&name, "name", "n",
 		"", "Integration flow name")
 	ExecuteCmd.Flags().StringVarP(&executionFile, "file", "f",
-		"", "Integration flow JSON file path")
+		"", "Integration payload JSON file path. For the payload structure, visit docs at"+
+			" https://cloud.google.com/application-integration/docs/reference/rest/v1/projects.locations.integrations/execute#request-body")
 
 	_ = ExecuteCmd.MarkFlagRequired("name")
 	_ = ExecuteCmd.MarkFlagRequired("file")