morpheus-cli 4.2.8 → 4.2.10

data/lib/morpheus/cli/price_sets_command.rb

@@ -39,30 +39,20 @@ class Morpheus::Cli::PriceSetsCommand
     end
     optparse.parse!(args)
     connect(options)
-    if args.count != 0
-      raise_command_error "wrong number of arguments, expected 0 and got (#{args.count}) #{args}\n#{optparse}"
-      return 1
+    params.merge!(parse_list_options(options))
+    params['phrase'] = args.join(' ') if args.count > 0 && params['phrase'].nil? # pass args as phrase, every list command should do this
+    @price_sets_interface.setopts(options)
+    if options[:dry_run]
+      print_dry_run @price_sets_interface.dry.list(params)
+      return
     end
-
-    begin
-      params.merge!(parse_list_options(options))
-
-      @price_sets_interface.setopts(options)
-      if options[:dry_run]
-        print_dry_run @price_sets_interface.dry.list(params)
-        return
-      end
-      json_response = @price_sets_interface.list(params)
-
-      render_result = render_with_format(json_response, options, 'priceSets')
-      return 0 if render_result
-
+    json_response = @price_sets_interface.list(params)
+    price_sets = json_response['priceSets']
+    render_response(json_response, options, 'priceSets') do
       title = "Morpheus Price Sets"
       subtitles = []
       subtitles += parse_list_subtitles(options)
       print_h1 title, subtitles
-
-      price_sets = json_response['priceSets']
       if price_sets.empty?
         print cyan,"No price sets found.",reset,"\n"
       else
@@ -71,25 +61,24 @@ class Morpheus::Cli::PriceSetsCommand
               id: (it['active'] ? cyan : yellow) + it['id'].to_s,
               name: it['name'],
               active: format_boolean(it['active']),
-              price_unit: it['priceUnit'],
+              priceUnit: it['priceUnit'],
               type: price_set_type_label(it['type']),
               price_count: it['prices'].count.to_s + cyan
           }
         end
         columns = [
-            :id, :name, :active, :price_unit, :type, '# of prices' => :price_count
+            :id, :name, :active, :priceUnit, :type, '# OF PRICES' => :price_count
         ]
         columns.delete(:active) if !params['includeInactive']
-
         print as_pretty_table(rows, columns, options)
         print_results_pagination(json_response)
-        print reset,"\n"
       end
       print reset,"\n"
-      return 0
-    rescue RestClient::Exception => e
-      print_rest_exception(e, options)
-      exit 1
+    end
+    if price_sets.empty?
+      return 1,  "0 price sets found"
+    else
+      return 0, nil
     end
   end
 

data/lib/morpheus/cli/prices_command.rb

@@ -30,35 +30,40 @@ class Morpheus::Cli::PricesCommand
       opts.on('-i', '--include-inactive [on|off]', String, "Can be used to enable / disable inactive filter. Default is on") do |val|
         params['includeInactive'] = val.to_s == 'on' || val.to_s == 'true' || val.to_s == '1' || val.to_s == ''
       end
+      opts.on('--platform PLATFORM', Array, "Filter by platform eg. linux, windows") do |val|
+        params['platform'] = val.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact
+      end
+      opts.on('--price-unit UNIT', Array, "Filter by priceUnit eg. hour, month") do |val|
+        params['priceUnit'] = val.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact
+      end
+      opts.on('--currency CURRENCY', Array, "Filter by currency eg. usd") do |val|
+        params['currency'] = val.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact
+      end
+      opts.on('--price-type TYPE', Array, "Filter by priceType eg. fixed,platform,software,compute,storage,datastore,memory,cores,cpu") do |val|
+        params['priceType'] = val.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact
+      end
       build_common_options(opts, options, [:list, :query, :json, :yaml, :csv, :fields, :dry_run, :remote])
       opts.footer = "List prices."
     end
     optparse.parse!(args)
+    #verify_args!(args:args, optparse:optparse, count:0)
     connect(options)
-    if args.count != 0
-      raise_command_error "wrong number of arguments, expected 0 and got (#{args.count}) #{args}\n#{optparse}"
-      return 1
+    
+    params.merge!(parse_list_options(options))
+    params['phrase'] = args.join(' ') if args.count > 0 && params['phrase'].nil? # pass args as phrase, every list command should do this
+    load_whoami()
+    @prices_interface.setopts(options)
+    if options[:dry_run]
+      print_dry_run @prices_interface.dry.list(params)
+      return
     end
-
-    begin
-      params.merge!(parse_list_options(options))
-
-      @prices_interface.setopts(options)
-      if options[:dry_run]
-        print_dry_run @prices_interface.dry.list(params)
-        return
-      end
-      json_response = @prices_interface.list(params)
-
-      render_result = render_with_format(json_response, options, 'prices')
-      return 0 if render_result
-
+    json_response = @prices_interface.list(params)
+    prices = json_response['prices']
+    render_response(json_response, options, 'prices') do
       title = "Morpheus Prices"
       subtitles = []
       subtitles += parse_list_subtitles(options)
       print_h1 title, subtitles
-
-      prices = json_response['prices']
       if prices.empty?
         print yellow,"No prices found.",reset,"\n"
       else
@@ -77,20 +82,17 @@ class Morpheus::Cli::PricesCommand
           }
         end
         columns = [
-            :id, :name, :active, :priceType, :tenant, :priceUnit, :priceAdjustment, :cost, :markup, :price
+            :id, :name, :active, {'PRICE TYPE' => :priceType}, :tenant, {'PRICE UNIT' => :priceUnit}, {'PRICE ADJUSTMENT' => :priceAdjustment}, :cost, :markup, :price
         ]
         columns.delete(:active) if !params['includeInactive']
 
         print as_pretty_table(rows, columns, options)
         print_results_pagination(json_response)
-        print reset,"\n"
       end
       print reset,"\n"
-      return 0
-    rescue RestClient::Exception => e
-      print_rest_exception(e, options)
-      exit 1
     end
+    return 1,  "0 prices found" if prices.empty?
+    return 0, nil
   end
 
   def get(args)
@@ -552,7 +554,12 @@ class Morpheus::Cli::PricesCommand
   end
 
   def currency_sym(currency)
-    Money::Currency.new((currency || 'usd').to_sym).symbol
+    begin
+      Money::Currency.new((currency.to_s.empty? ? 'usd' : currency).to_sym).symbol
+    rescue
+      # sometimes '' is getting passed in here, so figure that out...
+      Money::Currency.new(:usd).symbol
+    end
   end
 
   def price_prefix(price)
@@ -570,7 +577,7 @@ class Morpheus::Cli::PricesCommand
   end
 
   def price_type_label(type)
-    price_types[type] || type.capitalize
+    price_types[type] || type.to_s.capitalize
   end
 
   def price_types

data/lib/morpheus/cli/processes_command.rb

@@ -4,6 +4,9 @@ require 'morpheus/cli/mixins/processes_helper'
 class Morpheus::Cli::Processes
   include Morpheus::Cli::CliCommand
   include Morpheus::Cli::ProcessesHelper
+  include Morpheus::Cli::ProvisioningHelper
+  include Morpheus::Cli::InfrastructureHelper
+  include Morpheus::Cli::OptionSourceHelper
 
   set_command_name :'process'
 
@@ -16,9 +19,11 @@ class Morpheus::Cli::Processes
     #@appliance_name, @appliance_url = Morpheus::Cli::Remote.active_appliance
   end
 
-  def connect(opts)
-    @api_client = establish_remote_appliance_connection(opts)
+  def connect(options)
+    @api_client = establish_remote_appliance_connection(options)
     @processes_interface = @api_client.processes
+    #@instances_interface = @api_client.instances
+    @clouds_interface = @api_client.clouds
   end
 
   def handle(args)
@@ -42,21 +47,29 @@ class Morpheus::Cli::Processes
         options[:show_output] = true
         options[:details] = true
       end
-      opts.on('--app ID', String, "Limit results to specific app(s).") do |val|
+      opts.on('--app APP', String, "Limit results to specific app(s).") do |val|
         params['appIds'] = val.split(',').collect {|it| it.to_s.strip }.reject { |it| it.empty? }
       end
-      opts.on('--instance ID', String, "Limit results to specific instance(s).") do |val|
+      opts.on('--instance INSTANCE', String, "Limit results to specific instance(s).") do |val|
         params['instanceIds'] = val.split(',').collect {|it| it.to_s.strip }.reject { |it| it.empty? }
       end
-      opts.on('--container ID', String, "Limit results to specific container(s).") do |val|
+      opts.on('--container CONTAINER', String, "Limit results to specific container(s).") do |val|
         params['containerIds'] = val.split(',').collect {|it| it.to_s.strip }.reject { |it| it.empty? }
       end
-      opts.on('--host ID', String, "Limit results to specific host(s).") do |val|
+      opts.on('--host HOST', String, "Limit results to specific host(s).") do |val|
         params['serverIds'] = val.split(',').collect {|it| it.to_s.strip }.reject { |it| it.empty? }
       end
-      opts.on('--cloud ID', String, "Limit results to specific cloud(s).") do |val|
+      opts.on('--server HOST', String, "Limit results to specific servers(s).") do |val|
+        params['serverIds'] = val.split(',').collect {|it| it.to_s.strip }.reject { |it| it.empty? }
+      end
+      opts.add_hidden_option('--server')
+      opts.on('--cloud CLOUD', String, "Limit results to specific cloud(s).") do |val|
         params['zoneIds'] = val.split(',').collect {|it| it.to_s.strip }.reject { |it| it.empty? }
       end
+      opts.on('--user USER', String, "Limit results to user(s).") do |val|
+        #params['userId'] = val.split(',').collect {|it| it.to_s.strip }.reject { |it| it.empty? }
+        options[:user] = val
+      end
       build_common_options(opts, options, [:list, :query, :json, :yaml, :csv, :fields, :dry_run, :remote])
       opts.footer = "List historical processes."
     end
@@ -69,6 +82,67 @@ class Morpheus::Cli::Processes
     connect(options)
     begin
       params.merge!(parse_list_options(options))
+      if params['instanceIds']
+        params['instanceIds'] = params['instanceIds'].collect do |instance_id|
+          if instance_id.to_s =~ /\A\d{1,}\Z/
+            # just allow instance IDs
+            instance_id.to_i
+          else
+            instance = find_instance_by_name_or_id(instance_id)
+            if instance.nil?
+              return 1, "instance not found for '#{instance_id}'" # never happens because find exits
+            end
+            instance['id']
+          end
+        end
+      end
+      if params['serverIds']
+        params['serverIds'] = params['serverIds'].collect do |server_id|
+          if server_id.to_s =~ /\A\d{1,}\Z/
+            # just allow server IDs
+            server_id.to_i
+          else
+            server = find_server_by_name_or_id(server_id)
+            if server.nil?
+              return 1, "server not found for '#{server_id}'" # never happens because find exits
+            end
+            server['id']
+          end
+        end
+      end
+      if params['appIds']
+        params['appIds'] = params['appIds'].collect do |app_id|
+          if app_id.to_s =~ /\A\d{1,}\Z/
+            # just allow app IDs
+            app_id.to_i
+          else
+            app = find_app_by_name_or_id(app_id)
+            if app.nil?
+              return 1, "app not found for '#{app_id}'" # never happens because find exits
+            end
+            app['id']
+          end
+        end
+      end
+      if params['zoneIds']
+        params['zoneIds'] = params['zoneIds'].collect do |zone_id|
+          if zone_id.to_s =~ /\A\d{1,}\Z/
+            # just allow zone IDs
+            zone_id.to_i
+          else
+            zone = find_cloud_by_name_or_id(zone_id)
+            if zone.nil?
+              return 1, "cloud not found for '#{zone_id}'" # never happens because find exits
+            end
+            zone['id']
+          end
+        end
+      end
+      if options[:user]
+        user = find_available_user_option(options[:user])
+        return 1, "user not found by '#{options[:user]}'" if user.nil?
+        params['userId'] = user['id']
+      end
       @processes_interface.setopts(options)
       if options[:dry_run]
         print_dry_run @processes_interface.dry.list(params)

data/lib/morpheus/cli/projects_command.rb

@@ -0,0 +1,607 @@
+require 'morpheus/cli/cli_command'
+
+class Morpheus::Cli::Projects
+  include Morpheus::Cli::CliCommand
+  include Morpheus::Cli::ProvisioningHelper
+  include Morpheus::Cli::OptionSourceHelper
+  # include Morpheus::Cli::InfrastructureHelper
+  # include Morpheus::Cli::AccountsHelper
+
+  register_subcommands :list, :get, :add, :update, :remove,
+                       # :add_tags, :remove_tags,
+                       #:add_instance, :remove_instance
+                       #:add_host, :remove_host
+  
+  def connect(opts)
+    @api_client = establish_remote_appliance_connection(opts)
+    @projects_interface = @api_client.projects
+    @instances_interface = @api_client.instances
+    @servers_interface = @api_client.servers
+    @clouds_interface = @api_client.clouds
+  end
+
+  def handle(args)
+    handle_subcommand(args)
+  end
+
+  def list(args)
+    params = {}
+    options = {}
+    instance_ids, server_ids, cloud_ids = nil, nil, nil
+    optparse = Morpheus::Cli::OptionParser.new do |opts|
+      opts.banner = subcommand_usage()
+      opts.on('--instances [LIST]', Array, "Filter by Instance, comma separated list of instance names or IDs.") do |list|
+        instance_ids = list.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact.uniq
+      end
+      opts.on('--servers [LIST]', Array, "Filter by Server, comma separated list of server (host) names or IDs.") do |list|
+        server_ids = list.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact.uniq
+      end
+      opts.on('--clouds [LIST]', Array, "Filter by Cloud, comma separated list of cloud (zone) names or IDs.") do |list|
+        cloud_ids = list.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact.uniq
+      end
+      # opts.on('--owner [LIST]', Array, "Owner, comma separated list of usernames or IDs.") do |list|
+      #   params['ownerId'] = list.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact.uniq
+      # end
+      build_standard_get_options(opts, options)
+      opts.footer = <<-EOT
+List projects.
+EOT
+    end
+    optparse.parse!(args)
+    #verify_args!(args:args, optparse:optparse, count:0)
+    connect(options)
+    options[:phrase] = args.join(' ') if args.count > 0 && params[:phrase].nil? # pass args as phrase, every list command should do this
+    params.merge!(parse_list_options(options))
+    params['instanceId'] = parse_instance_id_list(instance_ids) if instance_ids
+    # todo server and cloud, missing parse_server_id_list() too
+    params['serverId'] = parse_server_id_list(server_ids) if server_ids
+    params['cloudId'] = parse_cloud_id_list(cloud_ids) if cloud_ids
+    @projects_interface.setopts(options)
+    if options[:dry_run]
+      print_dry_run @projects_interface.dry.list(params)
+      return
+    end
+    json_response = @projects_interface.list(params)
+    projects = json_response['projects']
+    render_response(json_response, options, 'projects') do
+      title = "Morpheus Projects"
+      subtitles = []
+      subtitles += parse_list_subtitles(options)
+      print_h1 title, subtitles
+      if projects.empty?
+        print yellow,"No projects found.",reset,"\n"
+      else
+        print cyan
+        columns = [
+          {"ID" => lambda {|it| it['id'] } },
+          {"NAME" => lambda {|it| it['name'] } },
+          {"DESCRIPTION" => lambda {|it| it['description'] } },
+          {"INSTANCES" => lambda {|it| it['instances'].size rescue '' } },
+          {"SERVERS" => lambda {|it| it['servers'].size rescue '' } },
+          {"CLOUDS" => lambda {|it| it['clouds'].size rescue '' } },
+          {"OWNER" => lambda {|it| it['owner'] ? it['owner']['username'] : '' } },
+          {"DATE CREATED" => lambda {|it| format_local_dt(it['dateCreated']) } },
+          {"LAST UPDATED" => lambda {|it| format_local_dt(it['lastUpdated']) } },
+        ]
+        print as_pretty_table(projects, columns, options)
+        print_results_pagination(json_response)
+      end
+      print reset,"\n"
+    end
+    if projects.empty?
+      return 1, "No projects found"
+    else
+      return 0, nil
+    end
+  end
+
+  def get(args)
+    options = {}
+    optparse = Morpheus::Cli::OptionParser.new do |opts|
+      opts.banner = subcommand_usage("[workflow]")
+      opts.on('--no-content', "Do not display script content." ) do
+        options[:no_content] = true
+      end
+      build_common_options(opts, options, [:json, :yaml, :csv, :fields, :dry_run, :remote])
+      opts.footer = <<-EOT
+Get details about a project.
+[project] is required. This is the name or id of a project.
+EOT
+    end
+    optparse.parse!(args)
+    verify_args!(args:args, optparse:optparse, min:1)
+    connect(options)
+    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)
+    @projects_interface.setopts(options)
+    if options[:dry_run]
+      if id.to_s =~ /\A\d{1,}\Z/
+        print_dry_run @projects_interface.dry.get(id.to_i)
+      else
+        print_dry_run @projects_interface.dry.get({name: id})
+      end
+      return
+    end
+    project = find_project_by_name_or_id(id)
+    exit 1 if project.nil?
+    # refetch it by id
+    json_response = {'project' => project}
+    unless id.to_s =~ /\A\d{1,}\Z/
+      json_response = @projects_interface.get(project['id'])
+    end
+    project = json_response['project']
+    render_response(json_response, options, 'project') do
+      print_h1 "Project Details"
+      print cyan
+      description_cols = {
+        "ID" => 'id',
+        "Name" => 'name',
+        "Description" => 'description',
+        "Tags" => lambda {|it| it['tags'] ? it['tags'].collect {|m| "#{m['name']}: #{m['value']}" }.join(', ') : '' },
+        "Owner" => lambda {|it| it['owner'] ? it['owner']['username'] : '' },
+        "Date Created" => lambda {|it| format_local_dt(it['dateCreated']) },
+        "Last Updated" => lambda {|it| format_local_dt(it['lastUpdated']) }
+      }
+      print_description_list(description_cols, project)
+      
+      project_instances = project["instances"]
+      if project_instances && project_instances.size > 0
+        print_h2 "Instances"
+        print cyan
+        print as_pretty_table(project_instances, [:id, :name], options)
+      end
+
+      project_hosts = project["servers"] || project["hosts"]
+      if project_hosts && project_hosts.size > 0
+        print_h2 "Hosts"
+        print cyan
+        print as_pretty_table(project_hosts, [:id, :name], options)
+      end
+
+      project_clouds = project["clouds"] || project["zones"]
+      if project_clouds && project_clouds.size > 0
+        print_h2 "Clouds"
+        print cyan
+        print as_pretty_table(project_clouds, [:id, :name], options)
+      end
+      print reset,"\n"
+    end
+    return 0, nil
+  end
+
+  def add(args)
+    exit_code, err = 0, nil
+    params, options, payload = {}, {}, {}
+    project_name, description, metadata, instance_ids, server_ids, cloud_ids = nil, nil, nil, nil, nil, nil
+    optparse = Morpheus::Cli::OptionParser.new do |opts|
+      opts.banner = subcommand_usage("[name]")
+      opts.on('--name NAME', String, "Project Name" ) do |val|
+        project_name = val
+      end
+      opts.on('--description NAME', String, "Project Description" ) do |val|
+        description = val
+      end
+      opts.on('--tags TAGS', String, "Tags in the format 'name:value, name:value'") do |val|
+        metadata = val
+      end
+      opts.on('--instances [LIST]', Array, "Instances, comma separated list of instance names or IDs.") do |list|
+        instance_ids = list.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact.uniq
+      end
+      opts.on('--servers [LIST]', Array, "Servers, comma separated list of server (host) names or IDs.") do |list|
+        server_ids = list.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact.uniq
+      end
+      opts.on('--clouds [LIST]', Array, "Clouds, comma separated list of cloud names or IDs.") do |list|
+        cloud_ids = list.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact.uniq
+      end
+      build_standard_add_options(opts, options)
+      opts.footer = <<-EOT
+Create a project.
+[name] is required. This is the name of the new project.
+EOT
+    end
+    optparse.parse!(args)
+    verify_args!(args:args, optparse:optparse, max:1)
+    if args[0]
+      project_name = args[0]
+    end
+    connect(options)
+    exit_code, err = 0, nil
+    # construct payload
+    if options[:payload]
+      payload = options[:payload]
+      payload.deep_merge!({'project' => parse_passed_options(options)})
+    else
+      payload['project'] = {}
+      payload.deep_merge!({'project' => parse_passed_options(options)})
+      payload['project']['name'] = project_name ? project_name : Morpheus::Cli::OptionTypes.prompt([{'fieldName' => 'name', 'fieldLabel' => 'Name', 'type' => 'text', 'required' => true, 'description' => 'Project Name'}], options[:options], @api_client)['name']
+      payload['project']['description'] = description ? description : Morpheus::Cli::OptionTypes.prompt([{'fieldName' => 'description', 'fieldLabel' => 'Description', 'type' => 'text', 'required' => false, 'description' => 'Project Description'}], options[:options], @api_client)['description']
+      # metadata tags
+      if metadata
+        metadata = parse_metadata(metadata)
+        payload['project']['tags'] = metadata if metadata
+      else
+        metadata = prompt_metadata(options)
+        payload['project']['tags'] = metadata if metadata
+      end
+      if instance_ids != nil
+        payload['project']['instances'] = parse_instance_id_list(instance_ids).collect { |it| {'id': it.to_i} }
+      else
+        # could prompt
+      end
+      # --instances
+      if instance_ids
+        payload['project']['instances'] = parse_instance_id_list(instance_ids).collect { |it| {'id': it.to_i} }
+      end
+      # --servers
+      if server_ids
+        payload['project']['servers'] = parse_server_id_list(server_ids).collect { |it| {'id': it.to_i} }
+      end
+      # --clouds
+      if cloud_ids
+        payload['project']['clouds'] = parse_cloud_id_list(cloud_ids).collect { |it| {'id': it.to_i} }
+      end
+    end
+    @projects_interface.setopts(options)
+    if options[:dry_run]
+      print_dry_run @projects_interface.dry.create(payload)
+      return
+    end
+    json_response = @projects_interface.create(payload)
+    project = json_response['project']
+    render_response(json_response, options, 'project') do
+      print_green_success "Project #{project['name']} created"
+      exit_code, err = get([project['id']] + (options[:remote] ? ["-r",options[:remote]] : []))
+    end
+    return exit_code, err
+  end
+
+  def update(args)
+    exit_code, err = 0, nil
+    params, options, payload = {}, {}, {}
+    project_name, description, metadata, add_metadata, remove_metadata = nil, nil, nil, nil, nil
+    instance_ids, server_ids, cloud_ids = nil, nil, nil
+    add_instance_ids, remove_instance_ids = nil, nil
+    add_server_ids, remove_server_ids = nil, nil
+    add_cloud_ids, remove_cloud_ids = nil, nil
+    optparse = Morpheus::Cli::OptionParser.new do |opts|
+      opts.banner = subcommand_usage("[project] [options]")
+      opts.on('--name NAME', String, "Project Name" ) do |val|
+        project_name = val
+      end
+      opts.on('--description NAME', String, "Project Description" ) do |val|
+        description = val
+      end
+      opts.on('--tags [TAGS]', String, "Project Tags in the format 'name:value, name:value'. This replaces all project tags.") do |val|
+        metadata = val ? val : []
+      end
+      opts.on('--add-tags TAGS', String, "Add Project Tags in the format 'name:value, name:value'. This will add/update project tags.") do |val|
+        add_metadata = val
+      end
+      opts.on('--remove-tags TAGS', String, "Remove Project Tags in the format 'name, name:value'. This removes project tags, the :value component is optional and must match if passed.") do |val|
+        remove_metadata = val
+      end
+      opts.on('--instances [LIST]', Array, "Instances, comma separated list of instance names or IDs.") do |list|
+        instance_ids = list ? list.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact.uniq : []
+      end
+      opts.on('--add-instances LIST', Array, "Add Instances, comma separated list of instance names or IDs to add.") do |list|
+        add_instance_ids = list.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact.uniq
+      end
+      opts.on('--remove-instances LIST', Array, "Remove Instances, comma separated list of instance names or IDs to remove.") do |list|
+        remove_instance_ids = list.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact.uniq
+      end
+      opts.on('--servers [LIST]', Array, "Servers, comma separated list of server (host) names or IDs.") do |list|
+        server_ids = list ? list.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact.uniq : []
+      end
+      opts.on('--add-servers LIST', Array, "Add Servers, comma separated list of server names or IDs to add.") do |list|
+        add_server_ids = list.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact.uniq
+      end
+      opts.on('--remove-servers LIST', Array, "Remove Servers, comma separated list of server names or IDs to remove.") do |list|
+        remove_server_ids = list.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact.uniq
+      end
+      opts.on('--clouds [LIST]', Array, "Clouds, comma separated list of cloud names or IDs.") do |list|
+        cloud_ids = list ? list.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact.uniq : []
+      end
+      opts.on('--add-clouds LIST', Array, "Add Clouds, comma separated list of cloud names or IDs to add.") do |list|
+        add_cloud_ids = list.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact.uniq
+      end
+      opts.on('--remove-clouds LIST', Array, "Remove Clouds, comma separated list of cloud names or IDs to remove.") do |list|
+        remove_cloud_ids = list.collect {|it| it.to_s.strip.empty? ? nil : it.to_s.strip }.compact.uniq
+      end
+      build_standard_update_options(opts, options)
+      opts.footer = <<-EOT
+Update a project.
+[project] is required. This is the name or id of a project.
+EOT
+    end
+    optparse.parse!(args)
+    verify_args!(args:args, optparse:optparse, count:1)
+    connect(options)
+    exit_code, err = 0, nil
+    project = find_project_by_name_or_id(args[0])
+    return 1, "project not found by '#{args[0]}'" if project.nil?
+    # construct payload
+    if options[:payload]
+      payload = options[:payload]
+      payload.deep_merge!({'project' => parse_passed_options(options)})
+    else
+      payload['project'] = {}
+      payload.deep_merge!({'project' => parse_passed_options(options)})
+      payload['project']['name'] = project_name if project_name
+      payload['project']['description'] = description if description
+      # metadata tags
+      if metadata
+        payload['project']['tags'] = parse_metadata(metadata)
+      end
+      if add_metadata
+        payload['project']['addTags'] = parse_metadata(add_metadata)
+      end
+      if remove_metadata
+        payload['project']['removeTags'] = parse_metadata(remove_metadata)
+      end
+      # --instances
+      if instance_ids
+        payload['project']['instances'] = parse_instance_id_list(instance_ids).collect { |it| {'id': it.to_i} }
+      end
+      if add_instance_ids
+        payload['project']['addInstances'] = parse_instance_id_list(add_instance_ids).collect { |it| {'id': it.to_i} }
+      end
+      if remove_instance_ids
+        payload['project']['removeInstances'] = parse_instance_id_list(remove_instance_ids).collect { |it| {'id': it.to_i} }
+      end
+      # --servers
+      if server_ids
+        payload['project']['servers'] = parse_server_id_list(server_ids).collect { |it| {'id': it.to_i} }
+      end
+      if add_server_ids
+        payload['project']['addServers'] = parse_server_id_list(add_server_ids).collect { |it| {'id': it.to_i} }
+      end
+      if remove_server_ids
+        payload['project']['removeServers'] = parse_server_id_list(remove_server_ids).collect { |it| {'id': it.to_i} }
+      end
+      # --clouds
+      if cloud_ids
+        cloud_ids = parse_cloud_id_list(cloud_ids)
+        return 1, "clouds not found" if cloud_ids.nil?
+        payload['project']['clouds'] = cloud_ids.collect { |it| {'id': it.to_i} }
+      end
+      if add_cloud_ids
+        add_cloud_ids = parse_cloud_id_list(add_cloud_ids)
+        return 1, "clouds not found" if add_cloud_ids.nil?
+        payload['project']['addClouds'] = add_cloud_ids.collect { |it| {'id': it.to_i} }
+      end
+      if remove_cloud_ids
+        remove_cloud_ids = parse_cloud_id_list(remove_cloud_ids)
+        return 1, "clouds not found" if remove_cloud_ids.nil?
+        payload['project']['removeClouds'] = remove_cloud_ids.collect { |it| {'id': it.to_i} }
+      end
+    end
+    @projects_interface.setopts(options)
+    if options[:dry_run]
+      print_dry_run @projects_interface.dry.update(project['id'], payload)
+      return
+    end
+    json_response = @projects_interface.update(project['id'], payload)
+    project = json_response['project']
+    render_response(json_response, options, 'project') do
+      print_green_success "Project #{project['name']} updated"
+      exit_code, err = get([project['id']] + (options[:remote] ? ["-r",options[:remote]] : []))
+    end
+    return exit_code, err
+  end
+
+  def add_tags(args)
+    params = {}
+    options = {}
+    payload = {}
+    metadata = nil
+    optparse = Morpheus::Cli::OptionParser.new do |opts|
+      opts.banner = subcommand_usage("[project] [tag=value] [tag=value]")
+      opts.on('--tags TAGS', String, "Project Tags in the format 'name:value, name:value'. This will add/update project tags.") do |val|
+        metadata = val
+      end
+      build_standard_update_options(opts, options)
+      opts.footer = <<-EOT
+Add metadata tags to a project.
+[project] is required. This is the name or id of a project.
+[tag=value] is required. This is a metadata tag in the format name=value, 1-N may be passed.
+EOT
+    end
+    optparse.parse!(args)
+    verify_args!(args:args, optparse:optparse, min:1) # maybe min:2 instead ?
+    connect(options)
+    exit_code, err = 0, nil
+    if args.count > 1
+      metadata = args[1..-1].join(" ")
+    end
+    project = find_project_by_name_or_id(args[0])
+    return 1, "project not found by '#{args[0]}'" if project.nil?
+    # construct payload
+
+    if options[:payload]
+      payload = options[:payload]
+      payload.deep_merge!({'project' => parse_passed_options(options)})
+    else
+      payload['project'] = {}
+      payload.deep_merge!({'project' => parse_passed_options(options)})
+      if options[:metadata]
+        payload['project']['addTags'] = parse_metadata(metadata)
+      end
+    end
+    @projects_interface.setopts(options)
+    if options[:dry_run]
+      print_dry_run @projects_interface.dry.update(project['id'], payload)
+      return
+    end
+    json_response = @projects_interface.update(project['id'], payload)
+    project = json_response['project']
+    render_response(json_response, options, 'project') do
+      print_green_success "Project #{project['name']} updated"
+      exit_code, err = get([project['id']] + (options[:remote] ? ["-r",options[:remote]] : []))
+    end
+    return exit_code, err
+  end
+
+  def remove_tags(args)
+    params = {}
+    options = {}
+    payload = {}
+    metadata = nil
+    optparse = Morpheus::Cli::OptionParser.new do |opts|
+      opts.banner = subcommand_usage("[project] [tags]")
+      opts.on('--tags TAGS', String, "Project Tags in the format 'name=value, name=value'. This will add/update project tags.") do |val|
+        metadata = val
+      end
+      build_standard_update_options(opts, options)
+      opts.footer = <<-EOT
+Remove metadata tags from a project.
+[project] is required. This is the name or id of a project.
+[tags] is required. This is a metadata tag in the format tag=value, tag2, tag3, 1-N may be passed, value is optional and must match if passed.
+EOT
+    end
+    optparse.parse!(args)
+    verify_args!(args:args, optparse:optparse, min:1) # maybe min:2 instead ?
+    connect(options)
+    exit_code, err = 0, nil
+    if args.count > 1
+      metadata = args[1..-1].join(" ")
+    end
+    project = find_project_by_name_or_id(args[0])
+    return 1, "project not found by '#{args[0]}'" if project.nil?
+    # construct payload
+
+    if options[:payload]
+      payload = options[:payload]
+      payload.deep_merge!({'project' => parse_passed_options(options)})
+    else
+      payload['project'] = {}
+      payload.deep_merge!({'project' => parse_passed_options(options)})
+      if metadata
+        payload['project']['removeTags'] = parse_metadata(metadata)
+      end
+    end
+    @projects_interface.setopts(options)
+    if options[:dry_run]
+      print_dry_run @projects_interface.dry.update(project['id'], payload)
+      return
+    end
+    json_response = @projects_interface.update(project['id'], payload)
+    project = json_response['project']
+    render_response(json_response, options, 'project') do
+      print_green_success "Project #{project['name']} updated"
+      exit_code, err = get([project['id']] + (options[:remote] ? ["-r",options[:remote]] : []))
+    end
+    return exit_code, err
+  end
+
+  def remove(args)
+    params = {}
+    options = {}
+    optparse = Morpheus::Cli::OptionParser.new do |opts|
+      opts.banner = subcommand_usage("[project]")
+      build_common_options(opts, options, [:auto_confirm, :json, :dry_run, :quiet, :remote])
+      # opts.on( '-f', '--force', "Force Delete" ) do
+      #   params[:force] = true
+      # end
+      opts.footer = <<-EOT
+Delete a project.
+[project] is required. This is the name or id of a project.
+EOT
+    end
+    optparse.parse!(args)
+    verify_args!(args:args, optparse:optparse, count:1)
+    connect(options)
+    exit_code, err = 0, nil
+    project = find_project_by_name_or_id(args[0])
+    return 1, "project not found by '#{args[0]}'" if project.nil?
+    unless options[:yes] || Morpheus::Cli::OptionTypes.confirm("Are you sure you want to delete the project #{project['name']}?")
+      return 9, "aborted command"
+    end
+    @projects_interface.setopts(options)
+    if options[:dry_run]
+      print_dry_run @projects_interface.dry.destroy(project['id'], params)
+      return
+    end
+    json_response = @projects_interface.destroy(project['id'], params)
+    render_response(json_response, options) do
+      print_green_success "Project #{project['name']} removed"
+    end
+    return exit_code, err
+  end
+
+  private
+
+  def find_project_by_name_or_id(val)
+    if val.to_s =~ /\A\d{1,}\Z/
+      return find_project_by_id(val)
+    else
+      return find_project_by_name(val)
+    end
+  end
+
+  def find_project_by_id(id)
+    begin
+      json_response = @projects_interface.get(id.to_i)
+      return json_response['project']
+    rescue RestClient::Exception => e
+      if e.response && e.response.code == 404
+        print_red_alert "Project not found by id #{id}"
+        return nil
+      else
+        raise e
+      end
+    end
+  end
+
+  def find_project_by_name(name)
+    projects = @projects_interface.list({name: name.to_s})['projects']
+    if projects.empty?
+      print_red_alert "Project not found by name #{name}"
+      return nil
+    elsif projects.size > 1
+      print_red_alert "#{projects.size} projects by name #{name}"
+      print_error "\n"
+      puts_error as_pretty_table(projects, [:id, :name], {color:red})
+      print_red_alert "Try passing ID instead"
+      return nil
+    else
+      return projects[0]
+    end
+  end
+
+  def find_project_type_by_name(val)
+    raise "find_project_type_by_name passed a bad name: #{val.inspect}" if val.to_s == ''
+    @all_project_types ||= @projects_interface.list_types({max:1000})['projectTypes']
+
+    if @all_project_types.nil? && !@all_project_types.empty?
+      print_red_alert "No project types found"
+      return nil
+    end
+    matching_project_types = @all_project_types.select { |it| val && (it['name'] == val || it['code'] == val ||  it['id'].to_s == val.to_s) }
+    if matching_project_types.size == 1
+      return matching_project_types[0]
+    elsif matching_project_types.size == 0
+      print_red_alert "Project Type not found by '#{val}'"
+    else
+      print_red_alert "#{matching_project_types.size} project types found by name #{name}"
+      rows = matching_project_types.collect do |it|
+        {id: it['id'], name: it['name'], code: it['code']}
+      end
+      print "\n"
+      puts as_pretty_table(rows, [:name, :code], {color:red})
+      return nil
+    end
+  end
+
+  def update_project_option_types(project_type)
+    [
+      {'fieldName' => 'name', 'fieldLabel' => 'Name', 'type' => 'text', 'required' => true, 'displayOrder' => 0}
+    ] + project_type['optionTypes']
+  end
+
+
+end