hammer_cli_foreman 0.0.4 → 0.0.5

data/lib/hammer_cli_foreman/common_parameter.rb

@@ -9,7 +9,6 @@ module HammerCLIForeman
     class ListCommand < HammerCLIForeman::ListCommand
       resource ForemanApi::Resources::CommonParameter, "index"
 
-      heading "Global parameter list"
       output do
         from "common_parameter" do
           field :name, "Name"
@@ -22,6 +21,9 @@ module HammerCLIForeman
 
     class SetCommand < HammerCLI::Apipie::WriteCommand
 
+      command_name "set"
+      desc "Set a global parameter."
+
       resource ForemanApi::Resources::CommonParameter
 
       option "--name", "NAME", "parameter name", :required => true
@@ -70,10 +72,7 @@ module HammerCLIForeman
       apipie_options :without => :id
     end
 
-    subcommand "list", "List global parameters.", HammerCLIForeman::CommonParameter::ListCommand
-    subcommand "set", "Set a global parameter.", HammerCLIForeman::CommonParameter::SetCommand
-    subcommand "delete", "Delete a global parameter.", HammerCLIForeman::CommonParameter::DeleteCommand
-
+    autoload_subcommands
   end
 end
 

data/lib/hammer_cli_foreman/compute_resource.rb

@@ -9,7 +9,6 @@ module HammerCLIForeman
     class ListCommand < HammerCLIForeman::ListCommand
       resource ForemanApi::Resources::ComputeResource, "index"
 
-      heading "Compute resource list"
       output do
         from "compute_resource" do
           field :id, "Id"
@@ -26,20 +25,20 @@ module HammerCLIForeman
 
       PROVIDER_SPECIFIC_FIELDS = {
         'ovirt' => [
-          HammerCLI::Output::DataField.new(:label => 'UUID', :path => ["compute_resource", "uuid"])
+          Fields::DataField.new(:label => 'UUID', :path => ["compute_resource", "uuid"])
         ],
         'ec2' => [
-          HammerCLI::Output::DataField.new(:label => 'Region', :path => ["compute_resource", "region"])
+          Fields::DataField.new(:label => 'Region', :path => ["compute_resource", "region"])
         ],
         'vmware' => [
-          HammerCLI::Output::DataField.new(:label => 'UUID', :path => ["compute_resource", "uuid"]),
-          HammerCLI::Output::DataField.new(:label => 'Server', :path => ["compute_resource", "server"])
+          Fields::DataField.new(:label => 'UUID', :path => ["compute_resource", "uuid"]),
+          Fields::DataField.new(:label => 'Server', :path => ["compute_resource", "server"])
         ],
         'openstack' => [
-          HammerCLI::Output::DataField.new(:label => 'Tenant', :path => ["compute_resource", "tenant"])
+          Fields::DataField.new(:label => 'Tenant', :path => ["compute_resource", "tenant"])
         ],
         'rackspace' => [
-          HammerCLI::Output::DataField.new(:label => 'Region', :path => ["compute_resource", "region"])
+          Fields::DataField.new(:label => 'Region', :path => ["compute_resource", "region"])
         ],
         'libvirt' => [
         ]
@@ -47,18 +46,17 @@ module HammerCLIForeman
 
       resource ForemanApi::Resources::ComputeResource, "show"
 
-      heading "Compute resource info"
       output ListCommand.output_definition do
         from "compute_resource" do
           field :url, "Url"
           field :description, "Description"
           field :user, "User"
-          field :created_at, "Created at", HammerCLI::Output::Fields::Date
-          field :updated_at, "Updated at", HammerCLI::Output::Fields::Date
+          field :created_at, "Created at", Fields::Date
+          field :updated_at, "Updated at", Fields::Date
         end
       end
 
-      def print_records data
+      def print_records(data)
         provider = data["compute_resource"]["provider"].downcase
         output_definition.fields.concat PROVIDER_SPECIFIC_FIELDS[provider]
         super data
@@ -100,11 +98,7 @@ module HammerCLIForeman
       apipie_options
     end
 
-    subcommand "list", "List compute resources.", HammerCLIForeman::ComputeResource::ListCommand
-    subcommand "info", "Detailed info about a compute resource.", HammerCLIForeman::ComputeResource::InfoCommand
-    subcommand "create", "Create new compute resource.", HammerCLIForeman::ComputeResource::CreateCommand
-    subcommand "update", "Update a compute resource.", HammerCLIForeman::ComputeResource::UpdateCommand
-    subcommand "delete", "Delete a compute resource.", HammerCLIForeman::ComputeResource::DeleteCommand
+    autoload_subcommands
   end
 
 end

data/lib/hammer_cli_foreman/domain.rb

@@ -9,7 +9,6 @@ module HammerCLIForeman
     class ListCommand < HammerCLIForeman::ListCommand
       resource ForemanApi::Resources::Domain, "index"
 
-      heading "Domain list"
       output do
         from "domain" do
           field :id, "Id"
@@ -31,16 +30,15 @@ module HammerCLIForeman
         domain
       end
 
-      heading "Domain info"
       output ListCommand.output_definition do
         from "domain" do
           field :fullname, "Full Name"
           field :dns_id, "DNS Id"
-          field :created_at, "Created at", HammerCLI::Output::Fields::Date
-          field :updated_at, "Updated at", HammerCLI::Output::Fields::Date
+          field :created_at, "Created at", Fields::Date
+          field :updated_at, "Updated at", Fields::Date
         end
         collection :parameters, "Parameters" do
-          field :parameter, nil, HammerCLI::Output::Fields::KeyValue
+          field :parameter, nil, Fields::KeyValue
         end
       end
 
@@ -79,6 +77,8 @@ module HammerCLIForeman
 
     class SetParameterCommand < HammerCLIForeman::Parameter::SetCommand
 
+      desc "Create or update parameter for a domain."
+
       option "--domain-name", "DOMAIN_NAME", "name of the domain the parameter is being set for"
       option "--domain-id", "DOMAIN_ID", "id of the domain the parameter is being set for"
 
@@ -101,6 +101,8 @@ module HammerCLIForeman
 
     class DeleteParameterCommand < HammerCLIForeman::Parameter::DeleteCommand
 
+      desc "Delete parameter for a domain."
+
       option "--domain-name", "DOMAIN_NAME", "name of the domain the parameter is being deleted for"
       option "--domain-id", "DOMAIN_ID", "id of the domain the parameter is being deleted for"
 
@@ -118,13 +120,7 @@ module HammerCLIForeman
       end
     end
 
-    subcommand "list", "List domains.", HammerCLIForeman::Domain::ListCommand
-    subcommand "info", "Detailed info about a domain.", HammerCLIForeman::Domain::InfoCommand
-    subcommand "create", "Create a new domain.", HammerCLIForeman::Domain::CreateCommand
-    subcommand "update", "Update a domain.", HammerCLIForeman::Domain::UpdateCommand
-    subcommand "delete", "Delete a domain.", HammerCLIForeman::Domain::DeleteCommand
-    subcommand "set_parameter", "Create or update parameter for a domain.", HammerCLIForeman::Domain::SetParameterCommand
-    subcommand "delete_parameter", "Delete parameter for a domain.", HammerCLIForeman::Domain::DeleteParameterCommand
+    autoload_subcommands
   end
 
 end

data/lib/hammer_cli_foreman/environment.rb

@@ -8,7 +8,6 @@ module HammerCLIForeman
     class ListCommand < HammerCLIForeman::ListCommand
       resource ForemanApi::Resources::Environment, "index"
 
-      heading "Environments"
       output do
         from "environment" do
           field :id, "Id"
@@ -23,11 +22,10 @@ module HammerCLIForeman
     class InfoCommand < HammerCLIForeman::InfoCommand
       resource ForemanApi::Resources::Environment, "show"
 
-      heading "Environment info"
       output ListCommand.output_definition do
         from "environment" do
-          field :created_at, "Created at", HammerCLI::Output::Fields::Date
-          field :updated_at, "Updated at", HammerCLI::Output::Fields::Date
+          field :created_at, "Created at", Fields::Date
+          field :updated_at, "Updated at", Fields::Date
         end
       end
 
@@ -63,11 +61,7 @@ module HammerCLIForeman
       apipie_options
     end
 
-    subcommand "list", "List environments.", HammerCLIForeman::Environment::ListCommand
-    subcommand "info", "Detailed info about an environment.", HammerCLIForeman::Environment::InfoCommand
-    subcommand "create", "Create new environment.", HammerCLIForeman::Environment::CreateCommand
-    subcommand "update", "Update an environment.", HammerCLIForeman::Environment::UpdateCommand
-    subcommand "delete", "Delete an environment.", HammerCLIForeman::Environment::DeleteCommand
+    autoload_subcommands
   end
 
 end

data/lib/hammer_cli_foreman/exception_handler.rb

@@ -13,7 +13,7 @@ module HammerCLIForeman
 
     protected
 
-    def handle_unprocessable_entity e
+    def handle_unprocessable_entity(e)
       response = JSON.parse(e.response)
       response = response[response.keys[0]]
 
@@ -22,13 +22,13 @@ module HammerCLIForeman
     end
 
 
-    def handle_argument_error e
+    def handle_argument_error(e)
       print_error e.message
       log_full_error e
       HammerCLI::EX_USAGE
     end
 
-    def handle_forbidden e
+    def handle_forbidden(e)
       print_error "Forbidden - server refused to process the request"
       log_full_error e
       HammerCLI::EX_NOPERM

data/lib/hammer_cli_foreman/fact.rb

@@ -0,0 +1,42 @@
+require 'hammer_cli'
+require 'foreman_api'
+require 'hammer_cli_foreman/commands'
+
+module HammerCLIForeman
+
+  class Fact < HammerCLI::AbstractCommand
+
+    class ListCommand < HammerCLIForeman::ListCommand
+
+      resource ForemanApi::Resources::FactValue, "index"
+
+      apipie_options
+
+      output do
+        from "fact" do
+          field :host, "Host"
+          field :fact, "Fact"
+          field :value, "Value"
+        end
+      end
+
+      def retrieve_data
+        self.class.unhash_facts(super)
+      end
+
+      def self.unhash_facts(facts_hash)
+        facts_hash.inject([]) do |list, (host, facts)|
+          list + facts.collect do |(fact, value)|
+            { :fact => { :host => host, :fact => fact, :value => value } }
+          end
+        end
+      end
+
+    end
+
+    autoload_subcommands
+  end
+
+end
+
+HammerCLI::MainCommand.subcommand 'fact', "Search Foreman's facts.", HammerCLIForeman::Fact

data/lib/hammer_cli_foreman/host.rb

@@ -2,6 +2,8 @@ require 'hammer_cli'
 require 'foreman_api'
 require 'hammer_cli_foreman/commands'
 require 'hammer_cli_foreman/parameter'
+require 'hammer_cli_foreman/report'
+require 'hammer_cli_foreman/puppet_class'
 
 module HammerCLIForeman
 
@@ -9,7 +11,6 @@ module HammerCLIForeman
     class ListCommand < HammerCLIForeman::ListCommand
       resource ForemanApi::Resources::Host, "index"
 
-      heading "Host list"
       output do
         from "host" do
           field :id, "Id"
@@ -32,11 +33,10 @@ module HammerCLIForeman
       def retrieve_data
         host = super
         host["host"]["environment_name"] = host["host"]["environment"]["environment"]["name"] rescue nil
-        host["parameters"] = HammerCLIForeman::Parameter.get_parameters resource_config, host
+        host["parameters"] = HammerCLIForeman::Parameter.get_parameters(resource_config, host)
         host
       end
 
-      heading "Host info"
       output ListCommand.output_definition do
         from "host" do
           field :uuid, "UUID"
@@ -59,10 +59,10 @@ module HammerCLIForeman
           field :sp_subnet, "SP Subnet"
           field :sp_subnet_id, "SP Subnet Id"
 
-          field :created_at, "Created at", HammerCLI::Output::Fields::Date
-          field :updated_at, "Updated at", HammerCLI::Output::Fields::Date
-          field :installed_at, "Installed at", HammerCLI::Output::Fields::Date
-          field :last_report, "Last report", HammerCLI::Output::Fields::Date
+          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"
@@ -80,7 +80,7 @@ module HammerCLIForeman
           field :comment, "Comment"
         end
         collection :parameters, "Parameters" do
-          field :parameter, nil, HammerCLI::Output::Fields::KeyValue
+          field :parameter, nil, Fields::KeyValue
         end
       end
     end
@@ -88,6 +88,7 @@ module HammerCLIForeman
 
     class StatusCommand < HammerCLIForeman::InfoCommand
 
+      command_name "status"
       resource ForemanApi::Resources::Host, "status"
 
       def print_data(records)
@@ -98,6 +99,7 @@ module HammerCLIForeman
 
     class PuppetRunCommand < HammerCLIForeman::InfoCommand
 
+      command_name "puppetrun"
       resource ForemanApi::Resources::Host, "puppetrun"
 
       def print_data(records)
@@ -106,6 +108,74 @@ module HammerCLIForeman
     end
 
 
+    class FactsCommand < HammerCLIForeman::ListCommand
+
+      command_name "facts"
+      resource ForemanApi::Resources::FactValue, "index"
+      identifiers :id, :name
+
+      apipie_options :without => declared_identifiers.keys
+
+      output do
+        from "fact" do
+          field :fact, "Fact"
+          field :value, "Value"
+        end
+      end
+
+      def request_params
+        params = method_options
+        params['host_id'] = get_identifier[0]
+        params
+      end
+
+      def retrieve_data
+        HammerCLIForeman::Fact::ListCommand.unhash_facts(super)
+      end
+
+    end
+
+
+    class PuppetClassesCommand < HammerCLIForeman::ListCommand
+
+      command_name "puppet_classes"
+      resource ForemanApi::Resources::Puppetclass
+
+      identifiers :id, :name
+
+      output HammerCLIForeman::PuppetClass::ListCommand.output_definition
+
+      def retrieve_data
+        HammerCLIForeman::PuppetClass::ListCommand.unhash_classes(super)
+      end
+
+      def request_params
+        params = method_options
+        params['host_id'] = get_identifier[0]
+        params
+      end
+
+      apipie_options
+    end
+
+
+    class ReportsCommand < HammerCLIForeman::ListCommand
+
+      identifiers :id, :name
+
+      command_name "reports"
+      resource ForemanApi::Resources::Report
+      output HammerCLIForeman::Report::ListCommand.output_definition
+
+      apipie_options :without => :search
+
+      def search
+        'host.id = %s' % get_identifier[0].to_s
+      end
+
+    end
+
+
     class CreateCommand < HammerCLIForeman::CreateCommand
 
       success_message "Host created"
@@ -119,7 +189,7 @@ module HammerCLIForeman
       def request_params
         params = super
         params['host']['compute_attributes']["nics_attributes"] = {
-          "new_nics"=>{"type"=>"bridge", "_delete"=>"", "bridge"=>""}, 
+          "new_nics"=>{"type"=>"bridge", "_delete"=>"", "bridge"=>""},
           "0"=>{"type"=>"network", "_delete"=>"", "network"=>"default", "bridge"=>""}
         }
         params
@@ -149,6 +219,8 @@ module HammerCLIForeman
 
     class SetParameterCommand < HammerCLIForeman::Parameter::SetCommand
 
+      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"
 
@@ -171,6 +243,8 @@ module HammerCLIForeman
 
     class DeleteParameterCommand < HammerCLIForeman::Parameter::DeleteCommand
 
+      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"
 
@@ -188,15 +262,7 @@ module HammerCLIForeman
       end
     end
 
-    subcommand "list", "List hosts.", HammerCLIForeman::Host::ListCommand
-    subcommand "info", "Detailed info about host.", HammerCLIForeman::Host::InfoCommand
-    subcommand "status", "Print host status.", HammerCLIForeman::Host::StatusCommand
-    subcommand "puppetrun", "Force puppet run on the agent.", HammerCLIForeman::Host::PuppetRunCommand
-    subcommand "create", "Create a new host.", HammerCLIForeman::Host::CreateCommand
-    subcommand "update", "Update a host.", HammerCLIForeman::Host::UpdateCommand
-    subcommand "delete", "Delete a host.", HammerCLIForeman::Host::DeleteCommand
-    subcommand "set_parameter", "Create or update parameter for a host.", HammerCLIForeman::Host::SetParameterCommand
-    subcommand "delete_parameter", "Delete parameter for a host.", HammerCLIForeman::Host::DeleteParameterCommand
+    autoload_subcommands
   end
 
 end

data/lib/hammer_cli_foreman/hostgroup.rb

@@ -9,7 +9,6 @@ module HammerCLIForeman
     class ListCommand < HammerCLIForeman::ListCommand
       resource ForemanApi::Resources::Hostgroup, "index"
 
-      heading "Hostgroup list"
       output do
         from "hostgroup" do
           field :id, "Id"
@@ -19,7 +18,7 @@ module HammerCLIForeman
           field :subnet_id, "Subnet Id"
           field :domain_id, "Domain Id"
           field :environment_id, "Environment Id"
-          field :puppetclass_ids, "Puppetclass Ids", HammerCLI::Output::Fields::List
+          field :puppetclass_ids, "Puppetclass Ids", Fields::List
           field :ancestry, "Ancestry"
         end
       end
@@ -33,10 +32,9 @@ module HammerCLIForeman
 
       identifiers :id
 
-      heading "Hostgroup info"
       output ListCommand.output_definition do
         collection :parameters, "Parameters" do
-          field :parameter, nil, HammerCLI::Output::Fields::KeyValue
+          field :parameter, nil, Fields::KeyValue
         end
       end
 
@@ -83,8 +81,33 @@ module HammerCLIForeman
     end
 
 
+    class PuppetClassesCommand < HammerCLIForeman::ListCommand
+
+      command_name "puppet_classes"
+      resource ForemanApi::Resources::Puppetclass
+
+      identifiers :id
+
+      output HammerCLIForeman::PuppetClass::ListCommand.output_definition
+
+      def retrieve_data
+        HammerCLIForeman::PuppetClass::ListCommand.unhash_classes(super)
+      end
+
+      def request_params
+        params = method_options
+        params['hostgroup_id'] = get_identifier[0]
+        params
+      end
+
+      apipie_options
+    end
+
+
     class SetParameterCommand < HammerCLIForeman::Parameter::SetCommand
 
+      desc "Create or update parameter for a hostgroup."
+
       option "--hostgroup-id", "HOSTGROUP_ID", "id of the hostgroup the parameter is being set for", :required => true
 
       success_message_for :update, "Hostgroup parameter updated"
@@ -101,6 +124,8 @@ module HammerCLIForeman
 
     class DeleteParameterCommand < HammerCLIForeman::Parameter::DeleteCommand
 
+      desc "Delete parameter for a hostgroup."
+
       option "--hostgroup-id", "HOSTGROUP_ID", "id of the hostgroup the parameter is being deleted for", :required => true
 
       success_message "Hostgroup parameter deleted"
@@ -112,15 +137,7 @@ module HammerCLIForeman
       end
     end
 
-
-    subcommand "list", "List hostgroups.", HammerCLIForeman::Hostgroup::ListCommand
-    subcommand "info", "Detailed info about hostgroup.", HammerCLIForeman::Hostgroup::InfoCommand
-    subcommand "create", "Create a new hostgroup.", HammerCLIForeman::Hostgroup::CreateCommand
-    subcommand "update", "Update a hostgroup.", HammerCLIForeman::Hostgroup::UpdateCommand
-    subcommand "delete", "Delete a hostgroup.", HammerCLIForeman::Hostgroup::DeleteCommand
-    subcommand "set_parameter", "Create or update parameter for a hostgroup.", HammerCLIForeman::Hostgroup::SetParameterCommand
-    subcommand "delete_parameter", "Delete parameter for a hostgroup.", HammerCLIForeman::Hostgroup::DeleteParameterCommand
-
+    autoload_subcommands
   end
 
 end