hammer_cli_foreman 0.18.2 → 0.19.0

data/lib/hammer_cli_foreman/api/session_authenticator_wrapper.rb

@@ -3,31 +3,29 @@ require 'uri'
 module HammerCLIForeman
   module Api
     class SessionAuthenticatorWrapper < ApipieBindings::Authenticators::Base
-      SESSION_STORAGE = '~/.hammer/sessions/'
 
-      attr_reader :session_id
+      attr_reader :session_id, :url, :auth_type
 
-      def initialize(authenticator, url, storage_dir = nil)
+      def initialize(authenticator, url, auth_type)
         @authenticator = authenticator
         @url = url
+        @auth_type = auth_type
+      end
 
-        @session_file = "#{uri.scheme}_#{uri.host}"
-        @storage_dir = storage_dir || File.expand_path(SESSION_STORAGE)
-
-        @permissions_ok = check_storage_permissions
-        warn _("Can't use session auth due to invalid permissions on session files.") unless @permissions_ok
+      def session
+        @session ||= Sessions.get(@url)
       end
 
       def clear
-        destroy_session
+        session.destroy
         @authenticator.clear if @authenticator.respond_to?(:clear)
       end
 
       def status
-        if load_session
-          _("Session exists, currently logged in as '%s'.") % @user
+        if session.valid?
+          _("Session exists, currently logged in as '%s'.") % session.user_name
         else
-          _("Using sessions, you are currently not logged in.")
+          _('Using sessions, you are currently not logged in.')
         end
       end
 
@@ -40,15 +38,13 @@ module HammerCLIForeman
       end
 
       def authenticate(request, args)
-        load_session
-
         user = @authenticator.user
 
-        @user_changed ||= (!user.nil? && user != @user)
+        @user_changed ||= (!user.nil? && user != session.user_name)
 
-        if !@user_changed && @permissions_ok && @session_id
+        if !@user_changed && Sessions.configured?(@url) && session.id
           jar = HTTP::CookieJar.new
-          jar.add(HTTP::Cookie.new('_session_id', @session_id, domain: uri.hostname.downcase, path: '/', for_domain: true))
+          jar.add(HTTP::Cookie.new('_session_id', session.id, domain: uri.hostname.downcase, path: '/', for_domain: true))
           request['Cookie'] = HTTP::Cookie.cookie_value(jar.cookies)
           request
         else
@@ -57,12 +53,11 @@ module HammerCLIForeman
       end
 
       def error(ex)
-        load_session
-        if ex.is_a?(RestClient::Unauthorized) && !@session_id.nil?
+        if ex.is_a?(RestClient::Unauthorized) && session.valid?
           if @user_changed
-            return UnauthorizedError.new(_("Invalid username or password, continuing with session for '%s'.") % @user)
+            return UnauthorizedError.new(_("Invalid username or password, continuing with session for '%s'.") % session.user_name)
           else
-            destroy_session
+            session.destroy
             return SessionExpired.new(_("Session has expired."))
           end
         else
@@ -72,81 +67,40 @@ module HammerCLIForeman
 
       def response(r)
         if (r.cookies['_session_id'] && r.code != 401)
-          @session_id = r.cookies['_session_id']
-          save_session(@session_id, @authenticator.user)
+          session.id = r.cookies['_session_id']
+          session.user_name = @authenticator.user
+          session.auth_type = @auth_type
+          session.store
         end
         @authenticator.response(r)
       end
 
       def user(ask=nil)
-        @authenticator.user(ask) if @authenticator.respond_to?(:user)
+        return unless @authenticator.respond_to?(:user)
+        if @auth_type == AUTH_TYPES[:basic_auth]
+          @authenticator.user(ask)
+        elsif @auth_type == AUTH_TYPES[:oauth_authentication_code_grant] ||
+              @auth_type = AUTH_TYPES[:oauth_password_grant]
+          @authenticator.user
+        end
       end
 
       def password(ask=nil)
         @authenticator.password(ask) if @authenticator.respond_to?(:password)
       end
 
-      def set_credentials(*args)
-        @authenticator.set_credentials(*args) if @authenticator.respond_to?(:set_credentials)
+      def set_auth_params(*args)
+        if @auth_type == AUTH_TYPES[:basic_auth]
+          @authenticator.set_credentials(*args)
+        elsif @auth_type == AUTH_TYPES[:oauth_authentication_code_grant] ||
+              @auth_type == AUTH_TYPES[:oauth_password_grant]
+          @authenticator.set_token(*args)
+        end
       end
 
-      protected
-
       def uri
         @uri ||= URI.parse(@url)
       end
-
-      def session_storage
-        "#{@storage_dir}/#{@session_file}"
-      end
-
-      def load_session
-        if File.exist?(session_storage)
-          session_data = JSON.parse(File.read(session_storage))
-          @user = session_data['user_name']
-          @session_id = session_data['session_id']
-        end
-      rescue JSON::ParserError
-        destroy_session
-        warn _('Invalid session file format.')
-        nil
-      end
-
-      def save_session(session_id, user_name)
-        File.open(session_storage, 'w', 0600) do |f|
-          session = JSON.generate({
-            :session_id => session_id,
-            :user_name => user_name
-          })
-          f.write(session)
-        end
-      end
-
-      def destroy_session
-        @user = @session_id = nil
-        File.delete(session_storage) if File.exist?(session_storage)
-      end
-
-      def check_storage_permissions
-        Dir.mkdir(@storage_dir, 0700) unless File.exist?(@storage_dir)
-        ensure_mode(@storage_dir, '40700') && ensure_mode(session_storage, '100600')
-      end
-
-      def ensure_mode(file, expected_mode)
-        return true unless File.exist?(file)
-        mode = File.stat(file).mode.to_s(8)
-        if mode != expected_mode
-          warn _("Invalid permissions for %{file}: %{mode}, expected %{expected_mode}.") % {
-            :mode => mode,
-            :expected_mode => expected_mode,
-            :file => file
-          }
-          false
-        else
-          true
-        end
-      end
-
     end
   end
 end

data/lib/hammer_cli_foreman/auth.rb

@@ -1,32 +1,67 @@
-module HammerCLIForeman
+require 'hammer_cli_foreman/openid_connect.rb'
+require 'hammer_cli_foreman/authenticate/login.rb'
 
+module HammerCLIForeman
   class Auth < HammerCLI::AbstractCommand
-
     class LoginCommand < HammerCLI::AbstractCommand
-      command_name "login"
+      command_name 'login'
       desc _("Set credentials")
 
-      option ["-u", "--username"], "USERNAME", _("Username to access the remote system")
-      option ["-p", "--password"], "PASSWORD", _("Password to access the remote system")
+      class Basic < HammerCLI::AbstractCommand
+        extend HammerCLIForeman::Authenticate::Login
 
-      def execute
-        if !(HammerCLIForeman.foreman_api_connection.authenticator.is_a?(HammerCLIForeman::Api::SessionAuthenticatorWrapper))
-          print_message(_("Can't perform login. Make sure sessions are enabled in hammer configuration file."))
-          return HammerCLI::EX_USAGE
+        command_name('basic')
+        desc('provide username and password')
+
+        option ["-u", "--username"], "USERNAME", _("Username to access the remote system")
+        option ["-p", "--password"], "PASSWORD", _("Password to access the remote system")
+
+        def execute
+          Basic.execute_with_params(
+            AUTH_TYPES[:basic_auth],
+            option_username || HammerCLI::Settings.get('_params', 'username'),
+            option_password || HammerCLI::Settings.get('_params', 'password')
+          )
         end
+      end
 
-        # Make sure we reflect also credentials set for the main hammer command
-        # ( hammer -u test auth login )
-        HammerCLIForeman.foreman_api_connection.authenticator.set_credentials(
-          option_username || HammerCLI::Settings.get('_params', 'username'),
-          option_password || HammerCLI::Settings.get('_params', 'password')
-        )
-        HammerCLIForeman.foreman_api_connection.authenticator.force_user_change
-        HammerCLIForeman.foreman_api_connection.login
+      class Oauth < HammerCLI::AbstractCommand
+        extend HammerCLIForeman::Authenticate::Login
 
-        print_message(_("Successfully logged in as '%s'.") % HammerCLIForeman.foreman_api_connection.authenticator.user)
-        HammerCLI::EX_OK
+        command_name('oauth')
+        desc('supports for both with/without 2fa')
+
+        option ["-u", "--username"], "USERNAME", _("Username to access the remote system")
+        option ["-p", "--password"], "PASSWORD", _("Password to access the remote system")
+        option ["-t", "--oidc-token-endpoint"], "OPENIDC-TOKEN-ENDPOINT", _("Openidc provider URL which issues access token")
+        option ["-a", "--oidc-authorization-endpoint"], "OPENIDC-AUTHORIZATION-ENDPOINT", _("Openidc provider URL which issues authentication code")
+        option ["-c", "--oidc-client-id"], "OPENIDC-CLIENT-ID", _("Client id used in the Openidc provider")
+        option ["-f", "--two-factor"], :flag, _("Authenticate with two factor")
+        option ["-r", "--oidc-redirect-uri"], "OPENIDC-REDIRECT-URI", _("Redirect URI for the authencation code grant flow")
+
+        def execute
+          if option_two_factor?
+            Oauth.execute_with_params(
+              AUTH_TYPES[:oauth_authentication_code_grant],
+              option_oidc_token_endpoint,
+              option_oidc_authorization_endpoint,
+              option_oidc_client_id,
+              option_oidc_redirect_uri
+            )
+          else
+            Oauth.execute_with_params(
+              AUTH_TYPES[:oauth_password_grant],
+              option_oidc_token_endpoint,
+              option_oidc_client_id,
+              option_username || HammerCLI::Settings.get('_params', 'username'),
+              option_password || HammerCLI::Settings.get('_params', 'password')
+            )
+          end
+        end
+        autoload_subcommands
       end
+
+      autoload_subcommands
     end
 
     class LogoutCommand < HammerCLI::AbstractCommand
@@ -52,5 +87,4 @@ module HammerCLIForeman
 
     autoload_subcommands
   end
-
 end

data/lib/hammer_cli_foreman/authenticate/login.rb

@@ -0,0 +1,21 @@
+module HammerCLIForeman
+  module Authenticate
+    module Login
+      def execute_with_params(auth_type, *args)
+        connection = HammerCLIForeman.foreman_api_reconnect(auth_type)
+        if !(connection.authenticator.is_a?(HammerCLIForeman::Api::SessionAuthenticatorWrapper))
+          HammerCLI.interactive_output.say(_("Can't perform login. Make sure sessions are enabled in hammer"\
+            " configuration file."))
+          return HammerCLI::EX_USAGE
+        end
+        connection.authenticator.set_auth_params(*args)
+        connection.authenticator.force_user_change
+        connection.login
+
+        HammerCLI.interactive_output.say(_("Successfully logged in as '%s'.") %
+          connection.authenticator.user)
+        HammerCLI::EX_OK
+      end
+    end
+  end
+end

data/lib/hammer_cli_foreman/command_extensions/hosts/help/compute_resources.rb

@@ -44,6 +44,7 @@ module HammerCLIForeman
               @help_block ||= Proc.new do
                 help do |h|
                   h.section(_('Provider specific options')) do |h|
+                    h.note(_('Bold attributes are required.'), richtext: true)
                     HammerCLIForeman.compute_resources.each do |name, provider|
                       next unless current_providers.include?(name)
 

data/lib/hammer_cli_foreman/command_extensions/ping.rb

@@ -0,0 +1,14 @@
+module HammerCLIForeman
+  module CommandExtensions
+    class Ping < HammerCLI::CommandExtensions
+      before_print do |data|
+        unless data['results']['foreman'].nil?
+          status = data['results']['foreman']['database']['active']
+          data['results']['foreman']['database']['active'] = status ? 'ok' : 'FAIL'
+          duration = data['results']['foreman']['database']['duration_ms']
+          data['results']['foreman']['database']['duration_ms'] = _("Duration: %sms") % duration
+        end
+      end
+    end
+  end
+end

data/lib/hammer_cli_foreman/command_extensions/status.rb

@@ -0,0 +1,44 @@
+module HammerCLIForeman
+  module CommandExtensions
+    class Status < HammerCLI::CommandExtensions
+      before_print do |data|
+        return if data['results'].nil?
+
+        normalize_plugins(data['results']['foreman']['plugins'])
+        data['results']['foreman']['smart_proxies'].each do |proxy|
+          proxy['features'] = normalize_features(proxy['features'])
+          proxy['failed_features'] = normalize_failed_features(proxy['failed_features'])
+        end
+      end
+
+      def self.normalize_plugins(plugins)
+        plugins.map! do |plugin|
+          name, version = plugin.split(': ', 2)[1].split(', ', 3)[0..1]
+          { name: name, version: version }
+        end
+      end
+
+      def self.normalize_features(features)
+        active_features = []
+        features.each_pair do |name, version|
+          active_features << {
+            name: name,
+            version: version
+          }
+        end
+        active_features
+      end
+
+      def self.normalize_failed_features(features)
+        failed_features = []
+        features.each_pair do |name, error|
+          failed_features << {
+            name: name,
+            error: error
+          }
+        end
+        failed_features
+      end
+    end
+  end
+end

data/lib/hammer_cli_foreman/command_extensions.rb

@@ -3,3 +3,5 @@ require 'hammer_cli_foreman/command_extensions/puppet_environment'
 require 'hammer_cli_foreman/command_extensions/puppet_environments'
 require 'hammer_cli_foreman/command_extensions/option_sources'
 require 'hammer_cli_foreman/command_extensions/hosts'
+require 'hammer_cli_foreman/command_extensions/ping'
+require 'hammer_cli_foreman/command_extensions/status'

data/lib/hammer_cli_foreman/commands.rb

@@ -279,7 +279,7 @@ module HammerCLIForeman
       meta = resource.action(action).apidoc[:metadata]
       if meta && meta[:search] && respond_to?(:option_search)
         self.class.extend_help do |h|
-          h.section(_('Search fields'), id: :search_fields_section) do |h|
+          h.section(_('Search / Order fields'), id: :search_fields_section) do |h|
             h.list(search_fields_help(meta[:search]))
           end
         end

data/lib/hammer_cli_foreman/compute_attribute.rb

@@ -21,6 +21,22 @@ module HammerCLIForeman
       attribute_list.size.times.map { |idx| idx.to_s }.zip(attribute_list).to_h
     end
 
+    def self.alter_interface(interface)
+      # move each attribute starting with "compute_" to compute_attributes
+        interface.keys.each do |key|
+          if key.start_with? 'compute_'
+            interface[key.gsub('compute_', '')] = interface.delete(key)
+          end
+        end
+        interface
+    end
+
+    def self.alter_interfaces_list(interfaces_list)
+      interfaces_list.collect do |nic|
+        alter_interface(nic)
+      end
+    end
+
     class Create < HammerCLIForeman::CreateCommand
       desc _('Create compute profile set of values')
 
@@ -41,7 +57,8 @@ module HammerCLIForeman
         compute_resource_name = HammerCLIForeman::ComputeResources.resource_provider(options['option_compute_resource_id'])
         interfaces_attr_name = ::HammerCLIForeman.compute_resources[compute_resource_name].interfaces_attrs_name
         params['compute_attribute']['vm_attrs'] = option_compute_attributes || {}
-        params['compute_attribute']['vm_attrs'][interfaces_attr_name]= HammerCLIForeman::ComputeAttribute.attribute_hash(option_interface_list) unless option_interface_list.empty?
+        interfaces_list = HammerCLIForeman::ComputeAttribute.alter_interfaces_list(options['option_interface_list']) unless options['option_interface_list'].empty?
+        params['compute_attribute']['vm_attrs'][interfaces_attr_name]= HammerCLIForeman::ComputeAttribute.attribute_hash(interfaces_list) if interfaces_list
         params['compute_attribute']['vm_attrs']['volumes_attributes'] = HammerCLIForeman::ComputeAttribute.attribute_hash(option_volume_list) unless option_volume_list.empty?
         params
       end
@@ -69,7 +86,6 @@ module HammerCLIForeman
       end
 
       def request_params
-
         params = HammerCLIForeman::ComputeAttribute.get_params(options)
         compute_resource_name = HammerCLIForeman::ComputeResources.resource_provider(options['option_compute_resource_id'])
         interfaces_attr_name = ::HammerCLIForeman.compute_resources[compute_resource_name].interfaces_attrs_name
@@ -85,7 +101,8 @@ module HammerCLIForeman
           vm_attrs['volumes_attributes'] ||= original_volumes
           vm_attrs[interfaces_attr_name] ||= original_interfaces
         end
-        vm_attrs[interfaces_attr_name] = HammerCLIForeman::ComputeAttribute.attribute_hash(options['option_interface_list']) unless options['option_interface_list'].empty?
+        interfaces_list = HammerCLIForeman::ComputeAttribute.alter_interfaces_list(options['option_interface_list']) unless options['option_interface_list'].empty?
+        vm_attrs[interfaces_attr_name] = HammerCLIForeman::ComputeAttribute.attribute_hash(interfaces_list) if interfaces_list
         vm_attrs['volumes_attributes'] = HammerCLIForeman::ComputeAttribute.attribute_hash(options['option_volume_list']) unless options['option_volume_list'].empty?
         params['compute_attribute']['vm_attrs'] = vm_attrs
         params
@@ -124,7 +141,7 @@ module HammerCLIForeman
         params['id'] = params['compute_attribute']['id']
 
         params['compute_attribute']['vm_attrs'][interfaces_attr_name] ||= {}
-        params['compute_attribute']['vm_attrs'][interfaces_attr_name][new_interface_id] = option_interface
+        params['compute_attribute']['vm_attrs'][interfaces_attr_name][new_interface_id] = HammerCLIForeman::ComputeAttribute.alter_interface(option_interface)
         params
       end
 
@@ -161,7 +178,7 @@ module HammerCLIForeman
         interfaces_attr_name = ::HammerCLIForeman.compute_resources[compute_resource_name].interfaces_attrs_name
         params['id'] = params['compute_attribute']['id']
         params['compute_attribute']['vm_attrs'][interfaces_attr_name] ||= {}
-        params['compute_attribute']['vm_attrs'][interfaces_attr_name][option_interface_id] = option_interface
+        params['compute_attribute']['vm_attrs'][interfaces_attr_name][option_interface_id] = HammerCLIForeman::ComputeAttribute.alter_interface(option_interface)
         params
       end
       success_message _('Interface was updated.')

data/lib/hammer_cli_foreman/compute_resource/base.rb

@@ -3,7 +3,7 @@ module HammerCLIForeman
     class Base
       def name; ''; end
       def compute_attributes; []; end
-      def interface_attributes; []; end
+      def interface_attributes; []; end # all attributes must start with compute_
       def volume_attributes; []; end
       def interfaces_attrs_name; 'interfaces_attributes'; end
       def host_attributes; []; end

data/lib/hammer_cli_foreman/compute_resource/ec2.rb

@@ -6,7 +6,7 @@ module HammerCLIForeman
       end
 
       def compute_attributes
-        %w[flavor_id image_id availability_zone security_group_ids managed_ip]
+        %w[availability_zone flavor_id groups security_group_ids managed_ip]
       end
 
       def provider_specific_fields

data/lib/hammer_cli_foreman/compute_resource/gce.rb

@@ -6,7 +6,7 @@ module HammerCLIForeman
       end
 
       def compute_attributes
-        %w[machine_type image_id network associate_external_ip]
+        %w[machine_type network associate_external_ip]
       end
 
       def interfaces_attrs_name

data/lib/hammer_cli_foreman/compute_resource/libvirt.rb

@@ -7,29 +7,32 @@ module HammerCLIForeman
 
       def compute_attributes
         [
-          ['cpus',   _('Number of CPUs')],
-          ['memory', _('String, amount of memory, value in bytes')]
+          ['cpus',   _('Number of CPUs'), { bold: true }],
+          ['memory', _('String, amount of memory, value in bytes'), { bold: true }],
+          ['cpu_mode', _('Possible values: %{modes}') % { modes: 'default, host-model, host-passthrough' }]
         ]
       end
 
       def host_attributes
         [
-          ['start',  _('Boolean (expressed as 0 or 1), whether to start the machine or not')]
+          ['start', _('Boolean (expressed as 0 or 1), whether to start the machine or not')]
         ]
       end
 
       def interface_attributes
         [
-          ['type',   _('Possible values: %s') % 'bridge, network'],
-          ['bridge', _('Name of interface according to type')],
-          ['model',  _('Possible values: %s') % 'virtio, rtl8139, ne2k_pci, pcnet, e1000']
+          ['compute_type',   _('Possible values: %s') % 'bridge, network'],
+          ['compute_bridge', _('Name of interface according to type')],
+          ['compute_model',  _('Possible values: %s') % 'virtio, rtl8139, ne2k_pci, pcnet, e1000'],
+          ['compute_network'], _('Libvirt instance network, e.g. default')
         ]
       end
 
       def volume_attributes
         [
-          ['pool_name',   _('One of available storage pools')],
-          ['capacity',    _('String value, eg. 10G')],
+          ['pool_name',   _('One of available storage pools'), { bold: true }],
+          ['capacity',    _('String value, e.g. 10G'), { bold: true }],
+          ['allocation'], _('Initial allocation, e.g. 0G'),
           ['format_type', _('Possible values: %s') % 'raw, qcow2']
         ]
       end

data/lib/hammer_cli_foreman/compute_resource/openstack.rb

@@ -6,7 +6,7 @@ module HammerCLIForeman
       end
 
       def compute_attributes
-        %w[flavor_ref image_ref tenant_id security_groups network]
+        %w[availability_zone boot_from_volume flavor_ref image_ref tenant_id security_groups network]
       end
 
       def provider_specific_fields

data/lib/hammer_cli_foreman/compute_resource/ovirt.rb

@@ -10,6 +10,7 @@ module HammerCLIForeman
           ['cluster',  _('ID of cluster to use')],
           ['template', _('Hardware profile to use')],
           ['cores',    _('Integer value, number of cores')],
+          ['sockets',  _('Integer value, number of sockets')],
           ['memory',   _('Amount of memory, integer value in bytes')]
         ]
       end
@@ -22,9 +23,9 @@ module HammerCLIForeman
 
       def interface_attributes
         [
-          ['name',      _('Compute name, e.g. eth0')],
-          ['network',   _('Select one of available networks for a cluster, must be an ID')],
-          ['interface', _('Interface type')]
+          ['compute_name',      _('Compute name, e.g. eth0')],
+          ['compute_network',   _('Select one of available networks for a cluster, must be an ID')],
+          ['compute_interface', _('Interface type')]
         ]
       end
 
@@ -32,7 +33,9 @@ module HammerCLIForeman
         [
           ['size_gb',        _('Volume size in GB, integer value')],
           ['storage_domain', _('ID of storage domain')],
-          ['bootable',       _('Boolean, only one volume can be bootable')]
+          ['bootable',       _('Boolean, only one volume can be bootable')],
+          ['preallocate',    _('Boolean, set %s to preallocate') % 'true'],
+          ['wipe_after_delete', _('Boolean, set %s to wipe disk after delete') % 'true']
         ]
       end
 

data/lib/hammer_cli_foreman/compute_resource/rackspace.rb

@@ -6,7 +6,7 @@ module HammerCLIForeman
       end
 
       def compute_attributes
-        %w[flavor_id image_id]
+        %w[flavor_id]
       end
 
       def provider_specific_fields

data/lib/hammer_cli_foreman/compute_resource/vmware.rb

@@ -9,13 +9,13 @@ module HammerCLIForeman
 
       def compute_attributes
         [
-          ['cpus',                 _('CPU count')],
-          ['corespersocket',       _('Number of cores per socket (applicable to hardware versions < 10 only)')],
-          ['memory_mb',            _('Integer number, amount of memory in MB')],
+          ['cluster',              _('Cluster ID from VMware'), { bold: true }],
+          ['corespersocket',       _('Number of cores per socket (applicable to hardware versions < 10 only)'), { bold: true }],
+          ['cpus',                 _('CPU count'), { bold: true }],
+          ['memory_mb',            _('Integer number, amount of memory in MB'), { bold: true }],
+          ['path',                 _('Path to folder'), { bold: true }],
+          ['resource_pool',        _('Resource Pool ID from VMware'), { bold: true }],
           ['firmware',             'automatic/bios/efi'],
-          ['cluster',              _('Cluster ID from VMware')],
-          ['resource_pool',        _('Resource Pool ID from VMware')],
-          ['path',                 _('Path to folder')],
           ['guest_id',             _('Guest OS ID form VMware')],
           ['hardware_version',     _('Hardware version ID from VMware')],
           ['memoryHotAddEnabled',  _('Must be a 1 or 0, lets you add CPU resources while the machine is on')],

data/lib/hammer_cli_foreman/hosts/common_update_options.rb

@@ -78,7 +78,7 @@ module HammerCLIForeman
         params['host']['overwrite'] = option_overwrite unless option_overwrite.nil?
 
         params['host']['host_parameters_attributes'] = parameter_attributes unless option_parameters.nil?
-        params['host']['host_parameters_attributes'] ||= option_typed_parameters unless option_typed_parameters.nil?
+        params['host']['host_parameters_attributes'] ||= typed_parameter_attributes unless option_typed_parameters.nil?
         params['host']['compute_attributes'] = option_compute_attributes || {}
 
         if action == :update

data/lib/hammer_cli_foreman/location.rb

@@ -13,7 +13,7 @@ module HammerCLIForeman
         field :id, _("Id")
         field :title, _("Title")
         field :name, _("Name")
-        field :description, _("Description")
+        field :description, _("Description"), Fields::Text
       end
 
       build_options