hammer_cli_foreman 3.0.0 → 3.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8287acd52ceb1bea84f8d518c3793d521d084ae771622ea7fc4dcccccd794a1b
-  data.tar.gz: b557d8fbbc7a451cd09084211c6e8a46a46ef3c198744cbb9c8dc7e861090e93
+  metadata.gz: 12d3353f9d530459e2a7f58e8f5603b1f52af4b2ee28aa51c6252bda84f5a5b3
+  data.tar.gz: cb323fd78217f8d5b5b1e7b69dd691ea85b3d332dcbbc2e375a668fc7043c298
 SHA512:
-  metadata.gz: 283fd640422b5749f6dc9094907bb1c68e72cef15696704f4493f548dd23ca13bd1066fca15616996d4ff6ba48648d904f5633e3cc3305c30464fa9e22c8bf44
-  data.tar.gz: 94b81f8171c914bdd1435ad00a17c1b1606402bac1ab9b21ca3014056ed285029bc79fe2b0646630d964d3b4715bd94b42a67a35f5399cace731ec44c87e4ee8
+  metadata.gz: 04b1444a37f464ab35afdfd3832cf0549448484e9932972603ca5d29d7660a4bd599ab0cae461730a7fca5735712451553983724cbe5a23d8f6ffb0135e5510e
+  data.tar.gz: b625099ac29545814e7c73a1d74f0b20559b117bfabac2e2074c448557386c40e81a280ae2fb31271e3e5bd60b1c29c4a60881acfe4d5cbcad3f91cda05ca21c

data/doc/release_notes.md

@@ -1,5 +1,18 @@
 Release notes
 =============
+### 3.1.0 (2021-11-10)
+* Update test data for 3.1
+* Make sure provided options override defaults, [#33711](http://projects.theforeman.org/issues/33711)
+* Make template create/update commands use resolver, [#33721](http://projects.theforeman.org/issues/33721)
+* Revert fix rake version
+* Fix rake version
+* Show status once for proxy info ([PR #581](https://github.com/theforeman/hammer-cli-foreman/pull/581)), [#19510](http://projects.theforeman.org/issues/19510)
+* Add proxy status and version to info command ([PR #572](https://github.com/theforeman/hammer-cli-foreman/pull/572))
+* Associate a vm via hammer to a compute resource, [#33047](http://projects.theforeman.org/issues/33047)
+* Fix option builders and tests, [#33226](http://projects.theforeman.org/issues/33226)
+* Bump to 3.1.0-develop
+* Add parent title option for hostgroup, [#32878](http://projects.theforeman.org/issues/32878)
+
 ### 3.0.0 (2021-08-04)
 * Puppet extraction ([PR #571](https://github.com/theforeman/hammer-cli-foreman/pull/571)), [#33174](http://projects.theforeman.org/issues/33174)
 * Update rel-eng notebook ([PR #573](https://github.com/theforeman/hammer-cli-foreman/pull/573))

data/lib/hammer_cli_foreman/command_extensions/subnet.rb

@@ -1,16 +1,7 @@
 module HammerCLIForeman
   module CommandExtensions
     class Subnet < HammerCLI::CommandExtensions
-      option "--dns", "DNS_NAME", _("DNS Proxy to use within this subnet"),
-             attribute_name: :option_dns_name,
-             referenced_resource: :smart_proxy
-      option "--dhcp", "DHCP_NAME", _("DHCP Proxy to use within this subnet"),
-             attribute_name: :option_dhcp_name,
-             referenced_resource: :smart_proxy
-      option "--tftp", "TFTP_NAME", _("TFTP Proxy to use within this subnet"),
-             attribute_name: :option_tftp_name,
-             referenced_resource: :smart_proxy
-      option "--prefix", "PREFIX", _("Network prefix in CIDR notation (e.g. 64) for this subnet")
+      option '--prefix', 'PREFIX', _('Network prefix in CIDR notation (e.g. 64) for this subnet')
 
       option_sources do |sources, command|
         sources.find_by_name('IdResolution').insert_relative(
@@ -20,6 +11,30 @@ module HammerCLIForeman
         )
         sources
       end
+
+      option_family associate: 'tftp' do
+        child '--tftp', 'TFTP_NAME', _('TFTP Proxy to use within this subnet'),
+              attribute_name: :option_tftp_name,
+              referenced_resource: :smart_proxy
+      end
+
+      option_family associate: 'dns' do
+        child '--dns', 'DNS_NAME', _('DNS Proxy to use within this subnet'),
+              attribute_name: :option_dns_name,
+              referenced_resource: :smart_proxy
+      end
+
+      option_family associate: 'dhcp' do
+        child '--dhcp', 'DHCP_NAME', _('DHCP Proxy to use within this subnet'),
+              attribute_name: :option_dhcp_name,
+              referenced_resource: :smart_proxy
+      end
+
+      option_family associate: 'bmc' do
+        child '--bmc', 'BMC_NAME', _('BMC Proxy to use within this subnet'),
+              attribute_name: :option_bmc_name,
+              referenced_resource: :smart_proxy
+      end
     end
   end
 end

data/lib/hammer_cli_foreman/command_extensions/user.rb

@@ -1,17 +1,21 @@
 module HammerCLIForeman
   module CommandExtensions
     class User < HammerCLI::CommandExtensions
-      option '--default-organization', 'DEFAULT_ORGANIZATION_NAME', _('Default organization name'),
-             aliased_resource: 'default_organization', referenced_resource: 'default_organization'
-      option '--default-location', 'DEFAULT_LOCATION_NAME', _('Default location name'),
-             aliased_resource: 'default_location', referenced_resource: 'default_location'
-
       option '--ask-password', 'ASK_PW', ' ', format: HammerCLI::Options::Normalizers::Bool.new
 
       option_sources do |sources, command|
         sources << HammerCLIForeman::OptionSources::UserParams.new(command)
         sources
       end
+
+      option_family associate: 'default_organization' do
+        child '--default-organization', 'DEFAULT_ORGANIZATION_NAME', _('Default organization name'),
+              aliased_resource: 'default_organization', referenced_resource: 'organization'
+      end
+      option_family associate: 'default_location' do
+        child '--default-location', 'DEFAULT_LOCATION_NAME', _('Default location name'),
+              aliased_resource: 'default_location', referenced_resource: 'location'
+      end
     end
   end
 end

data/lib/hammer_cli_foreman/commands.rb

@@ -97,9 +97,8 @@ module HammerCLIForeman
       configurator = BuilderConfigurator.new(searchables, dependency_resolver)
 
       builder = ForemanOptionBuilder.new(searchables)
-      builder.builders = []
+      builder.builders = super.builders
       builder.builders += configurator.builders_for(resource, resource.action(action)) if resource_defined?
-      builder.builders += super.builders
       builder
     end
 

data/lib/hammer_cli_foreman/compute_attribute.rb

@@ -12,7 +12,7 @@ module HammerCLIForeman
       profile = HammerCLIForeman.record_to_common_format(
           HammerCLIForeman.foreman_resource(:compute_profiles).call(:show, 'id' => options['option_compute_profile_id'] )
       )
-      params['compute_attribute'] = profile['compute_attributes'].select { |hash| hash['compute_resource_id'] == options['option_compute_resource_id']}[0] || {}
+      params['compute_attribute'] = profile['compute_attributes'].select { |hash| hash['compute_resource_id'].to_s == options['option_compute_resource_id'].to_s}[0] || {}
       params['compute_attribute'].delete('attributes') if params['compute_attribute']['attributes']
       params
     end

data/lib/hammer_cli_foreman/compute_resource.rb

@@ -107,6 +107,7 @@ module HammerCLIForeman
     class AssociateVmsCommand < HammerCLIForeman::Command
       action :associate
       command_name 'associate-vms'
+      option "--vm-id","VM ID", _("Associate a specific VM")
       success_message _("Virtual machines have been associated.")
       failure_message _("Could not associate the virtual machines")
 

data/lib/hammer_cli_foreman/hostgroup.rb

@@ -3,13 +3,20 @@ module HammerCLIForeman
   module HostgroupUpdateCreateCommons
 
     def self.included(base)
-      base.option "--parent", "PARENT_NAME",  _("Name of parent hostgroup")
-      base.option ["--root-password"], "ROOT_PASSWORD",  _("Root password")
-      base.option ["--ask-root-password", "--ask-root-pass"], "ASK_ROOT_PW", "",
+      base.option '--root-password', 'ROOT_PASSWORD', _('Root password')
+      base.option '--ask-root-password', 'ASK_ROOT_PW', '',
                   format: HammerCLI::Options::Normalizers::Bool.new
-      base.option "--subnet6", "SUBNET6_NAME", _("Subnet IPv6 name")
 
       base.build_options without: %i[root_pass]
+
+      base.option_family associate: 'subnet6' do
+        child '--subnet6', 'SUBNET6_NAME', _('Subnet IPv6 name')
+      end
+
+      base.option_family associate: 'parent' do
+        child '--parent', 'PARENT_NAME', _('Name of parent hostgroup')
+        child '--parent-title', 'PARENT_TITLE', _('Title of parent hostgroup')
+      end
     end
 
     def self.ask_password
@@ -19,8 +26,8 @@ module HammerCLIForeman
 
     def request_params
       params = super
-      params['hostgroup']["parent_id"] ||= resolver.hostgroup_id('option_name' => option_parent) if option_parent
-
+      params['hostgroup']['parent_id'] ||= resolver.hostgroup_id('option_name' => option_parent) if option_parent
+      params['hostgroup']['parent_id'] ||= resolver.hostgroup_id('option_title' => option_parent_title) if option_parent_title
       params['hostgroup']['root_pass'] = option_root_password if option_root_password
       params['hostgroup']['root_pass'] = HammerCLIForeman::HostgroupUpdateCreateCommons::ask_password if option_ask_root_password
 

data/lib/hammer_cli_foreman/hosts/common_update_options.rb

@@ -3,11 +3,12 @@ module HammerCLIForeman
     module CommonUpdateOptions
 
       def self.included(base)
-        base.option "--owner", "OWNER_LOGIN", _("Login of the owner"),
-          :attribute_name => :option_user_login
-        base.option "--owner-id", "OWNER_ID", _("ID of the owner"),
-          :attribute_name => :option_owner_id
-
+        base.option_family do
+          parent '--owner-id', 'OWNER_ID', _('ID of the owner'),
+                 attribute_name: :option_owner_id
+          child '--owner', 'OWNER_LOGIN', _('Login of the owner'),
+                attribute_name: :option_user_login
+        end
         base.option "--root-password", "ROOT_PW",
           _("Required if host is managed and value is not inherited from host group or default password in settings")
 
@@ -112,14 +113,13 @@ module HammerCLIForeman
 
       private
 
-     
       def owner_id(name, type)
         return unless name
         return resolver.usergroup_id('option_name' => name) if type == 'Usergroup'
 
         resolver.user_id('option_login' => name)
       end
-      
+
       def proxy_id(name)
         resolver.smart_proxy_id('option_name' => name) if name
       end

data/lib/hammer_cli_foreman/id_resolver.rb

@@ -1,14 +1,12 @@
 module HammerCLIForeman
 
   class Searchable
-    attr_reader :name, :description, :format
+    attr_reader :name, :description
 
     def initialize(name, description, options={})
       @name = name
       @description = description
       @editable = options[:editable].nil? ? true : options[:editable]
-      @format = options[:format]
-      @parent = options[:parent]
     end
 
     def plural_name
@@ -18,10 +16,6 @@ module HammerCLIForeman
     def editable?
       @editable
     end
-
-    def parent?
-      @parent
-    end
   end
 
   class Searchables
@@ -130,6 +124,12 @@ module HammerCLIForeman
       @searchables.for(resource)
     end
 
+    def template_kind_id(options = {})
+      name = options['option_type'] == 'snippet' ? nil : options['option_type']
+      options['option_name'] = name
+      get_id(:template_kinds, options)
+    end
+
     protected
 
     def define_id_finders

data/lib/hammer_cli_foreman/option_builders.rb

@@ -172,8 +172,7 @@ module HammerCLIForeman
     def build(builder_params={})
       resource_name_map = builder_params[:resource_mapping] || {}
       @searchables.for(@resource).collect do |s|
-        family = HammerCLI::Options::OptionFamily.new
-        family.parent(
+        option(
           optionamize("--#{s.name}"),
           s.name.upcase,
           s.description,
@@ -213,23 +212,43 @@ module HammerCLIForeman
 
     def build(builder_params={})
       resource_name_map = builder_params[:resource_mapping] || {}
-      dependent_options(@resource, resource_name_map)
+      command = builder_params[:command]
+      dependent_options(@resource, resource_name_map, command)
     end
 
     protected
 
-    def dependent_options(resource, resource_name_map)
+    def dependent_options(resource, resource_name_map, command)
       options = []
       searchables = @searchables.for(resource)
       resource_name = resource.singular_name
       aliased_name = aliased(resource_name, resource_name_map)
       types = searchables.map(&:name).push('id').map(&:capitalize).join('/')
       associated_resource = aliased_name.to_s.tr('_', ' ')
-      family = HammerCLI::Options::OptionFamily.new(
-        referenced_resource: resource_name,
-        aliased_resource: aliased_name,
-        description: _('%{types} of associated %{resource}') % { types: types, resource: associated_resource }
-      )
+      family = command.option_families.find do |f|
+        f.head.referenced_resource == resource_name
+      end
+      unless family
+        # If there is no family it probably means that we explicitly want to
+        # expand options via
+        # build_options do |o|
+        #   o.expand.including(:resource)
+        # end
+        # instead of generating it from API docs (it may not be even present there)
+        family = HammerCLI::Options::OptionFamily.new(
+          description: _('%{types} of associated %{resource}') % { types: types, resource: associated_resource },
+          creator: command
+        )
+        options << family.parent(
+          optionamize("--#{aliased_name}-id"),
+          "#{aliased_name}_id".upcase,
+          description("#{aliased_name}_id", @context[:action]),
+          attribute_name: HammerCLI.option_accessor_name("#{resource_name}_id"),
+          referenced_resource: resource_name,
+          aliased_resource: aliased_name,
+          format: HammerCLI::Options::Normalizers::Number.new
+        )
+      end
 
       unless searchables.empty?
         first = searchables[0]
@@ -240,31 +259,25 @@ module HammerCLIForeman
         options << family.child(
           optionamize("--#{aliased_name}"),
           "#{aliased_name}_#{first.name}".upcase,
-          first.description || " ",
-          attribute_name: HammerCLI.option_accessor_name("#{resource_name}_#{first.name}")
+          first.description || ' ',
+          attribute_name: HammerCLI.option_accessor_name("#{resource_name}_#{first.name}"),
+          referenced_resource: resource_name,
+          aliased_resource: aliased_name
         )
         # Other options are named by the resource plus the searchable name
         # Eg. --organization-label with accessor option_organization_label
         remaining.each do |s|
-          options << family.send(s.parent? ? :parent : :child,
+          options << family.child(
             optionamize("--#{aliased_name}-#{s.name}"),
             "#{aliased_name}_#{s.name}".upcase,
-            s.description || " ",
+            s.description || ' ',
             attribute_name: HammerCLI.option_accessor_name("#{resource_name}_#{s.name}"),
-            format: s.format
+            referenced_resource: resource_name,
+            aliased_resource: aliased_name
           )
         end
       end
 
-      unless options.any? { |o| o.handles?(optionamize("--#{aliased_name}-id")) }
-        options << family.parent(
-          optionamize("--#{aliased_name}-id"),
-          "#{aliased_name}_id".upcase,
-          description("#{aliased_name}_id", @context[:action]),
-          attribute_name: HammerCLI.option_accessor_name("#{resource_name}_id"),
-          format: HammerCLI::Options::Normalizers::Number.new
-        )
-      end
       options
     end
 
@@ -285,15 +298,25 @@ module HammerCLIForeman
   class UpdateDependentSearchablesOptionBuilder < DependentSearchablesOptionBuilder
     protected
 
-    def dependent_options(resource, resource_name_map)
+    def dependent_options(resource, resource_name_map, command)
       options = []
       searchables = @searchables.for(resource)
       resource_name = resource.singular_name
       aliased_name = aliased(resource_name, resource_name_map)
+      # Those options are not coming from API docs, thus create a new family only
       family = HammerCLI::Options::OptionFamily.new(
         prefix: 'new-',
         aliased_resource: aliased_name,
-        referenced_resource: resource_name
+        referenced_resource: resource_name,
+        creator: command
+      )
+
+      options << family.parent(
+        optionamize("--new-#{aliased_name}-id"),
+        "new_#{aliased_name}_id".upcase,
+        _('Use to update associated %s') % aliased_name,
+        attribute_name: HammerCLI.option_accessor_name("new_#{resource_name}_id"),
+        format: HammerCLI::Options::Normalizers::Number.new
       )
       unless searchables.empty?
         first = searchables[0]
@@ -304,73 +327,63 @@ module HammerCLIForeman
         options << family.child(
           optionamize("--new-#{aliased_name}"),
           "new_#{aliased_name}_#{first.name}".upcase,
-          _('Use to update'),
+          _('Use to update associated %s') % aliased_name,
           attribute_name: HammerCLI.option_accessor_name("new_#{resource_name}_#{first.name}")
         )
         # Other options are named by the resource plus the searchable name
         # Eg. --new-organization-label with accessor option_new_organization_label
         remaining.each do |s|
-          options << family.send(s.parent? ? :parent : :child,
+          options << family.child(
             optionamize("--new-#{aliased_name}-#{s.name}"),
             "new_#{aliased_name}_#{s.name}".upcase,
-            _('Use to update'),
+            _('Use to update associated %s') % aliased_name,
             attribute_name: HammerCLI.option_accessor_name("new_#{resource_name}_#{s.name}")
           )
         end
       end
 
-      unless options.any? { |o| o.handles?(optionamize("--new-#{aliased_name}-id")) }
-        options << family.parent(
-          optionamize("--new-#{aliased_name}-id"),
-          "new_#{aliased_name}_id".upcase,
-          _('Use to update'),
-          attribute_name: HammerCLI.option_accessor_name("new_#{resource_name}_id"),
-          format: HammerCLI::Options::Normalizers::Number.new
-        )
-      end
       options
     end
   end
 
   class DependentSearchablesArrayOptionBuilder < DependentSearchablesOptionBuilder
-
-
-    def dependent_options(resource, resource_name_map)
+    def dependent_options(resource, resource_name_map, command)
       options = []
       searchables = @searchables.for(resource)
       resource_name = resource.singular_name
 
       aliased_name = aliased(resource_name, resource_name_map)
       aliased_plural_name = aliased(resource.name, resource_name_map)
-
       unless searchables.empty?
         first = searchables[0]
         remaining = searchables[1..-1] || []
-        types = searchables.map(&:plural_name).map(&:capitalize).join('/')
-        associated_resource = aliased_plural_name.to_s.tr('_', ' ')
-        family = HammerCLI::Options::OptionFamily.new(
-          format: HammerCLI::Options::Normalizers::List.new,
-          referenced_resource: resource_name,
-          aliased_resource: aliased_name,
-          description: _('%{types} of associated %{resource}') % { types: types, resource: associated_resource }
-        )
+        # Find family created by HammerCLI::Apipie::OptionBuilder for *_ids options
+        family = command.option_families.find do |f|
+          f.head.referenced_resource == resource_name && f.formats.include?(HammerCLI::Options::Normalizers::List)
+        end
         # First option is named by the resource
         # Eg. --organizations with accessor option_organization_names
         options << family.child(
           optionamize("--#{aliased_plural_name}"),
           "#{aliased_name}_#{first.plural_name}".upcase,
           ' ',
-          attribute_name: HammerCLI.option_accessor_name("#{resource_name}_#{first.plural_name}")
+          attribute_name: HammerCLI.option_accessor_name("#{resource_name}_#{first.plural_name}"),
+          format: HammerCLI::Options::Normalizers::List.new,
+          referenced_resource: resource_name,
+          aliased_resource: aliased_name
         )
 
         # Other options are named by the resource plus the searchable name
         # Eg. --organization-labels with accessor option_organization_labels
         remaining.each do |s|
-          options << family.send(s.parent? ? :parent : :child,
+          options << family.child(
             optionamize("--#{aliased_name}-#{s.plural_name}"),
             "#{aliased_name}_#{s.plural_name}".upcase,
             ' ',
-            attribute_name: HammerCLI.option_accessor_name("#{resource_name}_#{s.plural_name}")
+            attribute_name: HammerCLI.option_accessor_name("#{resource_name}_#{s.plural_name}"),
+            format: HammerCLI::Options::Normalizers::List.new,
+            referenced_resource: resource_name,
+            aliased_resource: aliased_name
           )
         end
       end
@@ -421,4 +434,3 @@ module HammerCLIForeman
   end
 
 end
-

data/lib/hammer_cli_foreman/option_sources/id_params.rb

@@ -6,7 +6,7 @@ module HammerCLIForeman
       end
 
       def param_updatable?(param_resource)
-        param_resource && @command.respond_to?(HammerCLI.option_accessor_name("#{param_resource.singular_name}_id"))
+        param_resource && @command.respond_to?(cli_option_resource_id(param_resource))
       end
 
       def available_id_params
@@ -16,16 +16,23 @@ module HammerCLIForeman
       def needs_resolving?(param_option, param_resource, all_opts)
         return false unless param_updatable?(param_resource)
 
-        searchables_set = @command.searchables.for(param_resource).any? do |s|
+        cli_searchables_set = @command.searchables.for(param_resource).any? do |s|
           option = HammerCLI.option_accessor_name("#{param_resource.singular_name}_#{s.name}")
-          !all_opts[option].nil?
+          next false unless @command.respond_to?(option)
+
+          !@command.send(option).nil?
         end
-        return all_opts[param_option].nil? unless searchables_set
+        if cli_searchables_set
+          # Remove set '<resource_name>_id' option to force resolving in case of
+          # '<resource_name>_[name|title]' was set from CLI
+          all_opts.delete(param_option)
+          true
+        else
+          # Don't resolve if resource id was set via CLI
+          return false if @command.send(cli_option_resource_id(param_resource))
 
-        # Remove set '<resource_name>_id' option to force resolving in case of
-        # '<resource_name>_[name|title]' was set
-        all_opts.delete(param_option)
-        true
+          all_opts[param_option].nil?
+        end
       end
 
       def get_options(_defined_options, result)
@@ -63,6 +70,12 @@ module HammerCLIForeman
           e.resource
         )
       end
+
+      private
+
+      def cli_option_resource_id(resource)
+        HammerCLI.option_accessor_name("#{resource.singular_name}_id")
+      end
     end
   end
 end

data/lib/hammer_cli_foreman/option_sources/ids_params.rb

@@ -6,7 +6,7 @@ module HammerCLIForeman
       end
 
       def param_updatable?(param_resource)
-        param_resource && @command.respond_to?(HammerCLI.option_accessor_name("#{param_resource.singular_name}_ids"))
+        param_resource && @command.respond_to?(cli_option_resource_ids(param_resource))
       end
 
       def available_ids_params
@@ -16,16 +16,23 @@ module HammerCLIForeman
       def needs_resolving?(param_option, param_resource, all_opts)
         return false unless param_updatable?(param_resource)
 
-        searchables_set = @command.searchables.for(param_resource).any? do |s|
+        cli_searchables_set = @command.searchables.for(param_resource).any? do |s|
           option = HammerCLI.option_accessor_name("#{param_resource.singular_name}_#{s.plural_name}")
-          !all_opts[option].nil?
-        end
-        return all_opts[param_option].nil? unless searchables_set
+          next false unless @command.respond_to?(option)
 
-        # Remove set '<resource_name>_ids' option to force resolving in case of
-        # '<resource_name>_[names|titles]' was set
-        all_opts.delete(param_option)
-        true
+          !@command.send(option).nil?
+        end
+        if cli_searchables_set
+          # Remove set '<resource_name>_ids' option to force resolving in case of
+          # '<resource_name>_[names|titles]' were set from CLI
+          all_opts.delete(param_option)
+          true
+        else
+          # Don't resolve if resource ids were set via CLI
+          return false if @command.send(cli_option_resource_ids(param_resource))
+
+          all_opts[param_option].nil?
+        end
       end
 
       def get_options(_defined_options, result)
@@ -44,6 +51,12 @@ module HammerCLIForeman
         end
         result
       end
+
+      private
+
+      def cli_option_resource_ids(resource)
+        HammerCLI.option_accessor_name("#{resource.singular_name}_ids")
+      end
     end
   end
 end

data/lib/hammer_cli_foreman/smart_proxy.rb

@@ -10,8 +10,9 @@ module HammerCLIForeman
       output do
         field :id, _("Id")
         field :name, _("Name")
+        field :status, _("Status")
         field :url, _("URL")
-        field :_features, _( "Features"), Fields::List, :width => 25, :hide_blank => true
+        field :_features, _( "Features"), Fields::List, :hide_blank => true
       end
 
       def extend_data(proxy)
@@ -22,12 +23,13 @@ module HammerCLIForeman
       build_options
     end
 
-
     class InfoCommand < HammerCLIForeman::InfoCommand
-
       output ListCommand.output_definition do
-        collection :features,  _("Features"), :numbered => false do
-          custom_field Fields::Reference
+        field :version, _("Version")
+        field :hosts_count, _("Host_count")
+        collection :features, _("Features") do
+          field :name, _('Name')
+          field :version, _('Version')
         end
         HammerCLIForeman::References.taxonomies(self)
         HammerCLIForeman::References.timestamps(self)

data/lib/hammer_cli_foreman/template.rb

@@ -5,25 +5,13 @@ module HammerCLIForeman
     resource :provisioning_templates
 
     module TemplateCreateUpdateCommons
-
       def option_snippet
         option_type && (option_type == "snippet")
       end
 
-      def option_template_kind_id
-        table = kinds.inject({}){ |result, k| result.update(k["name"] => k["id"]) }
-        if option_snippet == false && table[option_type].nil?
-          signal_usage_error _("unknown template kind")
-        else
-          table[option_type]
-        end
-      end
-
-      def kinds
-        HammerCLIForeman.collection_to_common_format(
-          HammerCLIForeman.foreman_resource!(:template_kinds).call(:index))
-      end
-
+      # This method is for IdParams option source to make resolver work for
+      # --type option
+      def option_template_kind_id; end
     end
 
     class ListCommand < HammerCLIForeman::ListCommand