morpheus-cli 5.3.0 → 5.3.1.1

data/lib/morpheus/cli/image_builder_command.rb

@@ -30,15 +30,15 @@ class Morpheus::Cli::ImageBuilderCommand
 
   def connect(opts)
     @api_client = establish_remote_appliance_connection(opts)
-    @image_builder_interface = Morpheus::APIClient.new(@access_token,nil,nil, @appliance_url).image_builder
+    @image_builder_interface = @api_client.image_builder
     @image_builds_interface = @image_builder_interface.image_builds
     @boot_scripts_interface = @image_builder_interface.boot_scripts
     @preseed_scripts_interface = @image_builder_interface.preseed_scripts
-    @groups_interface = Morpheus::APIClient.new(@access_token,nil,nil, @appliance_url).groups
-    @clouds_interface = Morpheus::APIClient.new(@access_token,nil,nil, @appliance_url).clouds
-    @instances_interface = Morpheus::APIClient.new(@access_token,nil,nil, @appliance_url).instances
-    @instance_types_interface = Morpheus::APIClient.new(@access_token,nil,nil, @appliance_url).instance_types
-    @options_interface = Morpheus::APIClient.new(@access_token,nil,nil, @appliance_url).options
+    @groups_interface = @api_client.groups
+    @clouds_interface = @api_client.clouds
+    @instances_interface = @api_client.instances
+    @instance_types_interface = @api_client.instance_types
+    @options_interface = @api_client.options
     @active_group_id = Morpheus::Cli::Groups.active_groups[@appliance_name]
   end
 

data/lib/morpheus/cli/instance_types.rb

@@ -17,7 +17,7 @@ class Morpheus::Cli::InstanceTypes
 
   def connect(opts)
     @api_client = establish_remote_appliance_connection(opts)
-    @instance_types_interface = Morpheus::APIClient.new(@access_token,nil,nil, @appliance_url).instance_types
+    @instance_types_interface = @api_client.instance_types
   end
 
   def handle(args)

data/lib/morpheus/cli/instances.rb

@@ -84,6 +84,16 @@ class Morpheus::Cli::Instances
       opts.on('--status STATUS', "Filter by status i.e. provisioning,running,starting,stopping") do |val|
         params['status'] = (params['status'] || []) + val.to_s.split(',').collect {|s| s.strip }.select {|s| s != "" }
       end
+      opts.on( '--type CODE', String, "Filter by Instance Type code" ) do |val|
+        # commas used in names a lot so use --plan one --plan two
+        params['instanceType'] ||= []
+        params['instanceType'] << val
+      end
+      opts.on( '--environment CODE', String, "Filter by Environment code(s)" ) do |val|
+        # commas used in names a lot so use --plan one --plan two
+        params['environment'] ||= []
+        params['environment'] << val
+      end
       opts.on('--pending-removal', "Include instances pending removal.") do
         options[:showDeleted] = true
       end
@@ -120,8 +130,9 @@ class Morpheus::Cli::Instances
       end
       opts.on('-a', '--details', "Display all details: plan, stats, etc" ) do
         options[:details] = true
+        params['details'] = true # get more data from server this way
       end
-      build_common_options(opts, options, [:list, :query, :json, :yaml, :csv, :fields, :dry_run, :remote])
+      build_standard_list_options(opts, options)
       opts.footer = "List instances."
     end
     optparse.parse!(args)
@@ -130,159 +141,150 @@ class Morpheus::Cli::Instances
       options[:phrase] = args.join(" ")
     end
     connect(options)
-    begin
-      params.merge!(parse_list_options(options))
-      group = options[:group] ? find_group_by_name_or_id_for_provisioning(options[:group]) : nil
-      if group
-        params['siteId'] = group['id']
-      end
 
-      # argh, this doesn't work because group_id is required for options/clouds
-      # cloud = options[:cloud] ? find_cloud_by_name_or_id_for_provisioning(group_id, options[:cloud]) : nil
-      cloud = options[:cloud] ? find_zone_by_name_or_id(nil, options[:cloud]) : nil
-      if cloud
-        params['zoneId'] = cloud['id']
-      end
+    params.merge!(parse_list_options(options))
+    group = options[:group] ? find_group_by_name_or_id_for_provisioning(options[:group]) : nil
+    if group
+      params['siteId'] = group['id']
+    end
 
-      host = options[:host] ? find_host_by_name_or_id(options[:host]) : options[:host]
-      if host
-        params['serverId'] = host['id']
-      end
+    # argh, this doesn't work because group_id is required for options/clouds
+    # cloud = options[:cloud] ? find_cloud_by_name_or_id_for_provisioning(group_id, options[:cloud]) : nil
+    cloud = options[:cloud] ? find_zone_by_name_or_id(nil, options[:cloud]) : nil
+    if cloud
+      params['zoneId'] = cloud['id']
+    end
 
-      account = nil
-      #todo: user = find_available_user_option(owner_id)
+    host = options[:host] ? find_host_by_name_or_id(options[:host]) : options[:host]
+    if host
+      params['serverId'] = host['id']
+    end
 
-      if options[:owner]
-        created_by_ids = find_all_user_ids(account ? account['id'] : nil, options[:owner])
-        return if created_by_ids.nil?
-        params['createdBy'] = created_by_ids
-        params['ownerId'] = created_by_ids # 4.2.1+
+    account = nil
+    #todo: user = find_available_user_option(owner_id)
+
+    if options[:owner]
+      created_by_ids = find_all_user_ids(account ? account['id'] : nil, options[:owner])
+      return if created_by_ids.nil?
+      params['createdBy'] = created_by_ids
+      params['ownerId'] = created_by_ids # 4.2.1+
+    end
+
+    params['showDeleted'] = true if options[:showDeleted]
+    params['deleted'] = true if options[:deleted]
+    params['labels'] = options[:labels] if options[:labels]
+    if options[:tags]
+      options[:tags].each do |k,v|
+        params['tags.' + k] = v
       end
+    end
 
-      params['showDeleted'] = true if options[:showDeleted]
-      params['deleted'] = true if options[:deleted]
-      params['labels'] = options[:labels] if options[:labels]
-      if options[:tags]
-        options[:tags].each do |k,v|
-          params['tags.' + k] = v
+    @instances_interface.setopts(options)
+    if options[:dry_run]
+      print_dry_run @instances_interface.dry.list(params)
+      return 0
+    end
+    json_response = @instances_interface.list(params)
+    all_stats = json_response['stats'] || {}
+    # merge stats into each record just to be nice...
+    if options[:include_fields] || options[:all_fields]
+      if json_response['instances']
+        if all_stats
+          json_response['instances'].each do |it|
+            it['stats'] ||= all_stats[it['id'].to_s] || all_stats[it['id']]
+          end
         end
       end
+    end
+    render_response(json_response, options, "instances") do
+      instances = json_response['instances']
 
-      @instances_interface.setopts(options)
-      if options[:dry_run]
-        print_dry_run @instances_interface.dry.list(params)
-        return 0
+      title = "Morpheus Instances"
+      subtitles = []
+      if group
+        subtitles << "Group: #{group['name']}".strip
       end
-      json_response = @instances_interface.list(params)
-      if options[:json]
-        puts as_json(json_response, options, "instances")
-        return 0
-      elsif options[:yaml]
-        puts as_yaml(json_response, options, "instances")
-        return 0
-      elsif options[:csv]
-        # merge stats to be nice here..
-        if json_response['instances']
-          all_stats = json_response['stats'] || {}
-          if all_stats
-            json_response['instances'].each do |it|
-              it['stats'] ||= all_stats[it['id'].to_s] || all_stats[it['id']]
+      if cloud
+        subtitles << "Cloud: #{cloud['name']}".strip
+      end
+      if host
+        subtitles << "Host: #{host['name']}".strip
+      end
+      if options[:owner]
+        subtitles << "Created By: #{options[:owner]}"
+      end
+      subtitles += parse_list_subtitles(options)
+      print_h1 title, subtitles, options
+      if instances.empty?
+        print cyan,"No instances found.",reset,"\n"
+      else
+        # print_instances_table(instances)
+        # server returns stats in a separate key stats => {"id" => {} }
+        # the id is a string right now..for some reason..
+        all_stats = json_response['stats'] || {} 
+        if all_stats
+          instances.each do |it|
+            if !it['stats']
+              found_stats = all_stats[it['id'].to_s] || all_stats[it['id']]
+              it['stats'] = found_stats # || {}
             end
           end
         end
-        puts records_as_csv(json_response['instances'], options)
-      else
-        instances = json_response['instances']
 
-        title = "Morpheus Instances"
-        subtitles = []
-        if group
-          subtitles << "Group: #{group['name']}".strip
-        end
-        if cloud
-          subtitles << "Cloud: #{cloud['name']}".strip
-        end
-        if host
-          subtitles << "Host: #{host['name']}".strip
-        end
-        if options[:owner]
-          subtitles << "Created By: #{options[:owner]}"
-        end
-        subtitles += parse_list_subtitles(options)
-        print_h1 title, subtitles, options
-        if instances.empty?
-          print cyan,"No instances found.",reset,"\n"
-        else
-          # print_instances_table(instances)
-          # server returns stats in a separate key stats => {"id" => {} }
-          # the id is a string right now..for some reason..
-          all_stats = json_response['stats'] || {} 
-          if all_stats
-            instances.each do |it|
-              if !it['stats']
-                found_stats = all_stats[it['id'].to_s] || all_stats[it['id']]
-                it['stats'] = found_stats # || {}
-              end
+        rows = instances.collect {|instance| 
+          stats = instance['stats']
+          cpu_usage_str = !stats ? "" : generate_usage_bar((stats['usedCpu'] || stats['cpuUsage']).to_f, 100, {max_bars: 10})
+          memory_usage_str = !stats ? "" : generate_usage_bar(stats['usedMemory'], stats['maxMemory'], {max_bars: 10})
+          storage_usage_str = !stats ? "" : generate_usage_bar(stats['usedStorage'], stats['maxStorage'], {max_bars: 10})
+          if options[:details] || options[:stats]
+            if stats['maxMemory'] && stats['maxMemory'].to_i != 0
+              memory_usage_str = memory_usage_str + cyan + format_bytes_short(stats['usedMemory']).strip.rjust(8, ' ')  + " / " + format_bytes_short(stats['maxMemory']).strip
             end
-          end
-
-          rows = instances.collect {|instance| 
-            stats = instance['stats']
-            cpu_usage_str = !stats ? "" : generate_usage_bar((stats['usedCpu'] || stats['cpuUsage']).to_f, 100, {max_bars: 10})
-            memory_usage_str = !stats ? "" : generate_usage_bar(stats['usedMemory'], stats['maxMemory'], {max_bars: 10})
-            storage_usage_str = !stats ? "" : generate_usage_bar(stats['usedStorage'], stats['maxStorage'], {max_bars: 10})
-            if options[:details] || options[:stats]
-              if stats['maxMemory'] && stats['maxMemory'].to_i != 0
-                memory_usage_str = memory_usage_str + cyan + format_bytes_short(stats['usedMemory']).strip.rjust(8, ' ')  + " / " + format_bytes_short(stats['maxMemory']).strip
-              end
-              if stats['maxStorage'] && stats['maxStorage'].to_i != 0
-                storage_usage_str = storage_usage_str + cyan + format_bytes_short(stats['usedStorage']).strip.rjust(8, ' ') + " / " + format_bytes_short(stats['maxStorage']).strip
-              end
+            if stats['maxStorage'] && stats['maxStorage'].to_i != 0
+              storage_usage_str = storage_usage_str + cyan + format_bytes_short(stats['usedStorage']).strip.rjust(8, ' ') + " / " + format_bytes_short(stats['maxStorage']).strip
             end
-            row = {
-              id: instance['id'],
-              name: instance['name'],
-              connection: format_instance_connection_string(instance),
-              environment: instance['instanceContext'],
-              user: (instance['owner'] ? (instance['owner']['username'] || instance['owner']['id']) : (instance['createdBy'].is_a?(Hash) ? instance['createdBy']['username'] : instance['createdBy'])),
-              tenant: (instance['owner'] ? (instance['owner']['username'] || instance['owner']['id']) : (instance['createdBy'].is_a?(Hash) ? instance['createdBy']['username'] : instance['createdBy'])),
-              nodes: instance['containers'].count,
-              status: format_instance_status(instance, cyan),
-              type: instance['instanceType']['name'],
-              group: instance['group'] ? instance['group']['name'] : nil,
-              cloud: instance['cloud'] ? instance['cloud']['name'] : nil,
-              plan: instance['plan'] ? instance['plan']['name'] : '',
-              version: instance['instanceVersion'] ? instance['instanceVersion'] : '',
-              created: format_local_dt(instance['dateCreated']),
-              cpu: cpu_usage_str + cyan,
-              memory: memory_usage_str + cyan, 
-              storage: storage_usage_str + cyan
-            }
-            row
-          }
-          columns = [:id, {:name => {:max_width => 50}}, :group, :cloud, 
-              :type, :version, :environment, 
-              {:created => {:display_name => "CREATED"}}, 
-              # {:tenant => {:display_name => "TENANT"}}, 
-              {:user => {:display_name => "OWNER", :max_width => 20}}, 
-              :plan,
-              :nodes, {:connection => {:max_width => 30}}, :status, :cpu, :memory, :storage]
-          # custom pretty table columns ... this is handled in as_pretty_table now(), 
-          # todo: remove all these.. and try to always pass rows as the json data itself..
-          if options[:details] != true
-            columns.delete(:plan)
           end
-          print cyan
-          print as_pretty_table(rows, columns, options)
-          print reset
-          print_results_pagination(json_response)
+          row = {
+            id: instance['id'],
+            name: instance['name'],
+            connection: format_instance_connection_string(instance),
+            environment: instance['instanceContext'],
+            user: (instance['owner'] ? (instance['owner']['username'] || instance['owner']['id']) : (instance['createdBy'].is_a?(Hash) ? instance['createdBy']['username'] : instance['createdBy'])),
+            tenant: (instance['owner'] ? (instance['owner']['username'] || instance['owner']['id']) : (instance['createdBy'].is_a?(Hash) ? instance['createdBy']['username'] : instance['createdBy'])),
+            nodes: instance['containers'].count,
+            status: format_instance_status(instance, cyan),
+            type: instance['instanceType']['name'],
+            group: instance['group'] ? instance['group']['name'] : nil,
+            cloud: instance['cloud'] ? instance['cloud']['name'] : nil,
+            plan: instance['plan'] ? instance['plan']['name'] : '',
+            version: instance['instanceVersion'] ? instance['instanceVersion'] : '',
+            created: format_local_dt(instance['dateCreated']),
+            cpu: cpu_usage_str + cyan,
+            memory: memory_usage_str + cyan, 
+            storage: storage_usage_str + cyan
+          }
+          row
+        }
+        columns = [:id, {:name => {:max_width => 50}}, :group, :cloud, 
+            :type, :version, :environment, 
+            {:created => {:display_name => "CREATED"}}, 
+            # {:tenant => {:display_name => "TENANT"}}, 
+            {:user => {:display_name => "OWNER", :max_width => 20}}, 
+            :plan,
+            :nodes, {:connection => {:max_width => 30}}, :status, :cpu, :memory, :storage]
+        # custom pretty table columns ... this is handled in as_pretty_table now(), 
+        # todo: remove all these.. and try to always pass rows as the json data itself..
+        if options[:details] != true
+          columns.delete(:plan)
         end
-        print reset,"\n"
+        print cyan
+        print as_pretty_table(rows, columns, options)
+        print reset
+        print_results_pagination(json_response)
       end
-    rescue RestClient::Exception => e
-      print_rest_exception(e, options)
-      exit 1
+      print reset,"\n"
     end
+    return 0, nil
   end
 
   def count(args)
@@ -436,7 +438,7 @@ class Morpheus::Cli::Instances
       opts.on("--create-backup [on|off]", String, "Automation: Create Backups.") do |val|
         options[:create_backup] = ['on','true','1',''].include?(val.to_s.downcase) ? 'on' : 'off'
       end
-      opts.on("--security-groups LIST", String, "Security Groups, comma sepearated list of security group IDs") do |val|
+      opts.on("--security-groups LIST", String, "Security Groups, comma separated list of security group IDs") do |val|
         options[:security_groups] = val.split(",").collect {|s| s.strip }.select {|s| !s.to_s.empty? }
       end
       opts.on('--refresh [SECONDS]', String, "Refresh until status is running,failed. Default interval is #{default_refresh_interval} seconds.") do |val|
@@ -1200,7 +1202,15 @@ class Morpheus::Cli::Instances
         options[:details] = true
         options[:include_containers] = true
         options[:include_scaling] = true
+        options[:include_costs]
+      end
+      opts.on(nil, '--details', "Alias for --all" ) do
+        options[:details] = true
+        options[:include_containers] = true
+        options[:include_scaling] = true
+        options[:include_costs]
       end
+      opts.add_hidden_option('--details')
       opts.on( nil, '--containers', "Display Instance Containers" ) do
         options[:include_containers] = true
       end
@@ -1215,9 +1225,6 @@ class Morpheus::Cli::Instances
       opts.on( nil, '--scaling', "Display Instance Scaling Settings" ) do
         options[:include_scaling] = true
       end
-      opts.on( nil, '--costs', "Display Cost and Price" ) do
-        options[:include_costs] = true
-      end
       opts.on('--refresh [SECONDS]', String, "Refresh until status is running,failed. Default interval is #{default_refresh_interval} seconds.") do |val|
         options[:refresh_until_status] ||= "running,failed"
         if !val.to_s.empty?
@@ -1233,7 +1240,7 @@ class Morpheus::Cli::Instances
       # opts.on( nil, '--lb', "Display Load Balancer Details" ) do
       #   options[:include_lb] = true
       # end
-      build_common_options(opts, options, [:query, :json, :yaml, :csv, :fields, :dry_run, :remote])
+      build_standard_get_options(opts, options)
       opts.footer = "Get details about an instance.\n" + 
                     "[instance] is required. This is the name or id of an instance. Supports 1-N [instance] arguments."
     end
@@ -1249,36 +1256,29 @@ class Morpheus::Cli::Instances
     end
   end
 
-  def _get(arg, options={})
-    begin
-      if options[:dry_run]
-        @instances_interface.setopts(options)
-        if arg.to_s =~ /\A\d{1,}\Z/
-          print_dry_run @instances_interface.dry.get(arg.to_i)
-        else
-          print_dry_run @instances_interface.dry.get({name:arg})
-        end
-        return
-      end
-      instance = find_instance_by_name_or_id(arg =~ /\A\d{1,}\Z/ ? arg.to_i : arg)
-      @instances_interface.setopts(options)
-      json_response = @instances_interface.get(instance['id'])
-      if options[:quiet]
-        return 0
-      end
-      if options[:json]
-        puts as_json(json_response, options, "instance")
-        return 0
-      elsif options[:yaml]
-        puts as_yaml(json_response, options, "instance")
-        return 0
-      end
-
-      if options[:csv]
-        puts records_as_csv([json_response['instance']], options)
-        return 0
-      end
+  def _get(id, options={})
+    params = {}
+    params.merge!(parse_query_options(options))
+    # Use details=true to get more details from the appliance
+    # if options[:details] || options[:include_containers]  || options[:include_scaling]
+    if options[:details] || options[:include_containers]  || options[:include_scaling]
+      params['details'] = true
+    end
+    instance = nil
+    if id.to_s !~ /\A\d{1,}\Z/
+      instance = find_instance_by_name_or_id(id)
+      return 1, "Instance not found by name #{id}" if instance.nil?
+      id = instance['id']
+    end
+    if options[:dry_run]
+      print_dry_run @instances_interface.dry.get(id, params)
+      return 0, nil
+    end
+    @instances_interface.setopts(options)
+    json_response = @instances_interface.get(id, params)
+    render_response(json_response, options, "instance") do
       instance = json_response['instance']
+      pricing = instance['instancePrice']
       stats = instance['stats'] || json_response['stats'] || {}
       # load_balancers = json_response['loadBalancers'] || {}
       # metadata tags used to be returned as metadata and are now returned as tags
@@ -1296,7 +1296,12 @@ class Morpheus::Cli::Instances
       # containers are fetched via separate api call
       containers = nil
       if options[:include_containers]
-        containers = @instances_interface.containers(instance['id'])['containers']
+        # todo: can use instance['containerDetails'] in api 5.2.7/5.3.2
+        if instance['containerDetails']
+          containers = instance['containerDetails']
+        else
+          containers = @instances_interface.containers(instance['id'])['containers']
+        end
       end
 
       # threshold is fetched via separate api call too
@@ -1330,8 +1335,20 @@ class Morpheus::Cli::Instances
         "Layout" => lambda {|it| it['layout'] ? it['layout']['name'] : '' },
         "Version" => lambda {|it| it['instanceVersion'] },
         "Plan" => lambda {|it| it['plan'] ? it['plan']['name'] : '' },
-        # "Cost" => lambda {|it| it['hourlyCost'] ? format_money(it['hourlyCost'], (it['currency'] || 'USD'), {sigdig:15}).to_s + ' per hour' : '' },
-        # "Price" => lambda {|it| it['hourlyPrice'] ? format_money(it['hourlyPrice'], (it['currency'] || 'USD'), {sigdig:15}).to_s + ' per hour' : '' },
+        "Price" => lambda {|it|
+          if pricing
+            pricing['price'] ? format_money(pricing['price'], (pricing['currency'] || 'USD')).to_s + ' per ' + pricing['unit'].to_s : ''
+          elsif it['hourlyPrice']
+            format_money(it['hourlyPrice'], (it['currency'] || 'USD')).to_s + ' per hour'
+          end
+        },
+        "Cost" => lambda {|it| 
+          if pricing
+            pricing['cost'] ? format_money(pricing['cost'], (pricing['currency'] || 'USD')).to_s + ' per ' + pricing['unit'].to_s : ''
+          elsif it['hourlyCost']
+            format_money(it['hourlyCost'], (it['currency'] || 'USD')).to_s + ' per hour'
+          end
+        },
         "Environment" => 'instanceContext',
         "Labels" => lambda {|it| labels ? labels.join(',') : '' },
         "Tags" => lambda {|it| tags ? tags.collect {|m| "#{m['name']}: #{m['value']}" }.join(', ') : '' },
@@ -1343,6 +1360,7 @@ class Morpheus::Cli::Instances
           end
         },
         #"Tenant" => lambda {|it| it['tenant'] ? it['tenant']['name'] : '' },
+        "Apps" => lambda {|it| anded_list(it['apps'] ? it['apps'].collect {|app| app['name'] } : [])},
         "Date Created" => lambda {|it| format_local_dt(it['dateCreated']) },
         # "Last Updated" => lambda {|it| format_local_dt(it['lastUpdated']) },
         "Power Schedule" => lambda {|it| (it['powerSchedule'] && it['powerSchedule']['type']) ? it['powerSchedule']['type']['name'] : '' },
@@ -1356,12 +1374,17 @@ class Morpheus::Cli::Instances
       }
       description_cols.delete("Labels") if labels.nil? || labels.empty?
       description_cols.delete("Tags") if tags.nil? || tags.empty?
+      description_cols.delete("Apps") if instance['apps'].nil? || instance['apps'].empty?
       description_cols.delete("Power Schedule") if instance['powerSchedule'].nil?
       description_cols.delete("Expire Date") if instance['expireDate'].nil?
       description_cols.delete("Shutdown Date") if instance['shutdownDate'].nil?
       description_cols["Removal Date"] = lambda {|it| format_local_dt(it['removalDate'])} if instance['status'] == 'pendingRemoval'
       description_cols.delete("Last Deployment") if instance['lastDeploy'].nil?
       description_cols.delete("Locked") if instance['locked'] != true
+      price_value = (pricing ? pricing['price'] : instance['hourlyPrice']).to_i
+      cost_value = (pricing ? pricing['cost'] : instance['hourlyCost']).to_i
+      description_cols.delete("Price") if price_value == 0
+      description_cols.delete("Cost") if cost_value == 0 || cost_value == price_value
       #description_cols.delete("Environment") if instance['instanceContext'].nil?
       print_description_list(description_cols, instance)
 
@@ -1393,15 +1416,6 @@ class Morpheus::Cli::Instances
         print_stats_usage(stats)
       end
 
-      if options[:include_costs]
-        print_h2 "Instance Cost"
-        cost_columns = {
-          "Cost" => lambda {|it| it['hourlyCost'] ? format_money(it['hourlyCost'], (it['currency'] || 'USD'), {sigdig:15}).to_s + ' per hour' : '' },
-          "Price" => lambda {|it| it['hourlyPrice'] ? format_money(it['hourlyPrice'], (it['currency'] || 'USD'), {sigdig:15}).to_s + ' per hour' : '' },
-        }
-        print_description_list(cost_columns, instance)
-      end
-
       print reset, "\n"
 
       # if options[:include_lb]
@@ -1491,15 +1505,11 @@ class Morpheus::Cli::Instances
           print cyan, "Refreshing in #{options[:refresh_interval] > 1 ? options[:refresh_interval].to_i : options[:refresh_interval]} seconds"
           sleep_with_dots(options[:refresh_interval])
           print "\n"
-          _get(arg, options)
+          _get(instance['id'], options)
         end
       end
-
-      return 0
-    rescue RestClient::Exception => e
-      print_rest_exception(e, options)
-      exit 1
     end
+    return 0, nil
   end
 
   def list_containers(args)