Merge pull request #25 from apigee/issue24

support other built-in roles #24

apiclient/iam.go

@@ -94,6 +94,16 @@ func CreateIAMServiceAccount(name string, iamRole string) (err error) {
 		role = "roles/apigeeconnect.Agent"
 	case "watcher":
 		role = "roles/apigee.runtimeAgent"
+	case "admin":
+		role = "roles/apigee.admin"
+	case "readonly-admin":
+		role = "roles/apigee.readOnlyAdmin"
+	case "api-admin":
+		role = "roles/apigee.apiAdminV2"
+	case "dev-admin":
+		role = "roles/apigee.developerAdmin"
+	case "env-admin":
+		role = "roles/apigee.environmentAdmin"
 	case "all":
 		role = "not-necessary-to-add-this"
 	default:

cmd/iam/create.go

@@ -16,6 +16,7 @@ package iam
 
 import (
 	"fmt"
+	"strings"
 
 	"github.com/apigee/apigeecli/apiclient"
 	"github.com/spf13/cobra"
@@ -53,7 +54,7 @@ func init() {
 	CallCmd.Flags().BoolVarP(&generateName, "gen", "g",
 		false, "Generate account name")
 	CallCmd.Flags().StringVarP(&roleType, "role", "r",
-		"", "IAM Role Type")
+		"", "IAM Role Type, must be one of: "+strings.Join(roles, ", "))
 
 	_ = CallCmd.MarkFlagRequired("prj")
 	_ = CallCmd.MarkFlagRequired("role")

cmd/iam/iam.go

@@ -29,7 +29,8 @@ var Cmd = &cobra.Command{
 var name, projectID, roleType string
 var generateName, wid bool
 
-var roles = []string{"mart", "analytics", "all", "logger", "connect", "cassandra", "watcher", "sync"}
+var roles = []string{"mart", "analytics", "all", "logger", "connect", "cassandra", "watcher",
+	"sync", "admin", "api-admin", "env-admin", "dev-admin", "readonly-admin"}
 
 func init() {
 	Cmd.AddCommand(CallCmd)