morpheus-cli 4.2.12 → 4.2.17

data/lib/morpheus/cli/security_groups.rb

@@ -156,8 +156,8 @@ class Morpheus::Cli::SecurityGroups
         "Description" => 'description',
         "Scoped Cloud" => lambda {|it| it['zone'] ? it['zone']['name'] : 'All' },
         "Source" => lambda {|it| it['syncSource'] == 'external' ? 'SYNCED' : 'CREATED' },
-        # "Active" => lambda {|it| format_boolean(it['active']) },
         "Visibility" => 'visibility',
+        "Active" => lambda {|it| format_boolean(it['active']) },
         "Tenants" => lambda {|it| it['tenants'] ? it['tenants'].collect {|it| it['name'] }.uniq.sort.join(', ') : '' },
       }
       print_description_list(description_cols, security_group)
@@ -295,6 +295,9 @@ class Morpheus::Cli::SecurityGroups
       opts.on('--visibility [private|public]', String, "Visibility") do |val|
         options['visibility'] = val
       end
+      opts.on('--active [on|off]', String, "Can be used to disable a security group") do |val|
+        options[:options]['active'] = val.to_s == 'on' || val.to_s == 'true' || val.to_s == ''
+      end
       build_common_options(opts, options, [:options, :payload, :json, :dry_run, :remote])
       opts.footer = "Create a security group." + "\n" +
                     "[name] is required. This is the name of the security group."
@@ -469,6 +472,9 @@ class Morpheus::Cli::SecurityGroups
       opts.on('--visibility [private|public]', String, "Visibility") do |val|
         options['visibility'] = val
       end
+      opts.on('--active [on|off]', String, "Can be used to disable a security group") do |val|
+        options[:options]['active'] = val.to_s == 'on' || val.to_s == 'true' || val.to_s == ''
+      end
       build_common_options(opts, options, [:options, :payload, :json, :dry_run, :remote])
       opts.footer = "Update a security group." + "\n" +
                     "[security-group] is required. This is the name or id of the security group."

data/lib/morpheus/cli/service_plans_command.rb

@@ -650,13 +650,13 @@ class Morpheus::Cli::ServicePlanCommand
 
       if !plan
         print_red_alert "Service plan #{args[0]} not found"
-        exit 1
+        return 1
       end
 
-      if plan['active'] == true
-        print_green_success "Service plan #{plan['name']} already actived."
-        return 0
-      end
+      # if plan['active'] == true
+      #   print_green_success "Service plan #{plan['name']} already actived."
+      #   return 0
+      # end
 
       unless options[:yes] || ::Morpheus::Cli::OptionTypes::confirm("Are you sure you would like to activate the service plan '#{plan['name']}'?", options)
         return 9, "aborted command"
@@ -704,13 +704,13 @@ class Morpheus::Cli::ServicePlanCommand
 
       if !plan
         print_red_alert "Service plan #{args[0]} not found"
-        exit 1
+        return 1
       end
 
-      if plan['active'] == false
-        print_green_success "Service plan #{plan['name']} already deactived."
-        return 0
-      end
+      # if plan['active'] == false
+      #   print_green_success "Service plan #{plan['name']} already deactivated."
+      #   return 0
+      # end
 
       unless options[:yes] || ::Morpheus::Cli::OptionTypes::confirm("Are you sure you would like to deactivate the service plan '#{plan['name']}'?", options)
         return 9, "aborted command"

data/lib/morpheus/cli/setup.rb

@@ -281,7 +281,7 @@ EOT
       #print cyan
       #puts "Initializing remote appliance at URL: #{@appliance_url}"
 
-      # Master Account
+      # Master Tenant
       print_h2 "Create Master Tenant", options
       account_option_types = [
         {'fieldName' => 'accountName', 'fieldLabel' => 'Master Tenant Name', 'type' => 'text', 'required' => true, 'defaultValue' => (hub_info ? hub_info['companyName'] : nil), 'description' => 'A unique name for the Master Tenant (account).'},

data/lib/morpheus/cli/shell.rb

@@ -507,16 +507,17 @@ class Morpheus::Cli::Shell
         return 0
 
       elsif ["hello","hi","hey","hola"].include?(input.strip.downcase)
+        user_msg = input.strip.downcase
         # need a logged_in? method already damnit
-        #wallet = @wallet
         wallet = Morpheus::Cli::Credentials.new(@appliance_name, @appliance_url).load_saved_credentials
-        if wallet
-          # my_terminal.echo("#{input} %username!")
-          # todo: this morning|afternoon|evening would be pleasant
-          print "#{input} #{green}#{wallet['username']}#{reset}, how may I #{cyan}help#{reset} you?\n"
+        help_msg = case user_msg
+        when "hola"
+          "¿como puedo ayudarte? tratar #{cyan}help#{reset}"
         else
-          print "#{input}, how may I #{cyan}help#{reset} you?\n"
+          "how may I #{cyan}help#{reset} you?"
         end
+        greeting = "#{user_msg.capitalize}#{wallet ? (' '+green+wallet['username'].to_s+reset) : ''}, #{help_msg}#{reset}"
+        puts greeting
         return 0
       elsif input.strip =~ /^shell\s*/
         # just allow shell to fall through

data/lib/morpheus/cli/subnets_command.rb

@@ -609,7 +609,7 @@ class Morpheus::Cli::SubnetsCommand
     options = {}
     optparse = Morpheus::Cli::OptionParser.new do |opts|
       opts.banner = subcommand_usage("[subnet]")
-      build_common_options(opts, options, [:account, :auto_confirm, :json, :dry_run, :remote])
+      build_common_options(opts, options, [:auto_confirm, :json, :dry_run, :remote])
       opts.footer = "Delete a subnet." + "\n" +
                     "[subnet] is required. This is the name or id of a subnet."
     end

data/lib/morpheus/cli/tenants_command.rb

@@ -41,23 +41,24 @@ class Morpheus::Cli::TenantsCommand
   def list(args)
     options = {}
     optparse = Morpheus::Cli::OptionParser.new do |opts|
-      opts.banner = subcommand_usage()
-      build_common_options(opts, options, [:list, :query, :json, :yaml, :csv, :fields, :dry_run, :remote])
+      opts.banner = subcommand_usage("[search phrase]")
+      build_standard_list_options(opts, options)
       opts.footer = "List tenants."
     end
     optparse.parse!(args)
+    # verify_args!(args:args, optparse:optparse, count:0)
+    options[:phrase] = args.join(" ") if args.count > 0
     connect(options)
-    begin
-      params = {}
-      params.merge!(parse_list_options(options))
-      @accounts_interface.setopts(options)
-      if options[:dry_run]
-        print_dry_run @accounts_interface.dry.list(params)
-        return 0
-      end
-      json_response = @accounts_interface.list(params)
-      render_result = render_with_format(json_response, options, 'accounts')
-      return 0 if render_result
+
+    params = {}
+    params.merge!(parse_list_options(options))
+    @accounts_interface.setopts(options)
+    if options[:dry_run]
+      print_dry_run @accounts_interface.dry.list(params)
+      return 0, nil
+    end
+    json_response = @accounts_interface.list(params)
+    render_response(json_response, options, "accounts") do
       accounts = json_response['accounts']
       title = "Morpheus Tenants"
       subtitles = []
@@ -66,15 +67,13 @@ class Morpheus::Cli::TenantsCommand
       if accounts.empty?
         print cyan,"No tenants found.",reset,"\n"
       else
-        print_accounts_table(accounts)
+        print cyan
+        print as_pretty_table(accounts, list_account_column_definitions, options)
         print_results_pagination(json_response)
       end
       print reset,"\n"
-      return 0
-    rescue RestClient::Exception => e
-      print_rest_exception(e, options)
-      exit 1
     end
+    return 0, nil
   end
 
   def count(args)
@@ -110,112 +109,89 @@ class Morpheus::Cli::TenantsCommand
   def get(args)
     options = {}
     optparse = Morpheus::Cli::OptionParser.new do |opts|
-      opts.banner = subcommand_usage("[name]")
-      build_common_options(opts, options, [:json, :yaml, :csv, :fields, :outfile, :dry_run, :remote])
+      opts.banner = subcommand_usage("[tenant]")
+      build_standard_get_options(opts, options)
+      opts.footer = <<-EOT
+Get details about a tenant (account).
+[tenant] is required. This is the name or id of a tenant. Supports 1-N arguments.
+EOT
     end
     optparse.parse!(args)
-    if args.count != 1
-      raise_command_error "wrong number of arguments, expected 1 and got (#{args.count}) #{args.join(', ')}\n#{optparse}"
-    end
+    verify_args!(args:args, optparse:optparse, min:1)
     connect(options)
-    begin
-      @accounts_interface.setopts(options)
-      if options[:dry_run]
-        if args[0].to_s =~ /\A\d{1,}\Z/
-          print_dry_run @accounts_interface.dry.get(args[0].to_i)
-        else
-          print_dry_run @accounts_interface.dry.list({name:args[0]})
-        end
-        return
+    id_list = parse_id_list(args)
+    return run_command_for_each_arg(id_list) do |arg|
+      _get(arg, options)
+    end
+  end
+
+  def _get(id, options={})
+    args = [id] # heh
+    @accounts_interface.setopts(options)
+    if options[:dry_run]
+      if args[0].to_s =~ /\A\d{1,}\Z/
+        print_dry_run @accounts_interface.dry.get(args[0].to_i)
+      else
+        print_dry_run @accounts_interface.dry.list({name:args[0]})
       end
-      account = find_account_by_name_or_id(args[0])
-      exit 1 if account.nil?
+      return
+    end
+    account = find_account_by_name_or_id(args[0])
+    exit 1 if account.nil?
 
-      json_response = {'account' => account}
-      render_result = render_with_format(json_response, options, 'account')
-      return 0 if render_result
+    json_response = {'account' => account}
+    render_result = render_with_format(json_response, options, 'account')
+    return 0 if render_result
 
-      print_h1 "Tenant Details", [], options
-      
-      description_cols = {
-        "ID" => 'id',
-        "Name" => 'name',
-        "Description" => 'description',
-        "Subdomain" => 'subdomain',
-        "Currency" => 'currency',
-        "Created" => lambda {|it| format_local_dt(it['dateCreated']) },
-        "Updated" => lambda {|it| format_local_dt(it['lastUpdated']) },
-        "Status" => lambda {|it| 
-          status_state = nil
-          if account['active']
-            status_state = "#{green}ACTIVE#{cyan}"
-          else
-            status_state = "#{red}INACTIVE#{cyan}"
-          end
-          status_state
-        },
-      }
-      print_description_list(description_cols, account)
+    print_h1 "Tenant Details", [], options
+    
+    print_description_list(account_column_definitions, account, options)
 
-      print reset,"\n"
-      return 0
-    rescue RestClient::Exception => e
-      print_rest_exception(e, options)
-      exit 1
-    end
+    print reset,"\n"
+    return 0
+    
   end
 
+
   def add(args)
     options = {}
+    params = {}
     optparse = Morpheus::Cli::OptionParser.new do |opts|
-      opts.banner = subcommand_usage("[options]")
+      opts.banner = subcommand_usage("[name]")
       build_option_type_options(opts, options, add_account_option_types)
-      build_common_options(opts, options, [:options, :json, :remote, :dry_run])
+      build_standard_add_options(opts, options)
+      opts.footer = <<-EOT
+Create a new tenant.
+[name] is required. Name
+[role] is required. Base Role name or ID
+EOT
     end
     optparse.parse!(args)
+    verify_args!(args:args, optparse:optparse, min:0, max:2)
+    options[:options]['name'] = args[0] if args[0]
+    #options[:options]['role'] = {'id' => args[1]} if args[1]
     connect(options)
-    begin
-      if args.count > 1
-        raise_command_error "wrong number of arguments. Expected 0-1 and received #{args.count} #{args.join(' ')}\n#{optparse}", args, 127
-        #puts_error  "#{Morpheus::Terminal.angry_prompt}wrong number of arguments. Expected 0-1 and received #{args.count} #{args.join(' ')}\n#{optparse}"
-        #return 127
-      end
-      if args[0]
-        options[:options]['name'] = args[0]
-      end
-      params = Morpheus::Cli::OptionTypes.prompt(add_account_option_types, options[:options], @api_client, options[:params])
-      #puts "parsed params is : #{params.inspect}"
-      account_keys = ['name', 'description', 'currency']
-      account_payload = params.select {|k,v| account_keys.include?(k) }
-      account_payload['currency'] = account_payload['currency'].to_s.empty? ? "USD" : account_payload['currency'].upcase
-      
-      if params['role'].to_s != ''
-        role = find_role_by_name(nil, params['role'])
-        exit 1 if role.nil?
-        account_payload['role'] = {id: role['id']}
-      end
-      payload = {account: account_payload}
-      if options[:dry_run] && options[:json]
-        puts as_json(payload, options)
-        return 0
-      end
-      @accounts_interface.setopts(options)
-      if options[:dry_run]
-        print_dry_run @accounts_interface.dry.create(payload)
-        return
-      end
-      json_response = @accounts_interface.create(payload)
-      if options[:json]
-        print JSON.pretty_generate(json_response)
-        print "\n"
-      else
-        print_green_success "Tenant #{account_payload['name']} added"
-        get([account_payload["name"]])
-      end
-
-    rescue RestClient::Exception => e
-      print_rest_exception(e, options)
-      exit 1
+    
+    object_key = 'account' # 'tenant' someday
+    payload = {}
+    if options[:payload]
+      payload = options[:payload]
+      payload.deep_merge!({object_key => parse_passed_options(options)})
+    else
+      payload.deep_merge!({object_key => parse_passed_options(options)})
+      v_prompt = Morpheus::Cli::OptionTypes.prompt(add_account_option_types, options[:options], @api_client, options[:params])
+      payload.deep_merge!({object_key => v_prompt})
+    end
+    @accounts_interface.setopts(options)
+    if options[:dry_run]
+      print_dry_run @accounts_interface.dry.create(payload)
+      return
+    end
+    json_response = @accounts_interface.create(payload)
+    render_response(json_response, options, object_key) do
+      account = json_response[object_key]
+      print_green_success "Tenant #{account['name']} added"
+      return _get(account["id"], options)
     end
   end
 
@@ -223,68 +199,54 @@ class Morpheus::Cli::TenantsCommand
     options = {}
     params = {}
     optparse = Morpheus::Cli::OptionParser.new do |opts|
-      opts.banner = subcommand_usage("[name] [options]")
-      opts.on('--active [on|off]', String, "Can be used to disable a network") do |val|
-        params['active'] = val.to_s.empty? || val.to_s == 'on' || val.to_s == 'true'
-      end
+      opts.banner = subcommand_usage("[tenant]")
       build_option_type_options(opts, options, update_account_option_types)
-      build_common_options(opts, options, [:options, :json, :remote, :dry_run])
+      opts.on('--active [on|off]', String, "Can be used to disable a tenant") do |val|
+        options[:options]['active'] = val.to_s.empty? || val.to_s == 'on' || val.to_s == 'true'
+      end
+      build_standard_update_options(opts, options)
+      opts.footer = <<-EOT
+Update an existing tenant.
+[tenant] is required. Tenant name or ID
+EOT
     end
     optparse.parse!(args)
-    if args.count < 1
-      print_red_alert "Specify at least one option to update"
-      puts optparse
-      exit 1
-    end
+    verify_args!(args:args, optparse:optparse, count:1)
     connect(options)
-    begin
-      account = find_account_by_name_or_id(args[0])
-      exit 1 if account.nil?
 
-      #params = Morpheus::Cli::OptionTypes.prompt(update_account_option_types, options[:options], @api_client, options[:params])
-      params.deep_merge!(options[:options].reject {|k,v| k.is_a?(Symbol) }) if options[:options]
-
-      if params.empty?
-        puts optparse
-        exit 1
-      end
-
-      #puts "parsed params is : #{params.inspect}"
-      #account_keys = ['name', 'description', 'currency']
-      account_payload = params
-      account_payload['currency'] = account_payload['currency'].upcase unless account_payload['currency'].to_s.empty?
-      
-      if params['role'].to_s != ''
-        role = find_role_by_name(nil, params['role'])
-        exit 1 if role.nil?
-        account_payload['role'] = {id: role['id']}
-      end
-      payload = {account: account_payload}
-      if options[:dry_run] && options[:json]
-        puts as_json(payload, options)
-        return 0
-      end
-      @accounts_interface.setopts(options)
-      if options[:dry_run]
-        print_dry_run @accounts_interface.dry.update(account['id'], payload)
-        return
+    account = find_account_by_name_or_id(args[0])
+    return [1, "account not found"] if account.nil?
+    object_key = 'account' # 'tenant' someday
+    payload = {}
+    if options[:payload]
+      payload = options[:payload]
+      payload.deep_merge!({object_key => parse_passed_options(options)})
+    else
+      payload.deep_merge!({object_key => parse_passed_options(options)})
+      v_prompt = Morpheus::Cli::OptionTypes.prompt(update_account_option_types, options[:options].merge(:no_prompt => true), @api_client, options[:params])
+      payload.deep_merge!({object_key => v_prompt})
+      # remove empty role object.. todo: prompt() or deep_compact! needs to  handle this! 
+      if payload[object_key]['role'] && payload[object_key]['role'].empty?
+        payload[object_key].delete('role')
       end
-      json_response = @accounts_interface.update(account['id'], payload)
-
-      if options[:json]
-        print JSON.pretty_generate(json_response)
-        print "\n"
-      else
-        account_name = account_payload['name'] || account['name']
-        print_green_success "Tenant #{account_name} updated"
-        get([account_name])
+      if payload[object_key].empty?
+        raise_command_error "Specify at least one option to update.\n#{optparse}"
       end
-    rescue RestClient::Exception => e
-      print_rest_exception(e, options)
-      exit 1
+    end
+    @accounts_interface.setopts(options)
+    if options[:dry_run]
+      print_dry_run @accounts_interface.dry.update(account['id'], payload)
+      return
+    end
+    json_response = @accounts_interface.update(account['id'], payload)
+    render_response(json_response, options, object_key) do
+      account = json_response[object_key]
+      print_green_success "Tenant #{account['name']} updated"
+      return _get(account["id"], options)
     end
   end
 
+
   def remove(args)
     options = {}
     optparse = Morpheus::Cli::OptionParser.new do |opts|
@@ -333,13 +295,21 @@ class Morpheus::Cli::TenantsCommand
     [
       {'fieldName' => 'name', 'fieldLabel' => 'Name', 'type' => 'text', 'required' => true, 'displayOrder' => 1},
       {'fieldName' => 'description', 'fieldLabel' => 'Description', 'type' => 'text', 'displayOrder' => 2},
-      {'fieldName' => 'role', 'fieldLabel' => 'Base Role', 'type' => 'text', 'displayOrder' => 3},
-      {'fieldName' => 'currency', 'fieldLabel' => 'Currency', 'type' => 'text', 'displayOrder' => 4}
+      {'fieldContext' => 'role', 'fieldName' => 'id', 'fieldLabel' => 'Base Role', 'type' => 'select', 'optionSource' => lambda { 
+        @roles_interface.list(nil, {roleType:'account'})['roles'].collect {|it|
+          {"name" => (it["authority"] || it["name"]), "value" => it["id"]}
+        }
+      }, 'displayOrder' => 3},
+      {'fieldName' => 'currency', 'fieldLabel' => 'Currency', 'type' => 'text', 'defaultValue' => 'USD', 'displayOrder' => 4}
     ]
   end
 
   def update_account_option_types
-    add_account_option_types
+    list = add_account_option_types()
+    # list = list.reject {|it| ["interval"].include? it['fieldName'] }
+    list.each {|it| it.delete('required') }
+    list.each {|it| it.delete('defaultValue') }
+    list
   end
 
 end