morpheus-cli 8.0.0 → 8.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 49afeaaa11ed794659cc20a46d3fc145990269dc96f1394a1cd95160bbc2f786
-  data.tar.gz: cdcf88755b1638f3d1816dde28dcfea8f8c6be27b9300bdfc3c6e4798d3937dd
+  metadata.gz: 06ea65156f0a57e7af971494347aae4844608a6cb877087dc9394a127841a75e
+  data.tar.gz: a93ddec1dc9e647e23a111a168126dbfc3b683d04844731f5bc53a461305b140
 SHA512:
-  metadata.gz: 56f27f9ab0323061846e4b3df32d570efff1b944083fe27700a9d729540ad605a1a39a5c2005cc5a0e19880d5f039a7aa3dbc12ff2c31c9b511426ec736f83a7
-  data.tar.gz: 7d98ac1e35e7929f3f73ff3968d3a2d1221629d1a2c07d7cdd916b2aa184ca2d30041eaeaf2ac5d53c653650a0a36e0c39a33bebbb9bf02bdcf3e4f84e5173c8
+  metadata.gz: 9ca9b136bd006211eed46cd89103c2e7af9157d8111595f4e90b09a7a1e014570405256d64401c78311503e3d4b0aaecf5690526b2014985d79c515e0093eba3
+  data.tar.gz: f1355c6ec99b7a05e068e215ba4581a75a712bef545b661a04e7a31ad0b1711071d7f65c316e938171da60d66d0a297898068f5efb70e750a7dc987b2f12014e

data/Dockerfile

@@ -1,5 +1,5 @@
 FROM ruby:2.7.5
 
-RUN gem install morpheus-cli -v 8.0.0
+RUN gem install morpheus-cli -v 8.0.2
 
 ENTRYPOINT ["morpheus"]

data/lib/morpheus/api/appliance_settings_interface.rb

@@ -25,6 +25,13 @@ class Morpheus::ApplianceSettingsInterface < Morpheus::APIClient
     execute(opts)
   end
 
+  def locales(params={})
+    url = "#{base_path}/locales"
+    headers = { params: params, authorization: "Bearer #{@access_token}" }
+    opts = {method: :get, url: url, headers: headers}
+    execute(opts)
+  end
+
   def maintenance(params={}, payload={})
     url = "#{base_path}/maintenance"
     headers = { params: params, authorization: "Bearer #{@access_token}" }

data/lib/morpheus/api/virtual_images_interface.rb

@@ -44,6 +44,12 @@ class Morpheus::VirtualImagesInterface < Morpheus::APIClient
     execute(method: :put, url: url, headers: headers, payload: payload.to_json)
   end
 
+  def convert(id, payload)
+    url = "#{@base_url}/api/virtual-images/#{id}/convert"
+    headers = { :authorization => "Bearer #{@access_token}", 'Content-Type' => 'application/json' }
+    execute(method: :post, url: url, headers: headers, payload: payload.to_json)
+  end
+
   def destroy(id, params={})
     url = "#{@base_url}/api/virtual-images/#{id}"
     headers = { :authorization => "Bearer #{@access_token}", 'Content-Type' => 'application/json' }

data/lib/morpheus/cli/cli_command.rb

@@ -1278,7 +1278,7 @@ module Morpheus
         elsif options[:remote_url]
           credentials = Morpheus::Cli::Credentials.new(@appliance_name, @appliance_url)
           unless options[:skip_login]
-            @wallet = credentials.request_credentials(options, @do_save_credentials)
+            @wallet = credentials.request_credentials(options.merge({dry_run:false}), @do_save_credentials)
           end
         else
           credentials = Morpheus::Cli::Credentials.new(@appliance_name, @appliance_url)
@@ -1293,7 +1293,7 @@ module Morpheus
           
           if @wallet.nil? || @wallet['access_token'].nil?
             unless options[:skip_login]
-              @wallet = credentials.request_credentials(options, @do_save_credentials)
+              @wallet = credentials.request_credentials(options.merge({dry_run:false}), @do_save_credentials)
             end
           end
           
@@ -1767,7 +1767,22 @@ module Morpheus
           output = records.collect { |record| 
             options[:select_fields].collect { |field| 
               value = get_object_value(record, field)
-              value.is_a?(String) ? value : JSON.fast_generate(value)
+              if value.is_a?(String)
+                value
+              else
+                if options[:json]
+                  as_json(value, options)
+                elsif options[:yaml]
+                  output = as_yaml(value, options)
+                elsif options[:csv]
+                  as_csv(value, nil, options)
+                else
+                  # default behavior
+                  # value.is_a?(String) ? value : as_json(value, options)
+                  do_pretty = options.key?(:pretty_json) ? options[:pretty_json] : false # or true?
+                  do_pretty ? JSON.pretty_generate(value) : JSON.fast_generate(value)
+                end
+              end
             }.join(options[:delim] || ",")
           }.join(options[:newline] || "\n")
         elsif options[:json]

data/lib/morpheus/cli/commands/appliance_settings_command.rb

@@ -7,7 +7,7 @@ class Morpheus::Cli::ApplianceSettingsCommand
   set_command_name :'appliance-settings'
 
   register_subcommands :get, :update, :toggle_maintenance, :'reindex'
-  
+
   set_default_subcommand :get
 
   def connect(opts)
@@ -33,7 +33,7 @@ class Morpheus::Cli::ApplianceSettingsCommand
       raise_command_error "wrong number of arguments, expected 0 and got (#{args.count}) #{args}\n#{optparse}"
       return 1
     end
-    
+
     begin
       @appliance_settings_interface.setopts(options)
 
@@ -220,6 +220,25 @@ class Morpheus::Cli::ApplianceSettingsCommand
       opts.on("--stats-retainment-period DAYS", Integer, "Stats retainment period. The number of days stats should be available. Can be 30, 60, or 90.") do |val|
         params['statsRetainmentPeriod'] = val.to_i
       end
+      opts.on("--cloud-sync-interval-seconds SECONDS", Integer, "Cloud sync interval in seconds") do |val|
+        params['cloudSyncIntervalSeconds'] = val.to_i
+      end
+      opts.on("--cluster-sync-interval-seconds SECONDS", Integer, "Cluster sync interval in seconds") do |val|
+        params['clusterSyncIntervalSeconds'] = val.to_i
+      end
+      opts.on("--usage-retainment-period DAYS", Integer, "Retainment period for usage records") do |val|
+        params['usageRetainmentPeriod'] = val.to_i
+      end
+      opts.on("--invoice-retainment-period DAYS", Integer, "Retainment period for invoice records") do |val|
+        params['invoiceRetainmentPeriod'] = val.to_i
+      end
+      opts.on("--max-option-list-size NUMBER", Integer, "Max option list size (x10^3)") do |val|
+        params['maxOptionListSize'] = val.to_i
+      end
+      opts.on("--default-locale STRING", String, "Default locale for the appliance") do |val|
+        params['defaultLocale'] = val == 'null' ? nil : val
+      end
+
       build_common_options(opts, options, [:json, :payload, :dry_run, :quiet, :remote])
     end
 

data/lib/morpheus/cli/commands/apps.rb

@@ -506,7 +506,7 @@ class Morpheus::Cli::Apps
               # prompt for cloudFormation config
               # todo
             else
-              print yellow, "Unknown template type: #{template_type})", "\n"
+              print yellow, "Unknown template type: #{blueprint_type})", "\n"
             end
           end
         end

data/lib/morpheus/cli/commands/clouds.rb

@@ -1183,7 +1183,7 @@ EOT
 
     # Details (zoneType.optionTypes)
 
-    if cloud_type && cloud_type['optionTypes']
+    if cloud_type && cloud_type['optionTypes'] && cloud_type['code'] != 'standard'
       if !cloud_type['optionTypes'].find {|opt| opt['type'] == 'credential'}
         tmp_option_types << {'fieldName' => 'type', 'fieldLabel' => 'Credentials', 'type' => 'credential', 'optionSource' => 'credentials', 'required' => true, 'defaultValue' => 'local', 'config' => {'credentialTypes' => get_cloud_type_credential_types(cloud_type['code'])}, 'displayOrder' => 7}
         cloud_type['optionTypes'].select {|opt| ['username', 'password', 'serviceUsername', 'servicePassword'].include?(opt['fieldName'])}.each {|opt| opt['localCredential'] = true}

data/lib/morpheus/cli/commands/clusters.rb

@@ -6,6 +6,7 @@ class Morpheus::Cli::Clusters
   include Morpheus::Cli::ProcessesHelper
   include Morpheus::Cli::WhoamiHelper
   include Morpheus::Cli::AccountsHelper
+  include Morpheus::Cli::ExecutionRequestHelper
 
   register_subcommands :list, :count, :get, :view, :add, :update, :remove, :logs, :history, {:'history-details' => :history_details}, {:'history-event' => :history_event_details}
   register_subcommands :list_workers, :add_worker, :remove_worker, :update_worker_count
@@ -607,7 +608,7 @@ class Morpheus::Cli::Clusters
         end
 
         # Controller type
-        server_types = @server_types_interface.list({computeTypeId: cluster_type['controllerTypes'].first['id'], zoneTypeId: cloud['zoneType']['id'], useZoneProvisionTypes: true})['serverTypes'].reject {|it| it['provisionType']['code'] == 'manual'}
+        server_types = @server_types_interface.list({computeTypeId: cluster_type['controllerTypes'].first['id'], zoneTypeId: cloud['zoneType']['id'], useZoneProvisionTypes: true})['serverTypes'].reject {|it| it['provisionType']['code'] == 'manual'} unless ['kubernetes-cluster', 'mvm-cluster'].include?(cluster_type_code)
         controller_provision_type = nil
         resource_pool = nil
 
@@ -802,6 +803,9 @@ class Morpheus::Cli::Clusters
       opts.on('--managed [on|off]', String, "Can be used to enable / disable managed cluster. Default is on") do |val|
         options[:managed] = val.to_s == 'on' || val.to_s == 'true' || val.to_s == '1' || val.to_s == ''
       end
+      opts.on('--useAgent [on|off]', String, "Use the Agent to relay communications for the Kubernetes API instead of direct.") do |val|
+        options[:useAgent] = val.to_s
+      end
       opts.on('--autoRecoverPowerState [on|off]', String, "Automatically Power On VMs") do |val|
         options[:autoRecoverPowerState] = val.to_s == 'on' || val.to_s == 'true' || val.to_s == '1' || val.to_s == ''
       end
@@ -855,6 +859,7 @@ class Morpheus::Cli::Clusters
         cluster_payload['refresh'] = options[:refresh] if options[:refresh] == true
         cluster_payload['tenant'] = options[:tenant] if !options[:tenant].nil?
         cluster_payload['integrations'] = options[:integrations] if !options[:integrations].nil?
+        cluster_payload['useAgent'] = options[:useAgent] if !options[:useAgent].nil?
         payload = {"cluster" => cluster_payload}
       end
 
@@ -3004,6 +3009,12 @@ class Morpheus::Cli::Clusters
     options = {}
     optparse = Morpheus::Cli::OptionParser.new do |opts|
       opts.banner = subcommand_usage( "[cluster] [options]")
+      opts.on('--refresh [SECONDS]', String, "Refresh until execution is complete. Default interval is #{default_refresh_interval} seconds.") do |val|
+        options[:refresh_interval] = val.to_s.empty? ? default_refresh_interval : val.to_f
+      end
+      opts.on(nil, '--no-refresh', "Do not refresh" ) do
+        options[:no_refresh] = true
+      end
       build_option_type_options(opts, options, add_datastore_option_types)
       build_common_options(opts, options, [:options, :payload, :json, :dry_run, :remote])
       opts.footer = "Add datastore to a cluster.\n" +
@@ -3059,11 +3070,19 @@ class Morpheus::Cli::Clusters
       json_response = @clusters_interface.create_datastore(cluster['id'], payload)
       if options[:json]
         puts as_json(json_response)
-      elsif json_response['success']
-        if json_response['msg'] == nil
-          print_green_success "Added datastore to cluster #{cluster['name']}"
+      else
+        if json_response['success']
+          if json_response['msg'] == nil
+            print_green_success "Adding datastore to cluster #{cluster['name']}"
+          else
+            print_green_success json_response['msg']
+          end
+          execution_id = json_response['executionId']
+          if !options[:no_refresh] && execution_id
+            wait_for_execution_request(json_response['executionId'], options.merge({waiting_status:['new', 'pending', 'executing']}))
+          end
         else
-          print_green_success json_response['msg']
+          print_red_alert "Failed to create cluster datastore #{json_response['msg']}"
         end
       end
       return 0
@@ -3074,6 +3093,7 @@ class Morpheus::Cli::Clusters
   end
 
   def remove_datastore(args)
+    default_refresh_interval = 5
     params = {}
     options = {}
     optparse = Morpheus::Cli::OptionParser.new do |opts|
@@ -3081,6 +3101,12 @@ class Morpheus::Cli::Clusters
       opts.on( '-f', '--force', "Force Delete" ) do
         params[:force] = 'on'
       end
+      opts.on('--refresh [SECONDS]', String, "Refresh until execution is complete. Default interval is #{default_refresh_interval} seconds.") do |val|
+        options[:refresh_interval] = val.to_s.empty? ? default_refresh_interval : val.to_f
+      end
+      opts.on(nil, '--no-refresh', "Do not refresh" ) do
+        options[:no_refresh] = true
+      end
       build_standard_remove_options(opts, options)
       opts.footer = "Delete a datastore from a cluster.\n" +
         "[cluster] is required. This is the name or id of an existing cluster.\n" +
@@ -3114,12 +3140,18 @@ class Morpheus::Cli::Clusters
       return
     end
     json_response = @clusters_interface.destroy_datastore(cluster['id'], datastore['id'], params)
-    render_response(json_response, options) do
-      msg = "Datastore #{datastore['name']} is being removed from cluster #{cluster['name']}..."
-      if json_response['msg']
-        msg = json_response['msg']
+    if options[:json]
+      puts as_json(json_response)
+    else
+      if json_response['success']
+        print_green_success "Datastore #{datastore['name']} is being removed from cluster #{cluster['name']}"
+        execution_id = json_response['executionId']
+        if !options[:no_refresh] && execution_id
+          wait_for_execution_request(execution_id, options.merge({waiting_status:['new', 'pending', 'executing']}))
+        end
+      else
+        print_red_alert "Failed to remove cluster datastore #{json_response['msg']}"
       end
-      print_green_success msg
     end
     return 0, nil
   end
@@ -4535,7 +4567,12 @@ class Morpheus::Cli::Clusters
         elsif resource_pool_options.count > 1 && !options[:no_prompt]
           resource_pool_id = Morpheus::Cli::OptionTypes.prompt([{'fieldName' => 'resourcePool', 'type' => 'select', 'fieldLabel' => 'Resource Pool', 'selectOptions' => resource_pool_options, 'required' => true, 'skipSingleOption' => true, 'description' => 'Select resource pool.'}],options[:options],api_client, {})['resourcePool']
         else
-          resource_pool_id = resource_pool_options.first['id']
+          first_option = resource_pool_options.find {|it| !it['id'].nil? }
+          if first_option.nil?
+            print_red_alert "Cloud #{cloud['name']} has no available resource pools"
+            exit 1
+          end
+          resource_pool_id = first_option['id']
         end
         if resource_pool_id.to_s["poolGroup-"]
           resource_pool = @resource_pool_groups_interface.get(resource_pool_id)['resourcePoolGroup']

data/lib/morpheus/cli/commands/hosts.rb

@@ -1337,14 +1337,20 @@ class Morpheus::Cli::Hosts
       payload[:server][:plan] = {id: service_plan["id"]}
 
       # fetch volumes
-      volumes_response = @servers_interface.volumes(server['id'])
-      current_volumes = volumes_response['volumes'].sort {|x,y| x['displayOrder'] <=> y['displayOrder'] }
+      current_volumes = nil
+      if server['volumes']
+        current_volumes = server['volumes'].sort {|x,y| x['displayOrder'] <=> y['displayOrder'] }
+      else
+        volumes_response = @servers_interface.volumes(server['id'])
+        current_volumes = volumes_response['volumes'].sort {|x,y| x['displayOrder'] <=> y['displayOrder'] }
+      end
 
       # prompt for volumes
       vol_options = options 
       vol_options['siteId'] = group_id
       vol_options['zoneId'] = cloud_id
-      volumes = prompt_resize_volumes(current_volumes, service_plan, provision_type, vol_options)
+      vol_options['resourcePoolId'] = server['resourcePool']['id'] if server['resourcePool']
+      volumes = prompt_resize_volumes(current_volumes, service_plan, provision_type, vol_options, server)
       if !volumes.empty?
         payload[:volumes] = volumes
       end

data/lib/morpheus/cli/commands/instances.rb

@@ -2756,7 +2756,10 @@ class Morpheus::Cli::Instances
     connect(options)
     instance = find_instance_by_name_or_id(args[0])
     return 1, "instance not found" if instance.nil?
-    
+    # need to load full instance details in case fetched by name
+    if instance['containerDetails'].nil?
+      instance = find_instance_by_id(instance['id'])
+    end
     payload = {}
     if options[:payload]
       payload = options[:payload]
@@ -2811,13 +2814,24 @@ class Morpheus::Cli::Instances
       payload["instance"]["plan"] = {"id" => service_plan["id"]}
 
       volumes_response = @instances_interface.volumes(instance['id'])
-      current_volumes = volumes_response['volumes'].sort {|x,y| x['displayOrder'] <=> y['displayOrder'] }
+      current_volumes = nil
 
       # prompt for volumes
       vol_options = options 
       vol_options['siteId'] = group_id
       vol_options['zoneId'] = cloud_id
-      volumes = prompt_resize_volumes(current_volumes, service_plan, provision_type, vol_options)
+      vol_options['resourcePoolId'] = resource_pool_id  # server['resourcePool']['id'] if server['resourcePool']
+      server = instance['containerDetails'][0]['server'] rescue nil
+      if server.nil?
+        Morpheus::Logging::DarkPrinter.puts "Failed to load server info"
+        volumes_response = @instances_interface.volumes(instance['id'])
+        current_volumes = volumes_response['volumes'].sort {|x,y| x['displayOrder'] <=> y['displayOrder'] }
+      else
+         # or just use instance['volumes'] ?
+        current_volumes = server['volumes'].sort {|x,y| x['displayOrder'] <=> y['displayOrder'] }
+      end
+      vol_options['resourcePoolId'] = server['resourcePool']['id'] if server && server['resourcePool']
+      volumes = prompt_resize_volumes(current_volumes, service_plan, provision_type, vol_options, server)
       if !volumes.empty?
         payload["volumes"] = volumes
       end

data/lib/morpheus/cli/commands/library_cluster_layouts_command.rb

@@ -845,7 +845,7 @@ class Morpheus::Cli::LibraryClusterLayoutsCommand
   end
 
   def find_layout_by_name(name)
-    layouts = @library_cluster_layouts_interface.list(instance_type_id, {name: name.to_s})['layouts']
+    layouts = @library_cluster_layouts_interface.list({name: name.to_s})['layouts']
     if layouts.empty?
       print_red_alert "Cluster layout not found by name #{name}"
       return nil

data/lib/morpheus/cli/commands/shell.rb

@@ -635,8 +635,8 @@ class Morpheus::Cli::Shell
   # @return Hash like {:commands => [], :command_count => total}
   def load_history_commands(options={})
     phrase = options[:phrase]
-    sort_key = options[:sort] ? options[:sort].to_sym : nil
-    
+    #sort_key = options[:sort] ? options[:sort].to_sym : nil
+    #direction = options[:direction] # default sort is reversed to get newest first
     offset = options[:offset].to_i > 0 ? options[:offset].to_i : 0
     max = options[:max].to_i > 0 ? options[:max].to_i : 25
     
@@ -648,40 +648,21 @@ class Morpheus::Cli::Shell
     # collect records as [{:number => 1, :command => "instances list"}, etc]
     history_records = []
     history_count = 0
-    
-    # sort is a bit different for this command, the default sort is by number
-    # it sorts oldest -> newest, but shows the very last page by default.
-    if options[:sort] && ![:number, :command].include?(options[:sort])
-      sort_key = :number # nil
-    end
 
-    if options[:phrase] || sort_key || options[:direction] || options[:offset]
-      # this could be a large object...need to index our shell_history file lol
-      sort_key ||= :number
-      history_records = @history.keys.collect { |k| {number: k, command: @history[k]} }
-      if options[:direction] == 'desc'
-        history_records = history_records.sort {|x,y| y[sort_key] <=> x[sort_key] }
-      else
-        history_records = history_records.sort {|x,y| x[sort_key] <=> y[sort_key] }
-      end
-      if phrase
-        history_records = history_records.select {|it| it[:command].include?(phrase) || it[:number].to_s == phrase }
-      end
-      command_count = history_records.size
-      history_records = history_records[offset..(max-1)]
-    else
-      # default should try to be as fast as possible..
-      # no searching or sorting, default order by :number works
-      if offset != 0
-        cmd_numbers = @history.keys.last(max + offset).first(max)
-      else
-        cmd_numbers = @history.keys.last(max)
-      end
-      history_records = cmd_numbers.collect { |k| {number: k, command: @history[k]} }
-      command_count = @history.size
+    # only go so far back in command history, 1 million commands
+    # this could be a large object...need to index our shell_history file lol
+    # todo: this max needs to be done in load_history_from_log_file()
+    history_keys = @history.keys.last(1000000).reverse
+    # filter by phrase
+    if phrase
+      history_keys = history_keys.select {|k| (@history[k] && @history[k].include?(phrase)) || k.to_s == phrase }
     end
+    # no offset, just max
+    history_records = history_keys.first(max).collect { |k| {number: k, command: @history[k]} }
+    command_count = history_keys.size
+
     meta = {size:history_records.size, total:command_count.to_i, max:max, offset:offset}
-    return {:commands => history_records, :command_count => command_count, :meta => meta}
+    return {commands: history_records, command_count: command_count, meta: meta}
   end
 
   def print_history(options={})
@@ -696,16 +677,14 @@ class Morpheus::Cli::Shell
       end
     else
       print cyan
+      # by default show old->new as the shell history should bash history
+      history_records.reverse! unless options[:direction] == "desc"
       history_records.each do |history_record|
         puts "#{history_record[:number].to_s.rjust(3, ' ')}  #{history_record[:command]}"
       end
       if options[:show_pagination]
-        if options[:phrase] || options[:sort] || options[:direction] || options[:offset]
-          print_results_pagination(history_result[:meta])
-        else
-          # default order is weird, it's the last page of results, 1-25 is misleading and showing the indexes is stranger
-          print_results_pagination(history_result[:meta], {:message =>"Viewing most recent %{size} of %{total} %{label}"})
-        end
+        pagination_msg = options[:phrase] ? "Viewing most recent %{size} of %{total} commands matching '#{options[:phrase]}'" : "Viewing most recent %{size} of %{total} commands"
+        print_results_pagination(history_result[:meta], {:message =>pagination_msg})
         print reset, "\n"
       else
         print reset

data/lib/morpheus/cli/commands/virtual_images.rb

@@ -6,7 +6,7 @@ class Morpheus::Cli::VirtualImages
   include Morpheus::Cli::CliCommand
   include Morpheus::Cli::ProvisioningHelper
 
-  register_subcommands :list, :get, :add, :add_file, :remove_file, :update, :remove, :types => :virtual_image_types
+  register_subcommands :list, :get, :add, :add_file, :remove_file, :update, :remove, :convert, :types => :virtual_image_types
   register_subcommands :list_locations, :get_location, :remove_location
 
   # def initialize() 
@@ -172,11 +172,8 @@ EOT
         id
       else
         image = find_virtual_image_by_name_or_id(id)
-        if image
-          image['id']
-        else
-          raise_command_error "virtual image not found for name '#{id}'"
-        end
+        return 1, "virtual image not found for name '#{id}'" if image.nil?
+        image['id']
       end
     end
     return run_command_for_each_arg(id_list) do |arg|
@@ -302,7 +299,7 @@ EOT
     options = {}
     tenants_list = nil
     optparse = Morpheus::Cli::OptionParser.new do |opts|
-      opts.banner = subcommand_usage("[name] [options]")
+      opts.banner = subcommand_usage("[image] [options]")
       opts.on('--tenants LIST', Array, "Tenant Access, comma separated list of account IDs") do |list|
         if list.size == 1 && list[0] == 'null' # hacky way to clear it
           tenants_list = []
@@ -324,7 +321,7 @@ EOT
       end
       build_common_options(opts, options, [:options, :payload, :json, :dry_run, :remote])
       opts.footer = "Update a virtual image." + "\n" +
-                    "[name] is required. This is the name or id of a virtual image."
+                    "[image] is required. This is the name or id of a virtual image."
     end
     optparse.parse!(args)
     verify_args!(args:args, optparse:optparse, count:1)
@@ -377,6 +374,68 @@ EOT
     
   end
 
+  def convert(args)
+    image_name = args[0]
+    options = {}
+    # storageProviderId, format, name
+    optparse = Morpheus::Cli::OptionParser.new do |opts|
+      opts.banner = subcommand_usage("[image] [options]")
+      # opts.on('-n', '--name NAME', String, "Name (optional) of the new converted image. Default is name of the original image.") do |val|
+      #   options[:options]['name'] = val
+      # end
+      # opts.on('-f', '--format FORMAT', String, "Format (optional). Default is 'qcow2'") do |val|
+      #   options[:options]['format'] = val
+      # end
+      # opts.on('--storageProvider VALUE', String, "Storage Provider ID (optional). Default is storage provider of the original image.") do |val|
+      #   options[:options]['storageProvider'] = val.to_s
+      # end
+      build_option_type_options(opts, options, convert_virtual_image_option_types)
+      build_common_options(opts, options, [:options, :payload, :json, :dry_run, :remote])
+      opts.footer = "Convert a virtual image to a new format." + "\n" +
+        "[image] is required. This is the name or id of a virtual image."
+    end
+    optparse.parse!(args)
+    verify_args!(args:args, optparse:optparse, min:1, max:4)
+
+    connect(options)
+
+    virtual_image = find_virtual_image_by_name_or_id(image_name)
+    return 1, "Virtual image not found for #{image_name}" if virtual_image.nil?
+
+    passed_options = parse_passed_options(options)
+    payload = nil
+    if options[:payload]
+      payload = options[:payload]
+      payload.deep_merge!({virtual_image_object_key => passed_options}) unless passed_options.empty?
+    else
+      virtual_image_payload = passed_options
+      source_format = virtual_image['imageType']
+      v_prompt = Morpheus::Cli::OptionTypes.prompt(convert_virtual_image_option_types(source_format), options[:options], @api_client, {'virtualImageId': virtual_image['id']})
+      v_prompt.deep_compact!
+      virtual_image_payload.deep_merge!(v_prompt)
+      # support "storageProviderId" too
+      if virtual_image_payload['storageProviderId']
+        virtual_image_payload['storageProvider'] = virtual_image_payload.delete('storageProviderId')
+      end
+      # convert "storageProvider":1 to "storageProvider": {"id":1}
+      if virtual_image_payload['storageProvider'] && !virtual_image_payload['storageProvider'].is_a?(Hash)
+        virtual_image_payload['storageProvider'] = {'id' => virtual_image_payload['storageProvider'].to_i }
+      end
+      payload = virtual_image_payload
+    end
+    @virtual_images_interface.setopts(options)
+    if options[:dry_run]
+      print_dry_run @virtual_images_interface.dry.convert(virtual_image['id'], payload)
+      return
+    end
+    json_response = @virtual_images_interface.convert(virtual_image['id'], payload)
+    render_response(json_response, options, 'virtualImage') do
+      print_green_success "Updated virtual image #{virtual_image['name']}"
+      _get(virtual_image["id"], {}, options)
+    end
+    return 0, nil
+  end
+
   def virtual_image_types(args)
     options = {}
     optparse = Morpheus::Cli::OptionParser.new do |opts|
@@ -621,7 +680,7 @@ EOT
     do_gzip = false
     options = {}
     optparse = Morpheus::Cli::OptionParser.new do |opts|
-      opts.banner = subcommand_usage("[name] [filepath]")
+      opts.banner = subcommand_usage("[image] [filepath]")
       opts.on('--filename FILENAME', String, "Filename for uploaded file. Derived from [filepath] by default." ) do |val|
         file_name = val
       end
@@ -633,7 +692,7 @@ EOT
       end
       build_common_options(opts, options, [:json, :dry_run, :quiet, :remote])
       opts.footer = "Upload a virtual image file." + "\n" +
-                    "[name] is required. This is the name or id of a virtual image." + "\n" +
+                    "[image] is required. This is the name or id of a virtual image." + "\n" +
                     "[filepath] or --url is required. This is location of the virtual image file."
     end
     optparse.parse!(args)
@@ -700,7 +759,7 @@ EOT
   def remove_file(args)
     options = {}
     optparse = Morpheus::Cli::OptionParser.new do |opts|
-      opts.banner = subcommand_usage("[name] [filename]")
+      opts.banner = subcommand_usage("[image] [filename]")
       build_common_options(opts, options, [:auto_confirm, :json, :dry_run, :remote])
     end
     optparse.parse!(args)
@@ -940,11 +999,20 @@ EOT
 
   def find_virtual_image_by_name(name)
     json_results = @virtual_images_interface.list({name: name.to_s})
-    if json_results['virtualImages'].empty?
+    records = json_results['virtualImages']
+    if records.empty?
       print_red_alert "Virtual Image not found by name #{name}"
       return nil
+    elsif records.size > 1
+      print_red_alert "More than one Virtual Image found by name '#{name}'"
+      print_error "\n"
+      puts_error as_pretty_table(records, [:id, :name], {color:red})
+      print_red_alert "Try using ID instead"
+      print_error reset,"\n"
+      return nil
+    else
+      virtual_image = records[0]
     end
-    virtual_image = json_results['virtualImages'][0]
     return virtual_image
   end
 
@@ -1024,6 +1092,42 @@ EOT
     list
   end
 
+  def convert_virtual_image_option_types(source_format=nil)
+    [
+      {'shorthand' => '-f', 'fieldName' => 'format', 'type' => 'select', 'fieldLabel' => 'Format', 'selectOptions' => get_convert_image_formats(source_format), 'required' => true, 'defaultValue' => (source_format == 'qcow2' ? nil : 'qcow2')},
+      {'shorthand' => '-b', 'fieldName' => 'storageProviderId', 'type' => 'select', 'fieldLabel' => 'Bucket', 'optionSource' => 'storageProviders', 'required' => false, 'description' => 'Select Storage Provider Bucket or File Share.'},
+      {'shorthand' => '-n', 'fieldName' => 'name', 'fieldLabel' => 'Name', 'type' => 'text', 'required' => false}
+    ]
+  end
+
+  def convert_source_types
+    ["raw","qcow2","vmdk",'vmware', "vhd","ovf"]
+  end
+
+  def convert_destination_types
+    #["raw", "qcow2", "vmdk",'vmware',"vhd","ovf"]
+    convert_source_types
+  end
+
+  def get_convert_image_formats(source_format=nil)
+    categories = [
+      {'name' => 'QCOW2', 'value' => 'qcow2'},
+      {'name' => 'VMDK', 'value' => 'ovf'},
+      {'name' => 'VHD', 'value' => 'vhd'},
+      {'name' => 'Raw', 'value' => 'raw'}
+    ]
+    # if source format (imageType) is not in the list, then no options
+    # else reject the current value
+    if source_format
+      if !convert_source_types.include?(source_format.to_s.downcase)
+        categories = []
+      else
+        categories = categories.select {|it| it['value'] != source_format.to_s.downcase }
+      end
+    end
+    return categories
+  end
+
   def format_tenants(accounts)
     if accounts && accounts.size > 0
       accounts = accounts.sort {|a,b| a['name'] <=> b['name'] }.uniq {|it| it['id'] }