hammer_cli_foreman 0.0.4 → 0.0.5

data/lib/hammer_cli_foreman/parameter.rb

@@ -19,20 +19,27 @@ module HammerCLIForeman
     class SetCommand < HammerCLI::Apipie::Command
 
       include HammerCLI::Messages
-      resource ForemanApi::Resources::Parameter
 
       option "--name", "NAME", "parameter name", :required => true
       option "--value", "VALUE", "parameter value", :required => true
 
+      def self.command_name(name=nil)
+        super(name) || "set_parameter"
+      end
+
+      def self.resource(resource=nil)
+        super(resource) || HammerCLI::Apipie::ResourceDefinition.new(ForemanApi::Resources::Parameter)
+      end
+
       def execute
         if parameter_exist?
           update_parameter
-          output.print_message success_message_for :update if  success_message_for :update
+          output.print_message success_message_for :update if success_message_for :update
         else
           create_parameter
-          output.print_message success_message_for :create if  success_message_for :create
+          output.print_message success_message_for :create if success_message_for :create
         end
-        0
+        HammerCLI::EX_OK
       end
 
       def base_action_params
@@ -40,7 +47,7 @@ module HammerCLIForeman
       end
 
       def parameter_exist?
-        params = resource.index(base_action_params)[0]
+        params = resource.call(:index, base_action_params)[0]
         params.find { |p| p["parameter"]["name"] == name }
       end
 
@@ -52,7 +59,7 @@ module HammerCLIForeman
           }
         }.merge base_action_params
 
-        resource.update(params)
+        resource.call(:update, params)
       end
 
       def create_parameter
@@ -63,7 +70,7 @@ module HammerCLIForeman
           }
         }.merge base_action_params
 
-        resource.create(params)
+        resource.call(:create, params)
       end
 
     end
@@ -72,18 +79,25 @@ module HammerCLIForeman
     class DeleteCommand < HammerCLI::Apipie::Command
 
       include HammerCLI::Messages
-      resource ForemanApi::Resources::Parameter
 
       option "--name", "NAME", "parameter name", :required => true
 
+      def self.command_name(name=nil)
+        super(name) || "delete_parameter"
+      end
+
+      def self.resource(resource=nil)
+        super(resource) || HammerCLI::Apipie::ResourceDefinition.new(ForemanApi::Resources::Parameter)
+      end
+
       def execute
         params = {
           "id" => name
         }.merge base_action_params
 
-        resource.destroy(params)
+        resource.call(:destroy, params)
         output.print_message success_message if success_message
-        0
+        HammerCLI::EX_OK
       end
 
       def base_action_params

data/lib/hammer_cli_foreman/partition_table.rb

@@ -1,14 +1,16 @@
 require 'hammer_cli'
 require 'foreman_api'
 require 'hammer_cli_foreman/commands'
+require 'hammer_cli_foreman/associating_commands'
 
 module HammerCLIForeman
 
-  class PartitionTable < HammerCLI::AbstractCommand
+  class PartitionTable < HammerCLI::Apipie::Command
+
+    resource ForemanApi::Resources::Ptable
 
     class ListCommand < HammerCLIForeman::ListCommand
 
-      heading "Partition table list"
       output do
         from "ptable" do
           field :id, "Id"
@@ -17,28 +19,27 @@ module HammerCLIForeman
         end
       end
 
-      resource ForemanApi::Resources::Ptable, "index"
       apipie_options
     end
 
 
     class InfoCommand < HammerCLIForeman::InfoCommand
 
-      heading "Partition table info"
       output ListCommand.output_definition do
         from "ptable" 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
 
-      resource ForemanApi::Resources::Ptable, "show"
     end
 
 
     class DumpCommand < HammerCLIForeman::InfoCommand
 
-      resource ForemanApi::Resources::Ptable, "show"
+      command_name "dump"
+      desc "View partition table content."
+
 
       def print_data(partition_table)
         puts partition_table["ptable"]["layout"]
@@ -53,7 +54,6 @@ module HammerCLIForeman
 
       success_message "Partition table created"
       failure_message "Could not create the partition table"
-      resource ForemanApi::Resources::Ptable, "create"
 
       apipie_options :without => [:layout] + declared_identifiers.keys
     end
@@ -65,7 +65,6 @@ module HammerCLIForeman
 
       success_message "Partition table updated"
       failure_message "Could not update the partition table"
-      resource ForemanApi::Resources::Ptable, "update"
 
       apipie_options :without => [:layout] + declared_identifiers.keys
     end
@@ -75,16 +74,13 @@ module HammerCLIForeman
 
       success_message "Partition table deleted"
       failure_message "Could not delete the partition table"
-      resource ForemanApi::Resources::Ptable, "destroy"
     end
 
-    subcommand "list", "List partition tables.", HammerCLIForeman::PartitionTable::ListCommand
-    subcommand "info", "Detailed info about a partition table.", HammerCLIForeman::PartitionTable::InfoCommand
-    subcommand "dump", "View partition table content.", HammerCLIForeman::PartitionTable::DumpCommand
-    subcommand "create", "Create a new partition table.", HammerCLIForeman::PartitionTable::CreateCommand
-    subcommand "update", "Update a partition table.", HammerCLIForeman::PartitionTable::UpdateCommand
-    subcommand "delete", "Delete a partition table.", HammerCLIForeman::PartitionTable::DeleteCommand
 
+    include HammerCLIForeman::AssociatingCommands::OperatingSystem
+
+
+    autoload_subcommands
   end
 
 end

data/lib/hammer_cli_foreman/puppet_class.rb

@@ -0,0 +1,55 @@
+require 'hammer_cli'
+require 'foreman_api'
+require 'hammer_cli_foreman/commands'
+
+module HammerCLIForeman
+
+  class PuppetClass < HammerCLI::Apipie::Command
+
+    resource ForemanApi::Resources::Puppetclass
+
+    class ListCommand < HammerCLIForeman::ListCommand
+
+      output do
+        from "puppetclass" do
+          field :id, "Id"
+          field :name, "Name"
+        end
+      end
+
+      def retrieve_data
+        self.class.unhash_classes(super)
+      end
+
+      def self.unhash_classes(classes)
+        classes.inject([]) { |list, (pp_module, pp_module_classes)| list + pp_module_classes }
+      end
+
+      apipie_options
+    end
+
+
+    class InfoCommand < HammerCLIForeman::InfoCommand
+
+      #FIXME: show environments and hostgroups
+      output ListCommand.output_definition do
+        from "puppetclass" do
+          collection :lookup_keys, "Smart variables" do
+            from :lookup_key do
+              field :key, "Parameter"
+              field :default_value, "Default value"
+            end
+          end
+        end
+      end
+
+    end
+
+
+    autoload_subcommands
+  end
+
+end
+
+HammerCLI::MainCommand.subcommand 'puppet_class', "Browse and read reports.", HammerCLIForeman::PuppetClass
+

data/lib/hammer_cli_foreman/report.rb

@@ -0,0 +1,97 @@
+require 'hammer_cli'
+require 'foreman_api'
+require 'hammer_cli_foreman/commands'
+
+module HammerCLIForeman
+
+  class Report < HammerCLI::Apipie::Command
+
+    resource ForemanApi::Resources::Report
+
+    class ListCommand < HammerCLIForeman::ListCommand
+
+      output do
+        from "report" do
+          field :id, "Id"
+          field :host_name, "Host"
+          field :reported_at, "Last report", Fields::Date
+          from "status" do
+            field :applied, "Applied"
+            field :restarted, "Restarted"
+            field :failed, "Failed"
+            field :failed_restarts, "Restart Failures"
+            field :skipped, "Skipped"
+            field :pending, "Pending"
+          end
+        end
+      end
+
+      apipie_options
+    end
+
+
+    class InfoCommand < HammerCLIForeman::InfoCommand
+
+      identifiers :id
+
+      output do
+        from "report" do
+          field :id, "Id"
+          field :host_name, "Host"
+          field :reported_at, "Reported at", Fields::Date
+          label "Report status" do
+            from "status" do
+              field :applied, "Applied"
+              field :restarted, "Restarted"
+              field :failed, "Failed"
+              field :failed_restarts, "Restart Failures"
+              field :skipped, "Skipped"
+              field :pending, "Pending"
+            end
+          end
+          label "Report metrics" do
+            from "metrics" do
+              from "time" do
+                field :config_retrieval, "config_retrieval"
+                field :exec, "exec"
+                field :file, "file"
+                field :package, "package"
+                field :service, "service"
+                field :user, "user"
+                field :yumrepo, "yumrepo"
+                field :filebucket, "filebucket"
+                field :cron, "cron"
+                field :total, "total"
+              end
+            end
+          end
+          field :logs, "Logs", Fields::Collection do
+            from :log do
+              from :source do
+                field :source, "Resource"
+              end
+              from :message do
+                field :message, "Message"
+              end
+            end
+          end
+        end
+      end
+
+    end
+
+
+    class DeleteCommand < HammerCLIForeman::DeleteCommand
+      identifiers :id
+      success_message "Report has been deleted"
+      failure_message "Could not delete the report"
+    end
+
+
+    autoload_subcommands
+  end
+
+end
+
+HammerCLI::MainCommand.subcommand 'report', "Browse and read reports.", HammerCLIForeman::Report
+

data/lib/hammer_cli_foreman/resource_supported_test.rb

@@ -12,7 +12,7 @@ module HammerCLIForeman
     end
 
     def resource_supported?
-      resource.index
+      resource.call(:index)
       true
     rescue RestClient::ResourceNotFound => e
       false

data/lib/hammer_cli_foreman/smart_proxy.rb

@@ -9,7 +9,6 @@ module HammerCLIForeman
       resource ForemanApi::Resources::SmartProxy, "index"
 
       #FIXME: search by unknown type returns 500 from the server, propper error handling should resove this
-      heading "Smart Proxy list"
       output do
         from "smart_proxy" do
           field :id, "Id"
@@ -32,12 +31,11 @@ module HammerCLIForeman
         sp
       end
 
-      heading "Smart proxy info"
       output ListCommand.output_definition do
         from "smart_proxy" do
-          field :_features,  "Features",   HammerCLI::Output::Fields::List
-          field :created_at, "Created at", HammerCLI::Output::Fields::Date
-          field :updated_at, "Updated at", HammerCLI::Output::Fields::Date
+          field :_features,  "Features",   Fields::List
+          field :created_at, "Created at", Fields::Date
+          field :updated_at, "Updated at", Fields::Date
         end
       end
 
@@ -65,7 +63,7 @@ module HammerCLIForeman
 
 
     class DeleteCommand < HammerCLIForeman::DeleteCommand
-      
+
       success_message "Smart proxy deleted"
       failure_message "Could not delete the proxy"
       resource ForemanApi::Resources::SmartProxy, "destroy"
@@ -73,11 +71,7 @@ module HammerCLIForeman
       apipie_options
     end
 
-    subcommand "list", "List smart proxies.", HammerCLIForeman::SmartProxy::ListCommand
-    subcommand "info", "Detailed info about an smart proxy.", HammerCLIForeman::SmartProxy::InfoCommand
-    subcommand "create", "Create new smart proxy.", HammerCLIForeman::SmartProxy::CreateCommand
-    subcommand "update", "Update an smart proxy.", HammerCLIForeman::SmartProxy::UpdateCommand
-    subcommand "delete", "Delete an smart proxy.", HammerCLIForeman::SmartProxy::DeleteCommand
+    autoload_subcommands
   end
 
 end

data/lib/hammer_cli_foreman/subnet.rb

@@ -8,7 +8,6 @@ module HammerCLIForeman
     class ListCommand < HammerCLIForeman::ListCommand
       resource ForemanApi::Resources::Subnet, "index"
 
-      heading "Subnet list"
       output do
         from "subnet" do
           field :id, "Id"
@@ -26,17 +25,16 @@ module HammerCLIForeman
 
       resource ForemanApi::Resources::Subnet, "show"
 
-      heading "Subnet info"
       output ListCommand.output_definition do
         from "subnet" do
           field :priority, "Priority"
-          field :dns, "DNS", HammerCLI::Output::Fields::Server
+          field :dns, "DNS", Fields::Server
           field :dns_primary, "Primary DNS"
           field :dns_secondary, "Secondary DNS"
-          field :domain_ids, "Domain ids", HammerCLI::Output::Fields::List
-          field :tftp, "TFTP", HammerCLI::Output::Fields::Server
+          field :domain_ids, "Domain ids", Fields::List
+          field :tftp, "TFTP", Fields::Server
           field :tftp_id, "TFTP id"
-          field :dhcp, "DHCP", HammerCLI::Output::Fields::Server
+          field :dhcp, "DHCP", Fields::Server
           field :dhcp_id, "DHCP id"
           field :vlanid, "vlan id"
           field :gateway, "Gateway"
@@ -77,11 +75,7 @@ module HammerCLIForeman
       apipie_options
     end
 
-    subcommand "list", "List subnets.", HammerCLIForeman::Subnet::ListCommand
-    subcommand "info", "Detailed info about an subnet.", HammerCLIForeman::Subnet::InfoCommand
-    subcommand "create", "Create new subnet.", HammerCLIForeman::Subnet::CreateCommand
-    subcommand "update", "Update an subnet.", HammerCLIForeman::Subnet::UpdateCommand
-    subcommand "delete", "Delete an subnet.", HammerCLIForeman::Subnet::DeleteCommand
+    autoload_subcommands
   end
 
 end

data/lib/hammer_cli_foreman/template.rb

@@ -1,14 +1,16 @@
 require 'hammer_cli'
 require 'foreman_api'
 require 'hammer_cli_foreman/commands'
+require 'hammer_cli_foreman/associating_commands'
 
 module HammerCLIForeman
 
-  class Template < HammerCLI::AbstractCommand
+  class Template < HammerCLI::Apipie::Command
+
+    resource ForemanApi::Resources::ConfigTemplate
 
     class ListCommand < HammerCLIForeman::ListCommand
 
-      heading "Template list"
       output do
         from "config_template" do
           field :id, "Id"
@@ -28,7 +30,6 @@ module HammerCLIForeman
         end
       end
 
-      resource ForemanApi::Resources::ConfigTemplate, "index"
       apipie_options
     end
 
@@ -39,10 +40,9 @@ module HammerCLIForeman
       #       Api currently does not accept names containing a dot
       identifiers :id
 
-      heading "Template info"
       output ListCommand.output_definition do
         from "config_template" do
-          field :operatingsystem_ids, "OS ids", HammerCLI::Output::Fields::List
+          field :operatingsystem_ids, "OS ids", Fields::List
         end
       end
 
@@ -56,13 +56,14 @@ module HammerCLIForeman
         template
       end
 
-      resource ForemanApi::Resources::ConfigTemplate, "show"
     end
 
 
     class ListKindsCommand < HammerCLIForeman::ListCommand
 
-      heading "Template kind list"
+      command_name "kinds"
+      desc "List available config template kinds."
+
       output do
         from "template_kind" do
           field :name, "Name"
@@ -80,9 +81,10 @@ module HammerCLIForeman
 
     class DumpCommand < HammerCLIForeman::InfoCommand
 
-      identifiers :id
+      command_name "dump"
+      desc "View config template content."
 
-      resource ForemanApi::Resources::ConfigTemplate, "show"
+      identifiers :id
 
       def print_data(template)
         puts template["config_template"]["template"]
@@ -98,7 +100,6 @@ module HammerCLIForeman
 
       success_message "Config template created"
       failure_message "Could not create the config template"
-      resource ForemanApi::Resources::ConfigTemplate, "create"
 
       def snippet
         type == "snippet"
@@ -123,7 +124,6 @@ module HammerCLIForeman
 
       success_message "Config template updated"
       failure_message "Could not update the config template"
-      resource ForemanApi::Resources::ConfigTemplate, "update"
 
       def snippet
         type == "snippet"
@@ -145,17 +145,13 @@ module HammerCLIForeman
 
       success_message "Config template deleted"
       failure_message "Could not delete the config template"
-      resource ForemanApi::Resources::ConfigTemplate, "destroy"
     end
 
-    subcommand "list", "List config templates.", HammerCLIForeman::Template::ListCommand
-    subcommand "info", "Detailed info about a config template.", HammerCLIForeman::Template::InfoCommand
-    subcommand "dump", "View config template content.", HammerCLIForeman::Template::DumpCommand
-    subcommand "create", "Create a new config template.", HammerCLIForeman::Template::CreateCommand
-    subcommand "update", "Update a config template.", HammerCLIForeman::Template::UpdateCommand
-    subcommand "delete", "Delete a config template.", HammerCLIForeman::Template::DeleteCommand
-    subcommand "kinds", "List available config template kinds.", HammerCLIForeman::Template::ListKindsCommand
 
+    include HammerCLIForeman::AssociatingCommands::OperatingSystem
+
+
+    autoload_subcommands
   end
 
 end