hammer_cli_csv 2.0.0 → 2.1.0

data/lib/hammer_cli_csv/content_view_filters.rb

@@ -14,41 +14,39 @@ module HammerCLICsv
       REPOSITORIES = 'Repositories'
       RULES = 'Rules'
 
-      def export
-        CSV.open(option_file || '/dev/stdout', 'wb', {:force_quotes => false}) do |csv|
-          csv << [NAME, CONTENTVIEW, ORGANIZATION, TYPE, DESCRIPTION, REPOSITORIES, RULES]
-          @api.resource(:organizations).call(:index, {
-              :per_page => 999999
-          })['results'].each do |organization|
-            next if option_organization && organization['name'] != option_organization
-
-            @api.resource(:content_views).call(:index, {
-                'per_page' => 999999,
-                'organization_id' => organization['id'],
-                'nondefault' => true
-            })['results'].each do |contentview|
-              @api.resource(:content_view_filters).call(:index, {
-                  'content_view_id' => contentview['id']
-              })['results'].collect do |filter|
-                filter_type = "#{filter['inclusion'] == true ? 'Include' : 'Exclude'} #{export_filter_type(filter['type'])}"
-
-                rules = nil
-                case filter['type']
-                when /rpm/
-                  rules = export_rpm_rules(filter)
-                when /erratum/
-                  rules = export_erratum_rules(filter)
-                when /package_group/
-                  rules = export_package_group_rules(filter)
-                else
-                  raise "Unknown filter rule type '#{filter['type']}'"
-                end
-
-                name = filter['name']
-                repositories = export_column(filter, 'repositories', 'name')
-                csv << [name, contentview['name'], organization['name'], filter_type, filter['description'],
-                        repositories, rules]
+      def export(csv)
+        csv << [NAME, CONTENTVIEW, ORGANIZATION, TYPE, DESCRIPTION, REPOSITORIES, RULES]
+        @api.resource(:organizations).call(:index, {
+            :per_page => 999999
+        })['results'].each do |organization|
+          next if option_organization && organization['name'] != option_organization
+
+          @api.resource(:content_views).call(:index, {
+              'per_page' => 999999,
+              'organization_id' => organization['id'],
+              'nondefault' => true
+          })['results'].each do |contentview|
+            @api.resource(:content_view_filters).call(:index, {
+                'content_view_id' => contentview['id']
+            })['results'].collect do |filter|
+              filter_type = "#{filter['inclusion'] == true ? 'Include' : 'Exclude'} #{export_filter_type(filter['type'])}"
+
+              rules = nil
+              case filter['type']
+              when /rpm/
+                rules = export_rpm_rules(filter)
+              when /erratum/
+                rules = export_erratum_rules(filter)
+              when /package_group/
+                rules = export_package_group_rules(filter)
+              else
+                raise "Unknown filter rule type '#{filter['type']}'"
               end
+
+              name = filter['name']
+              repositories = export_column(filter, 'repositories', 'name')
+              csv << [name, contentview['name'], organization['name'], filter_type, filter['description'],
+                      repositories, rules]
             end
           end
         end
@@ -155,8 +153,6 @@ module HammerCLICsv
           puts 'done' if option_verbose?
         end
 
-      rescue RuntimeError => e
-        raise "#{e}\n       #{line}"
       end
 
       private

data/lib/hammer_cli_csv/content_views.rb

@@ -16,44 +16,42 @@ module HammerCLICsv
       REPOSITORIES = 'Repositories or Composites'
       ENVIRONMENTS = "Lifecycle Environments"
 
-      def export
-        CSV.open(option_file || '/dev/stdout', 'wb', {:force_quotes => false}) do |csv|
-          csv << [NAME, LABEL, ORGANIZATION, COMPOSITE, REPOSITORIES, ENVIRONMENTS]
-          @api.resource(:organizations).call(:index, {
-              :per_page => 999999
-          })['results'].each do |organization|
-            next if option_organization && organization['name'] != option_organization
-
-            composite_contentviews = []
-            @api.resource(:content_views).call(:index, {
-                'per_page' => 999999,
-                'organization_id' => organization['id'],
-                'nondefault' => true
-            })['results'].each do |contentview|
-              name = contentview['name']
-              label = contentview['label']
-              orgname = organization['name']
-              environments = CSV.generate do |column|
-                column << environment_names(contentview)
-              end
-              environments.delete!("\n")
-              composite = contentview['composite'] == true ? 'Yes' : 'No'
-              if composite == 'Yes'
-                contentviews = CSV.generate do |column|
-                  column << contentview['components'].collect do |component|
-                    component['content_view']['name']
-                  end
+      def export(csv)
+        csv << [NAME, LABEL, ORGANIZATION, COMPOSITE, REPOSITORIES, ENVIRONMENTS]
+        @api.resource(:organizations).call(:index, {
+            :per_page => 999999
+        })['results'].each do |organization|
+          next if option_organization && organization['name'] != option_organization
+
+          composite_contentviews = []
+          @api.resource(:content_views).call(:index, {
+              'per_page' => 999999,
+              'organization_id' => organization['id'],
+              'nondefault' => true
+          })['results'].each do |contentview|
+            name = contentview['name']
+            label = contentview['label']
+            orgname = organization['name']
+            environments = CSV.generate do |column|
+              column << environment_names(contentview)
+            end
+            environments.delete!("\n")
+            composite = contentview['composite'] == true ? 'Yes' : 'No'
+            if composite == 'Yes'
+              contentviews = CSV.generate do |column|
+                column << contentview['components'].collect do |component|
+                  component['content_view']['name']
                 end
-                contentviews.delete!("\n")
-                composite_contentviews << [name, 1, label, orgname, composite, contentviews, environments]
-              else
-                repositories = export_column(contentview, 'repositories', 'name')
-                csv << [name, label, orgname, composite, repositories, environments]
               end
+              contentviews.delete!("\n")
+              composite_contentviews << [name, 1, label, orgname, composite, contentviews, environments]
+            else
+              repositories = export_column(contentview, 'repositories', 'name')
+              csv << [name, label, orgname, composite, repositories, environments]
             end
-            composite_contentviews.each do |contentview|
-              csv << contentview
-            end
+          end
+          composite_contentviews.each do |contentview|
+            csv << contentview
           end
         end
       end
@@ -131,8 +129,6 @@ module HammerCLICsv
           puts _('done') if option_verbose?
         end
 
-      rescue RuntimeError => e
-        raise "#{e}\n       #{line}"
       end
 
       def environment_names(contentview)

data/lib/hammer_cli_csv/csv.rb

@@ -3,9 +3,25 @@ require 'hammer_cli/exit_codes'
 
 module HammerCLICsv
   class CsvCommand < HammerCLI::AbstractCommand
+    def help
+      self.class.help(invocation_path, CsvSortedBuilder.new)
+    end
+
+    class CsvSortedBuilder < SortedBuilder
+      def add_list(heading, items)
+        items.delete_if do |item|
+          if item.class == Clamp::Subcommand::Definition
+            !item.subcommand_class.supported?
+          else
+            false
+          end
+        end
+        super(heading, items)
+      end
+    end
   end
 
   HammerCLI::MainCommand.subcommand('csv',
-                                    _('import to, or export from a running foretello server'),
+                                    _('import to or export from a running foreman server'),
                                     HammerCLICsv::CsvCommand)
 end

data/lib/hammer_cli_csv/domains.rb

@@ -12,23 +12,21 @@ module HammerCLICsv
 
       SEPARATOR = ' = '
 
-      def export
-        CSV.open(option_file || '/dev/stdout', 'wb', {:force_quotes => true}) do |csv|
-          csv << [NAME, ORGANIZATIONS, LOCATIONS, DESCRIPTION, SMART_PROXY, PARAMETERS]
-          search_options = {:per_page => 999999}
-          search_options['search'] = "organization=\"#{option_organization}\"" if option_organization
-          @api.resource(:domains).call(:index, search_options)['results'].each do |domain|
-            domain = @api.resource(:domains).call(:show, {'id' => domain['id']})
-            raise "Domain 'id=#{domain['id']}' not found" if !domain || domain.empty?
-
-            name = domain['name']
-            organizations = option_organization ? option_organization : export_column(domain, 'organizations', 'name')
-            locations = export_column(domain, 'locations', 'name')
-            description = domain['fullname']
-            capsule = foreman_smart_proxy(:id => domain['dns_id'])
-            parameters = export_parameters(domain['parameters'])
-            csv << [name, organizations, locations, description, capsule, parameters]
-          end
+      def export(csv)
+        csv << [NAME, ORGANIZATIONS, LOCATIONS, DESCRIPTION, SMART_PROXY, PARAMETERS]
+        search_options = {:per_page => 999999}
+        search_options['search'] = "organization=\"#{option_organization}\"" if option_organization
+        @api.resource(:domains).call(:index, search_options)['results'].each do |domain|
+          domain = @api.resource(:domains).call(:show, {'id' => domain['id']})
+          raise "Domain 'id=#{domain['id']}' not found" if !domain || domain.empty?
+
+          name = domain['name']
+          organizations = option_organization ? option_organization : export_column(domain, 'organizations', 'name')
+          locations = export_column(domain, 'locations', 'name')
+          description = domain['fullname']
+          capsule = foreman_smart_proxy(:id => domain['dns_id'])
+          parameters = export_parameters(domain['parameters'])
+          csv << [name, organizations, locations, description, capsule, parameters]
         end
       end
 
@@ -92,8 +90,6 @@ module HammerCLICsv
 
           print "done\n" if option_verbose?
         end
-      rescue RuntimeError => e
-        raise "#{e}\n       #{line}"
       end
 
       private

data/lib/hammer_cli_csv/export.rb

@@ -4,21 +4,29 @@ module HammerCLICsv
       command_name 'export'
       desc         'export into directory'
 
+      def self.supported?
+        true
+      end
+
       option %w(-v --verbose), :flag, _('be verbose')
       option %w(--threads), 'THREAD_COUNT', _('Number of threads to hammer with'),
              :default => 1, :hidden => true
-      option '--dir', 'DIRECTORY', _('directory to import from')
+      option '--dir', 'DIRECTORY', _('directory to export to')
       option %w(--organization), 'ORGANIZATION', _('Only process organization matching this name')
 
       RESOURCES = %w(
-        organizations locations puppet_environments operating_systems
+        settings organizations locations puppet_environments operating_systems
         domains architectures partition_tables lifecycle_environments host_collections
         provisioning_templates
         subscriptions activation_keys hosts content_hosts reports roles users
       )
+      SUPPORTED_RESOURCES = %w(
+        settings
+      )
       RESOURCES.each do |resource|
         dashed = resource.sub('_', '-')
-        option "--#{dashed}", 'FILE', "csv file for #{dashed}"
+        option "--#{dashed}", 'FILE', "csv file for #{dashed}",
+               :hidden => !SUPPORTED_RESOURCES.include?(resource)
       end
 
       def execute
@@ -46,7 +54,7 @@ module HammerCLICsv
                                       :username => @username,
                                       :password => @password
                                     })
-          skipped_resources = %w( locations puppet_environments operating_systems
+          skipped_resources = %w( settings locations puppet_environments operating_systems
                                   domains architectures partition_tables lifecycle_environments
                                   provisioning_templates
                                   hosts reports )
@@ -58,7 +66,7 @@ module HammerCLICsv
                                            :password => @password,
                                            :api_version => 2
                                          })
-          skipped_resources = []
+          skipped_resources = (RESOURCES - SUPPORTED_RESOURCES)
         end
 
         # Swing the hammers

data/lib/hammer_cli_csv/host_collections.rb

@@ -8,29 +8,27 @@ module HammerCLICsv
       LIMIT = 'Limit'
       DESCRIPTION = 'Description'
 
-      def export
-        CSV.open(option_file || '/dev/stdout', 'wb') do |csv|
-          csv << [NAME, ORGANIZATION, LIMIT, DESCRIPTION]
-          if @server_status['release'] == 'Headpin'
-            @headpin.get(:organizations).each do |organization|
-              next if option_organization && organization['name'] != option_organization
-              @headpin.get("organizations/#{organization['label']}/system_groups").each do |systemgroup|
-                csv << [systemgroup['name'], organization['name'],
-                        systemgroup['max_systems'].to_i < 0 ? 'Unlimited' : systemgroup['max_systems'],
-                        systemgroup['description']]
-              end
+      def export(csv)
+        csv << [NAME, ORGANIZATION, LIMIT, DESCRIPTION]
+        if @server_status['release'] == 'Headpin'
+          @headpin.get(:organizations).each do |organization|
+            next if option_organization && organization['name'] != option_organization
+            @headpin.get("organizations/#{organization['label']}/system_groups").each do |systemgroup|
+              csv << [systemgroup['name'], organization['name'],
+                      systemgroup['max_systems'].to_i < 0 ? 'Unlimited' : systemgroup['max_systems'],
+                      systemgroup['description']]
             end
-          else
-            @api.resource(:organizations).call(:index, {'per_page' => 999999})['results'].each do |organization|
-              next if option_organization && organization['name'] != option_organization
-              @api.resource(:host_collections).call(:index, {
-                  'organization_id' => organization['id']
-              })['results'].each do |hostcollection|
-                limit = hostcollection['unlimited_content_hosts'] ? 'Unlimited' : hostcollection['max_content_hosts']
-                csv << [hostcollection['name'], organization['name'],
-                        limit,
-                        hostcollection['description']]
-              end
+          end
+        else
+          @api.resource(:organizations).call(:index, {'per_page' => 999999})['results'].each do |organization|
+            next if option_organization && organization['name'] != option_organization
+            @api.resource(:host_collections).call(:index, {
+                'organization_id' => organization['id']
+            })['results'].each do |hostcollection|
+              limit = hostcollection['unlimited_content_hosts'] ? 'Unlimited' : hostcollection['max_content_hosts']
+              csv << [hostcollection['name'], organization['name'],
+                      limit,
+                      hostcollection['description']]
             end
           end
         end

data/lib/hammer_cli_csv/host_groups.rb

@@ -19,96 +19,104 @@ module HammerCLICsv
       PUPPET_PROXY = 'Puppet Proxy'
       PUPPET_CA_PROXY = 'Puppet CA Proxy'
       CONTENT_SOURCE = 'Content Source'
+      PASSWORD = 'Password'
+      PUPPET_CLASSES = 'Puppet Classes'
 
-      def export
-        CSV.open(option_file || '/dev/stdout', 'wb', {:force_quotes => true}) do |csv|
-          csv << [NAME, PARENT, ORGANIZATIONS, LOCATIONS, SUBNET, DOMAIN, OPERATING_SYSTEM,
-                  ENVIRONMENT, COMPUTE_PROFILE, PARTITION_TABLE, MEDIUM, ARCHITECTURE, REALM,
-                  PUPPET_PROXY, PUPPET_CA_PROXY, CONTENT_SOURCE]
-          search_options = {:per_page => 999999}
-          search_options['search'] = "organization=\"#{option_organization}\"" if option_organization
-          @api.resource(:hostgroups).call(:index, search_options)['results'].each do |hostgroup|
-            hostgroup = @api.resource(:hostgroups).call(:show, {'id' => hostgroup['id']})
-            raise "Host Group 'id=#{hostgroup['id']}' not found" if !hostgroup || hostgroup.empty?
+      def export(csv)
+        csv << [NAME, PARENT, ORGANIZATIONS, LOCATIONS, SUBNET, DOMAIN, OPERATING_SYSTEM,
+                ENVIRONMENT, COMPUTE_PROFILE, PARTITION_TABLE, MEDIUM, ARCHITECTURE, REALM,
+                PUPPET_PROXY, PUPPET_CA_PROXY, CONTENT_SOURCE, PASSWORD, PUPPET_CLASSES]
+        search_options = {:per_page => 999999}
+        search_options['search'] = "organization=\"#{option_organization}\"" if option_organization
+        @api.resource(:hostgroups).call(:index, search_options)['results'].each do |hostgroup|
+          hostgroup = @api.resource(:hostgroups).call(:show, {'id' => hostgroup['id']})
+          raise "Host Group 'id=#{hostgroup['id']}' not found" if !hostgroup || hostgroup.empty?
 
-            name = hostgroup['name']
-            organizations = export_column(hostgroup, 'organizations', 'name')
-            locations = export_column(hostgroup, 'locations', 'name')
-            subnet = hostgroup['subnet_name']
-            operating_system = hostgroup['operatingsystem_name']
-            domain = hostgroup['domain_name']
-            puppet_environment = hostgroup['environment_name']
-            compute_profile = hostgroup['compute_profile_name']
-            partition_table = hostgroup['ptable_name']
-            medium = hostgroup['medium_name']
-            architecture = hostgroup['architecture_name']
-            realm = hostgroup['realm_name']
-            puppet_proxy = hostgroup['puppet_proxy_id'] ? foreman_host(:id => hostgroup['puppet_proxy_id']) : nil
-            puppet_ca_proxy = hostgroup['puppet_ca_proxy_id'] ? foreman_host(:id => hostgroup['puppet_ca_proxy_id']) : nil
-            # TODO: http://projects.theforeman.org/issues/7597
-            # content_source = hostgroup['content_source_id'] ? foreman_host(:id => hostgroup['content_source_id']) : nil
-            content_source = nil
-            parent = hostgroup['ancestry'] ? foreman_hostgroup(:id => hostgroup['ancestry']) : nil
-
-            csv << [name, parent, organizations, locations, subnet, domain, operating_system,
-                    puppet_environment, compute_profile, partition_table, medium, architecture,
-                    realm, puppet_proxy, puppet_ca_proxy, content_source]
+          name = hostgroup['name']
+          organizations = export_column(hostgroup, 'organizations', 'name')
+          locations = export_column(hostgroup, 'locations', 'name')
+          subnet = hostgroup['subnet_name']
+          operating_system = hostgroup['operatingsystem_name']
+          domain = hostgroup['domain_name']
+          puppet_environment = hostgroup['environment_name']
+          compute_profile = hostgroup['compute_profile_name']
+          partition_table = hostgroup['ptable_name']
+          medium = hostgroup['medium_name']
+          architecture = hostgroup['architecture_name']
+          realm = hostgroup['realm_name']
+          puppet_proxy = hostgroup['puppet_proxy_id'] ? foreman_host(:id => hostgroup['puppet_proxy_id']) : nil
+          puppet_ca_proxy = hostgroup['puppet_ca_proxy_id'] ? foreman_host(:id => hostgroup['puppet_ca_proxy_id']) : nil
+          content_source = hostgroup['content_source_id'] ? foreman_host(:id => hostgroup['content_source_id']) : nil
+          parent = hostgroup['ancestry'] ? foreman_hostgroup(:id => hostgroup['ancestry']) : nil
+          password = nil
+          puppet_classes = export_column(hostgroup, 'puppetclasses') do |puppet_class|
+            "#{puppet_class['module_name']}/#{puppet_class['name']}"
           end
+
+          # TODO: http://projects.theforeman.org/issues/6273
+          # API call to get the smart class variable override values
+
+          csv << [name, parent, organizations, locations, subnet, domain, operating_system,
+                  puppet_environment, compute_profile, partition_table, medium, architecture,
+                  realm, puppet_proxy, puppet_ca_proxy, content_source, password, puppet_classes]
         end
       end
 
       def import
         @existing = {}
-        @api.resource(:hostgroups).call(:index, {:per_page => 999999})['results'].each do |host|
-          @existing[host['name']] = host['id'] if host
+        @api.resource(:hostgroups).call(:index, {:per_page => 999999})['results'].each do |host_group|
+          @existing[host_group['name']] = host_group['id'] if host_group
         end
 
         thread_import do |line|
-          create_hosts_from_csv(line)
+          create_from_csv(line)
         end
       end
 
-      def create_hosts_from_csv(line)
+      def create_from_csv(line)
         return if option_organization && !CSV.parse_line(line[ORGANIZATIONS], {:skip_blanks => true}).include?(option_organization)
 
+        params = {
+          'hostgroup' => {
+            'architecture_id' => foreman_architecture(:name => line[ARCHITECTURE]),
+            'operatingsystem_id' => foreman_operatingsystem(:name => line[OPERATING_SYSTEM]),
+            'medium_id' => foreman_medium(:name => line[MEDIUM]),
+            'ptable_id' => foreman_partitiontable(:name => line[PARTITION_TABLE]),
+            'root_pass' => line[PASSWORD],
+            'organization_ids' => collect_column(line[ORGANIZATIONS]) do |organization|
+              foreman_organization(:name => organization)
+            end,
+            'location_ids' => collect_column(line[LOCATIONS]) do |location|
+              foreman_location(:name => location)
+            end
+          }
+        }
+
         count(line[COUNT]).times do |number|
           name = namify(line[NAME], number)
+          params['hostgroup']['name'] = name
+
           if !@existing.include? name
             print "Creating host group '#{name}'..." if option_verbose?
-            # @api.resource(:hosts).call(:create, {
-            #     'host' => {
-            #       'name' => name,
-            #       'root_pass' => 'changeme',
-            #       'mac' => namify(line[MACADDRESS], number),
-            #       'organization_id' => foreman_organization(:name => line[ORGANIZATION]),
-            #       'location_id' => foreman_location(:name => line[LOCATION]),
-            #       'environment_id' => foreman_environment(:name => line[ENVIRONMENT]),
-            #       'operatingsystem_id' => foreman_operatingsystem(:name => line[OPERATINGSYSTEM]),
-            #       'architecture_id' => foreman_architecture(:name => line[ARCHITECTURE]),
-            #       'domain_id' => foreman_domain(:name => line[DOMAIN]),
-            #       'ptable_id' => foreman_partitiontable(:name => line[PARTITIONTABLE])
-            #     }
-            # })
+            hostgroup = @api.resource(:hostgroups).call(:create, params)
+            @existing[name] = hostgroup['id']
           else
             print "Updating host '#{name}'..." if option_verbose?
-            # @api.resource(:hosts).call(:update, {
-            #     'id' => @existing[name],
-            #     'host' => {
-            #       'name' => name,
-            #       'mac' => namify(line[MACADDRESS], number),
-            #       'organization_id' => foreman_organization(:name => line[ORGANIZATION]),
-            #       'environment_id' => foreman_environment(:name => line[ENVIRONMENT]),
-            #       'operatingsystem_id' => foreman_operatingsystem(:name => line[OPERATINGSYSTEM]),
-            #       'architecture_id' => foreman_architecture(:name => line[ARCHITECTURE]),
-            #       'domain_id' => foreman_domain(:name => line[DOMAIN]),
-            #       'ptable_id' => foreman_partitiontable(:name => line[PARTITIONTABLE])
-            #     }
-            # })
+            params['id'] = @existing[name]
+            hostgroup = @api.resource(:hostgroups).call(:update, params)
+          end
+
+          # TODO: puppet classes
+          puppetclass_ids = collect_column(line[PUPPET_CLASSES]) do |puppet_class|
+            module_name, name = puppet_class.split('/')
+            foreman_puppet_class(:name => name)
           end
+          existing_ids = hostgroup['puppet_classes'].collect { |puppet_class| puppet_class['id'] }
+          # DELETE existing_ids - puppetclass_ids
+          # POST puppetclass_ids - existing_ids
+
           print "done\n" if option_verbose?
         end
-      rescue RuntimeError => e
-        raise "#{e}\n       #{line}"
       end
     end
   end

data/lib/hammer_cli_csv/hosts.rb

@@ -22,41 +22,39 @@ module HammerCLICsv
       ENABLED = 'Enabled'
       MANAGED = 'Managed'
 
-      def export
-        CSV.open(option_file || '/dev/stdout', 'wb', {:force_quotes => true}) do |csv|
-          csv << [NAME, ORGANIZATION, LOCATION, ENVIRONMENT, OPERATINGSYSTEM, ARCHITECTURE,
-                  MACADDRESS, DOMAIN, PARTITIONTABLE, SUBNET, REALM, MEDIUM, HOSTGROUP,
-                  COMPUTERESOURCE, COMPUTEPROFILE, IMAGE, ENABLED, MANAGED]
-          search_options = {:per_page => 999999}
-          search_options['search'] = "organization=\"#{option_organization}\"" if option_organization
-          @api.resource(:hosts).call(:index, search_options)['results'].each do |host|
-            host = @api.resource(:hosts).call(:show, {'id' => host['id']})
-            raise "Host 'id=#{host['id']}' not found" if !host || host.empty?
+      def export(csv)
+        csv << [NAME, ORGANIZATION, LOCATION, ENVIRONMENT, OPERATINGSYSTEM, ARCHITECTURE,
+                MACADDRESS, DOMAIN, PARTITIONTABLE, SUBNET, REALM, MEDIUM, HOSTGROUP,
+                COMPUTERESOURCE, COMPUTEPROFILE, IMAGE, ENABLED, MANAGED]
+        search_options = {:per_page => 999999}
+        search_options['search'] = "organization=\"#{option_organization}\"" if option_organization
+        @api.resource(:hosts).call(:index, search_options)['results'].each do |host|
+          host = @api.resource(:hosts).call(:show, {'id' => host['id']})
+          raise "Host 'id=#{host['id']}' not found" if !host || host.empty?
 
-            name = host['name']
-            organization = host['organization_name']
-            location = host['location_name']
-            environment = host['environment_name']
-            operatingsystem = host['operatingsystem_name']
-            architecture = host['architecture_name']
-            mac = host['mac']
-            domain = host['domain_name']
-            ptable = host['ptable_name']
-            subnet = host['subnet_name']
-            realm = host['realm_name']
-            medium = host['medium_name']
-            hostgroup = host['hostgroup_name']
-            compute_resource = host['compute_resource_name']
-            compute_profile = host['compute_profile_name']
-            image = host['image_name']
+          name = host['name']
+          organization = host['organization_name']
+          location = host['location_name']
+          environment = host['environment_name']
+          operatingsystem = host['operatingsystem_name']
+          architecture = host['architecture_name']
+          mac = host['mac']
+          domain = host['domain_name']
+          ptable = host['ptable_name']
+          subnet = host['subnet_name']
+          realm = host['realm_name']
+          medium = host['medium_name']
+          hostgroup = host['hostgroup_name']
+          compute_resource = host['compute_resource_name']
+          compute_profile = host['compute_profile_name']
+          image = host['image_name']
 
-            enabled = host['enabled'] ? 'Yes' : 'No'
-            managed = host['managed'] ? 'Yes' : 'No'
+          enabled = host['enabled'] ? 'Yes' : 'No'
+          managed = host['managed'] ? 'Yes' : 'No'
 
-            csv << [name, organization, location, environment, operatingsystem, architecture,
-                    mac, domain, ptable, subnet, realm, medium, hostgroup, compute_resource,
-                    compute_profile, image, enabled, managed]
-          end
+          csv << [name, organization, location, environment, operatingsystem, architecture,
+                  mac, domain, ptable, subnet, realm, medium, hostgroup, compute_resource,
+                  compute_profile, image, enabled, managed]
         end
       end
 
@@ -110,8 +108,6 @@ module HammerCLICsv
           end
           print "done\n" if option_verbose?
         end
-      rescue RuntimeError => e
-        raise "#{e}\n       #{line}"
       end
     end
   end