hammer_cli_foreman 0.0.18 → 0.1.0

data/lib/hammer_cli_foreman/environment.rb

@@ -1,18 +1,18 @@
 require 'hammer_cli'
-require 'foreman_api'
 require 'hammer_cli_foreman/smart_class_parameter'
 
 
 module HammerCLIForeman
 
-  class Environment < HammerCLI::AbstractCommand
+  class Environment < HammerCLIForeman::Command
+
+    resource :environments
 
     class ListCommand < HammerCLIForeman::ListCommand
-      resource ForemanApi::Resources::Environment, "index"
 
       output do
-        field :id, "Id"
-        field :name, "Name"
+        field :id, _("Id")
+        field :name, _("Name")
       end
 
       apipie_options
@@ -20,11 +20,10 @@ module HammerCLIForeman
 
 
     class InfoCommand < HammerCLIForeman::InfoCommand
-      resource ForemanApi::Resources::Environment, "show"
 
       output ListCommand.output_definition do
-        field :created_at, "Created at", Fields::Date
-        field :updated_at, "Updated at", Fields::Date
+        field :created_at, _("Created at"), Fields::Date
+        field :updated_at, _("Updated at"), Fields::Date
       end
 
       apipie_options
@@ -33,9 +32,8 @@ module HammerCLIForeman
 
     class CreateCommand < HammerCLIForeman::CreateCommand
 
-      success_message "Environment created"
-      failure_message "Could not create the environment"
-      resource ForemanApi::Resources::Environment, "create"
+      success_message _("Environment created")
+      failure_message _("Could not create the environment")
 
       apipie_options
     end
@@ -43,9 +41,8 @@ module HammerCLIForeman
 
     class UpdateCommand < HammerCLIForeman::UpdateCommand
 
-      success_message "Environment updated"
-      failure_message "Could not update the environment"
-      resource ForemanApi::Resources::Environment, "update"
+      success_message _("Environment updated")
+      failure_message _("Could not update the environment")
 
       apipie_options
     end
@@ -53,9 +50,8 @@ module HammerCLIForeman
 
     class DeleteCommand < HammerCLIForeman::DeleteCommand
 
-      success_message "Environment deleted"
-      failure_message "Could not delete the environment"
-      resource ForemanApi::Resources::Environment, "destroy"
+      success_message _("Environment deleted")
+      failure_message _("Could not delete the environment")
 
       apipie_options
     end
@@ -63,7 +59,7 @@ module HammerCLIForeman
     class SCParamsCommand < HammerCLIForeman::SmartClassParametersList
 
       apipie_options :without => [:host_id, :hostgroup_id, :puppetclass_id, :environment_id]
-      option ['--id', '--name'], 'ENVIRONMENT_ID', 'environment id/name',
+      option ['--id', '--name'], 'ENVIRONMENT_ID', _('environment id/name'),
             :required => true, :attribute_name => :environment_id
     end
 
@@ -74,4 +70,3 @@ module HammerCLIForeman
 end
 
 HammerCLI::MainCommand.subcommand 'environment', "Manipulate environments.", HammerCLIForeman::Environment
-

data/lib/hammer_cli_foreman/exception_handler.rb

@@ -7,6 +7,7 @@ module HammerCLIForeman
     def mappings
       super + [
         [HammerCLIForeman::OperationNotSupportedError, :handle_unsupported_operation],
+        [RestClient::InternalServerError, :handle_internal_error],
         [RestClient::Forbidden, :handle_forbidden],
         [RestClient::UnprocessableEntity, :handle_unprocessable_entity],
         [ArgumentError, :handle_argument_error],
@@ -23,6 +24,12 @@ module HammerCLIForeman
     end
 
 
+    def handle_internal_error(e)
+      handle_foreman_error(e)
+      HammerCLI::EX_SOFTWARE
+    end
+
+
     def handle_argument_error(e)
       print_error e.message
       log_full_error e
@@ -31,7 +38,7 @@ module HammerCLIForeman
 
 
     def handle_forbidden(e)
-      print_error "Forbidden - server refused to process the request"
+      print_error _("Forbidden - server refused to process the request")
       log_full_error e
       HammerCLI::EX_NOPERM
     end
@@ -45,17 +52,32 @@ module HammerCLIForeman
 
 
     def handle_not_found(e)
-      response = JSON.parse(e.response)
-      response = HammerCLIForeman.record_to_common_format(response)
-      message = response['message'] || e.message
+      handle_foreman_error(e)
+      HammerCLI::EX_NOT_FOUND
+    end
+
+
+    def handle_foreman_error(e)
+      begin
+        response = JSON.parse(e.response)
+        response = HammerCLIForeman.record_to_common_format(response)
+        message = response['message'] || e.message
+      rescue JSON::ParserError => parse_e
+        message = e.message
+      end
 
       print_error message
       log_full_error e
-      HammerCLI::EX_NOT_FOUND
+    end
+
+    def handle_apipie_docloading_error(e)
+      rake_command = 'foreman-rake apipie:cache'
+      print_error _("Could not load API description from the server\n"\
+          "  - is your server down?\n"\
+          "  - was \"#{rake_command}\" run on the server when using apipie cache? (typical production settings))\n")
+      log_full_error e
+      HammerCLI::EX_CONFIG
     end
 
   end
 end
-
-
-

data/lib/hammer_cli_foreman/fact.rb

@@ -4,14 +4,14 @@ module HammerCLIForeman
 
     class ListCommand < HammerCLIForeman::ListCommand
 
-      resource ForemanApi::Resources::FactValue, "index"
+      resource :fact_values, :index
 
       apipie_options
 
       output do
-        field :host, "Host"
-        field :fact, "Fact"
-        field :value, "Value"
+        field :host, _("Host")
+        field :fact, _("Fact")
+        field :value, _("Value")
       end
 
       def retrieve_data
@@ -33,4 +33,4 @@ module HammerCLIForeman
 
 end
 
-HammerCLI::MainCommand.subcommand 'fact', "Search facts.", HammerCLIForeman::Fact
+HammerCLI::MainCommand.subcommand 'fact', _("Search facts."), HammerCLIForeman::Fact

data/lib/hammer_cli_foreman/host.rb

@@ -2,6 +2,8 @@ require 'hammer_cli_foreman/report'
 require 'hammer_cli_foreman/puppet_class'
 require 'hammer_cli_foreman/smart_class_parameter'
 
+require 'highline/import'
+
 module HammerCLIForeman
 
   module CommonHostUpdateOptions
@@ -16,6 +18,8 @@ module HammerCLIForeman
             # - temporarily disabled params that will be removed from the api ------------------
             :provision_method, :capabilities, :flavour_ref, :image_ref, :start,
             :network, :cpus, :memory, :provider, :type, :tenant_id, :image_id,
+            # - avoids future conflicts as :root_pass is currently missing in the api docs
+            :root_pass,
             # ----------------------------------------------------------------------------------
             :compute_resource_id, :ptable_id] + base.declared_identifiers.keys
 
@@ -29,6 +33,10 @@ module HammerCLIForeman
       base.option "--partition-table-id", "PARTITION_TABLE", " "
       base.option "--puppetclass-ids", "PUPPETCLASS_IDS", " ",
         :format => HammerCLI::Options::Normalizers::List.new
+      base.option "--root-password", "ROOT_PW", " "
+      base.option "--ask-root-password", "ASK_ROOT_PW", " ",
+        :format => HammerCLI::Options::Normalizers::Bool.new
+
 
       bme_options = {}
       bme_options[:default] = 'true' if base.action.to_sym == :create
@@ -40,14 +48,21 @@ module HammerCLIForeman
       bme_options[:format] = HammerCLI::Options::Normalizers::Bool.new
       base.option "--enabled", "ENABLED", " ",  bme_options
 
-      base.option "--parameters", "PARAMS", "Host parameters.",
+      base.option "--parameters", "PARAMS", _("Host parameters."),
         :format => HammerCLI::Options::Normalizers::KeyValueList.new
-      base.option "--compute-attributes", "COMPUTE_ATTRS", "Compute resource attributes.",
+      base.option "--compute-attributes", "COMPUTE_ATTRS", _("Compute resource attributes."),
         :format => HammerCLI::Options::Normalizers::KeyValueList.new
-      base.option "--volume", "VOLUME", "Volume parameters", :multivalued => true,
+      base.option "--volume", "VOLUME", _("Volume parameters"), :multivalued => true,
         :format => HammerCLI::Options::Normalizers::KeyValueList.new
-      base.option "--interface", "INTERFACE", "Interface parameters.", :multivalued => true,
+      base.option "--interface", "INTERFACE", _("Interface parameters."), :multivalued => true,
         :format => HammerCLI::Options::Normalizers::KeyValueList.new
+      base.option "--provision-method", "METHOD", " ",
+        :format => HammerCLI::Options::Normalizers::Enum.new(['build', 'image'])
+    end
+
+    def self.ask_password
+      prompt = "Enter the root password for the host: "
+      ask(prompt) {|q| q.echo = false}
     end
 
     def request_params
@@ -71,6 +86,12 @@ module HammerCLIForeman
         params['host']['interfaces_attributes'] = nested_attributes(option_interface_list)
       end
 
+      params['host']['root_pass'] = option_root_password unless option_root_password.nil?
+
+      if option_ask_root_password
+        params['host']['root_pass'] = HammerCLIForeman::CommonHostUpdateOptions::ask_password
+      end
+
       params
     end
 
@@ -96,19 +117,19 @@ module HammerCLIForeman
   end
 
 
-  class Host < HammerCLI::Apipie::Command
+  class Host < HammerCLIForeman::Command
 
-    resource ForemanApi::Resources::Host
+    resource :hosts
 
     class ListCommand < HammerCLIForeman::ListCommand
       # FIXME: list compute resource (model)
       output do
-        field :id, "Id"
-        field :name, "Name"
-        field :operatingsystem_id, "Operating System Id"
-        field :hostgroup_id, "Host Group Id"
-        field :ip, "IP"
-        field :mac, "MAC"
+        field :id, _("Id")
+        field :name, _("Name")
+        field :operatingsystem_id, _("Operating System Id")
+        field :hostgroup_id, _("Host Group Id")
+        field :ip, _("IP")
+        field :mac, _("MAC")
       end
 
       apipie_options
@@ -120,53 +141,82 @@ module HammerCLIForeman
       def extend_data(host)
         host["environment_name"] = host["environment"]["environment"]["name"] rescue nil
         host["parameters"] = HammerCLIForeman::Parameter.get_parameters(resource_config, :host, host)
+        host["_bmc_interfaces"] =
+          host["interfaces"].select{|intfs| intfs["type"] == "Nic::BMC" } rescue []
+        host["_managed_interfaces"] =
+          host["interfaces"].select{|intfs| intfs["type"] == "Nic::Managed" } rescue []
         host
       end
 
       output ListCommand.output_definition do
-        field :uuid, "UUID"
-        field :certname, "Cert name"
-
-        field :environment_name, "Environment"
-        field :environment_id, "Environment Id"
-
-        field :managed, "Managed"
-        field :enabled, "Enabled"
-        field :build, "Build"
-
-        field :use_image, "Use image"
-        field :disk, "Disk"
-        field :image_file, "Image file"
-
-        field :sp_name, "SP Name"
-        field :sp_ip, "SP IP"
-        field :sp_mac, "SP MAC"
-        field :sp_subnet, "SP Subnet"
-        field :sp_subnet_id, "SP Subnet Id"
-
-        field :created_at, "Created at", Fields::Date
-        field :updated_at, "Updated at", Fields::Date
-        field :installed_at, "Installed at", Fields::Date
-        field :last_report, "Last report", Fields::Date
-
-        field :puppet_ca_proxy_id, "Puppet CA Proxy Id"
-        field :medium_id, "Medium Id"
-        field :model_id, "Model Id"
-        field :owner_id, "Owner Id"
-        field :subnet_id, "Subnet Id"
-        field :domain_id, "Domain Id"
-        field :puppet_proxy_id, "Puppet Proxy Id"
-        field :owner_type, "Owner Type"
-        field :ptable_id, "Partition Table Id"
-        field :architecture_id, "Architecture Id"
-        field :image_id, "Image Id"
-        field :compute_resource_id, "Compute Resource Id"
-
-        field :comment, "Comment"
-
-        collection :parameters, "Parameters" do
+        field :uuid, _("UUID")
+        field :certname, _("Cert name")
+
+        field :environment_name, _("Environment")
+        field :environment_id, _("Environment Id")
+
+        field :managed, _("Managed")
+        field :enabled, _("Enabled")
+        field :build, _("Build")
+
+        field :use_image, _("Use image")
+        field :disk, _("Disk")
+        field :image_file, _("Image file")
+
+        field :sp_name, _("SP Name")
+        field :sp_ip, _("SP IP")
+        field :sp_mac, _("SP MAC")
+        field :sp_subnet, _("SP Subnet")
+        field :sp_subnet_id, _("SP Subnet Id")
+
+        field :created_at, _("Created at"), Fields::Date
+        field :updated_at, _("Updated at"), Fields::Date
+        field :installed_at, _("Installed at"), Fields::Date
+        field :last_report, _("Last report"), Fields::Date
+
+        field :puppet_ca_proxy_id, _("Puppet CA Proxy Id")
+        field :medium_id, _("Medium Id")
+        field :model_id, _("Model Id")
+        field :owner_id, _("Owner Id")
+        field :subnet_id, _("Subnet Id")
+        field :domain_id, _("Domain Id")
+        field :puppet_proxy_id, _("Puppet Proxy Id")
+        field :owner_type, _("Owner Type")
+        field :ptable_id, _("Partition Table Id")
+        field :architecture_id, _("Architecture Id")
+        field :image_id, _("Image Id")
+        field :compute_resource_id, _("Compute Resource Id")
+
+        field :comment, _("Comment")
+
+        collection :parameters, _("Parameters") do
           field nil, nil, Fields::KeyValue
         end
+
+        collection :_bmc_interfaces, _("BMC Network Interfaces"), :hide_blank => true do
+          field :id, _("Id")
+          field :name, _("Name")
+          field :ip, _("IP")
+          field :mac, _("MAC")
+          field :domain_id, _("Domain Id")
+          field :domain_name, _("Domain Name")
+          field :subnet_id, _("Subnet Id")
+          field :subnet_name, _("Subnet Name")
+          field :username, _("BMC Username")
+          field :password, _("BMC Password")
+        end
+
+        collection :_managed_interfaces, _("Managed Network Interfaces"), :hide_blank => true do
+          field :id, _("Id")
+          field :name, _("Name")
+          field :ip, _("IP")
+          field :mac, _("MAC")
+          field :domain_id, _("Domain Id")
+          field :domain_name, _("Domain Name")
+          field :subnet_id, _("Subnet Id")
+          field :subnet_name, _("Subnet Name")
+        end
+
       end
 
       apipie_options
@@ -180,8 +230,8 @@ module HammerCLIForeman
       command_name "status"
 
       output do
-        field :status, "Status"
-        field :power, "Power"
+        field :status, _("Status")
+        field :power, _("Power")
       end
 
       def retrieve_data
@@ -195,7 +245,7 @@ module HammerCLIForeman
         params = {
           'id' => get_identifier[0],
         }
-        status = resource.call(:status, params)[0]
+        status = resource.call(:status, params)
         status["status"]
       end
 
@@ -204,7 +254,7 @@ module HammerCLIForeman
           'id' => get_identifier[0],
           'power_action' => :state
         }
-        status = resource.call(:power, params)[0]
+        status = resource.call(:power, params)
         status["power"]
       end
 
@@ -215,10 +265,10 @@ module HammerCLIForeman
     class PuppetRunCommand < HammerCLIForeman::InfoCommand
 
       command_name "puppetrun"
-      action "puppetrun"
+      action :puppetrun
 
       def print_data(records)
-        print_message 'Puppet run triggered'
+        print_message _('Puppet run triggered')
       end
 
       apipie_options
@@ -228,14 +278,14 @@ module HammerCLIForeman
     class FactsCommand < HammerCLIForeman::ListCommand
 
       command_name "facts"
-      resource ForemanApi::Resources::FactValue, "index"
+      resource :fact_values, :index
       identifiers :id, :name
 
       apipie_options :without => declared_identifiers.keys
 
       output do
-        field :fact, "Fact"
-        field :value, "Value"
+        field :fact, _("Fact")
+        field :value, _("Value")
       end
 
       def request_params
@@ -254,8 +304,8 @@ module HammerCLIForeman
 
     class PuppetClassesCommand < HammerCLIForeman::ListCommand
 
-      command_name "puppet_classes"
-      resource ForemanApi::Resources::Puppetclass
+      command_name "puppet-classes"
+      resource :puppetclasses
 
       identifiers :id, :name
 
@@ -280,7 +330,7 @@ module HammerCLIForeman
       identifiers :id, :name
 
       command_name "reports"
-      resource ForemanApi::Resources::Report
+      resource :reports
       output HammerCLIForeman::Report::ListCommand.output_definition
 
       apipie_options :without => :search
@@ -292,17 +342,22 @@ module HammerCLIForeman
 
     class CreateCommand < HammerCLIForeman::CreateCommand
 
-      success_message "Host created"
-      failure_message "Could not create the host"
-      action "create"
+      success_message _("Host created")
+      failure_message _("Could not create the host")
 
       include HammerCLIForeman::CommonHostUpdateOptions
 
-      validate_options do
-        unless option(:option_hostgroup_id).exist?
-          all(:option_environment_id, :option_architecture_id, :option_domain_id,
-            :option_puppet_proxy_id, :option_operatingsystem_id,
-            :option_partition_table_id).required
+      def validate_options
+        super
+        unless validator.option(:option_hostgroup_id).exist?
+          if option_managed
+            validator.all(:option_environment_id, :option_architecture_id, :option_domain_id,
+                          :option_puppet_proxy_id, :option_operatingsystem_id,
+                          :option_partition_table_id).required
+          else
+            # unmanaged host only requires environment
+            validator.option(:option_environment_id).required
+          end
         end
       end
     end
@@ -310,8 +365,8 @@ module HammerCLIForeman
 
     class UpdateCommand < HammerCLIForeman::UpdateCommand
 
-      success_message "Host updated"
-      failure_message "Could not update the host"
+      success_message _("Host updated")
+      failure_message _("Could not update the host")
 
       include HammerCLIForeman::CommonHostUpdateOptions
     end
@@ -319,8 +374,8 @@ module HammerCLIForeman
 
     class DeleteCommand < HammerCLIForeman::DeleteCommand
 
-      success_message "Host deleted"
-      failure_message "Could not delete the host"
+      success_message _("Host deleted")
+      failure_message _("Could not delete the host")
 
       apipie_options
     end
@@ -328,16 +383,16 @@ module HammerCLIForeman
 
     class SetParameterCommand < HammerCLIForeman::Parameter::SetCommand
 
-      resource ForemanApi::Resources::Parameter
+      resource :parameters
 
-      desc "Create or update parameter for a host."
+      desc _("Create or update parameter for a host.")
 
-      option "--host-name", "HOST_NAME", "name of the host the parameter is being set for"
-      option "--host-id", "HOST_ID", "id of the host the parameter is being set for"
+      option "--host-name", "HOST_NAME", _("name of the host the parameter is being set for")
+      option "--host-id", "HOST_ID", _("id of the host the parameter is being set for")
 
-      success_message_for :update, "Host parameter updated"
-      success_message_for :create, "New host parameter created"
-      failure_message "Could not set host parameter"
+      success_message_for :update, _("Host parameter updated")
+      success_message_for :create, _("New host parameter created")
+      failure_message _("Could not set host parameter")
 
       def validate_options
         super
@@ -354,14 +409,14 @@ module HammerCLIForeman
 
     class DeleteParameterCommand < HammerCLIForeman::Parameter::DeleteCommand
 
-      resource ForemanApi::Resources::Parameter
+      resource :parameters
 
-      desc "Delete parameter for a host."
+      desc _("Delete parameter for a host.")
 
-      option "--host-name", "HOST_NAME", "name of the host the parameter is being deleted for"
-      option "--host-id", "HOST_ID", "id of the host the parameter is being deleted for"
+      option "--host-name", "HOST_NAME", _("name of the host the parameter is being deleted for")
+      option "--host-id", "HOST_ID", _("id of the host the parameter is being deleted for")
 
-      success_message "Host parameter deleted"
+      success_message _("Host parameter deleted")
 
       def validate_options
         super
@@ -379,31 +434,37 @@ module HammerCLIForeman
     class StartCommand < HammerCLI::Apipie::WriteCommand
 
       identifiers :id, :name
-      action "power"
+      action :power
 
       command_name "start"
-      desc "Power a host on"
-      success_message "The host is starting."
+      desc _("Power a host on")
+      success_message _("The host is starting.")
 
-      def power_action
+      def option_power_action
         :start
       end
 
+      def request_params
+        params = method_options
+        params['id'] = get_identifier[0]
+        params
+      end
+
       apipie_options :without => :power_action
     end
 
 
     class StopCommand < HammerCLI::Apipie::WriteCommand
 
-      option '--force', :flag, "Force turning off a host"
+      option '--force', :flag, _("Force turning off a host")
 
       identifiers :id, :name
-      action "power"
+      action :power
 
       command_name "stop"
-      desc "Power a host off"
+      desc _("Power a host off")
 
-      def power_action
+      def option_power_action
         if option_force?
           :cycle
         else
@@ -413,35 +474,47 @@ module HammerCLIForeman
 
       def success_message
         if option_force?
-          "Power off forced."
+          _("Power off forced.")
         else
-          "Powering the host off."
+          _("Powering the host off.")
         end
       end
 
+      def request_params
+        params = method_options
+        params['id'] = get_identifier[0]
+        params
+      end
+
       apipie_options :without => :power_action
     end
 
     class RebootCommand < HammerCLI::Apipie::WriteCommand
 
       identifiers :id, :name
-      action "power"
+      action :power
 
       command_name "reboot"
-      desc "Reboot a host"
-      success_message "Host reboot started."
+      desc _("Reboot a host")
+      success_message _("Host reboot started.")
 
-      def power_action
+      def option_power_action
         :soft
       end
 
+      def request_params
+        params = method_options
+        params['id'] = get_identifier[0]
+        params
+      end
+
       apipie_options :without => :power_action
     end
 
     class SCParamsCommand < HammerCLIForeman::SmartClassParametersList
 
       apipie_options :without => [:host_id, :hostgroup_id, :puppetclass_id, :environment_id]
-      option ['--id', '--name'], 'HOST_ID', 'host id/name',
+      option ['--id', '--name'], 'HOST_ID', _('host id/name'),
               :attribute_name => :option_host_id, :required => true
     end
 
@@ -450,4 +523,4 @@ module HammerCLIForeman
 
 end
 
-HammerCLI::MainCommand.subcommand 'host', "Manipulate hosts.", HammerCLIForeman::Host
+HammerCLI::MainCommand.subcommand 'host', ("Manipulate hosts."), HammerCLIForeman::Host