hammer_cli_foreman 0.1.0 → 0.1.1

data/lib/hammer_cli_foreman/references.rb

@@ -0,0 +1,122 @@
+module HammerCLIForeman
+
+  module References
+
+
+    def self.timestamps(dsl)
+      dsl.build do
+        field :created_at, _("Created at"), Fields::Date
+        field :updated_at, _("Updated at"), Fields::Date
+      end
+    end
+
+    def self.taxonomies(dsl)
+      dsl.build do
+        collection :locations, _("Locations"), :numbered => false, :hide_blank => true do
+          custom_field Fields::Reference
+        end
+        collection :organizations, _("Organizations"), :numbered => false, :hide_blank => true do
+          custom_field Fields::Reference
+        end
+      end
+    end
+
+    def self.users(dsl)
+      dsl.build do
+        collection :users, _("Users"), :numbered => false do
+          custom_field Fields::Reference, :name_key => :login
+        end
+      end
+    end
+
+    def self.smart_proxies(dsl)
+      dsl.build do
+        collection :smart_proxies, _("Smart proxies"), :numbered => false do
+          custom_field Fields::Reference
+        end
+      end
+    end
+
+    def self.compute_resources(dsl)
+      dsl.build do
+        collection :compute_resources, _("Compute resources"), :numbered => false do
+          custom_field Fields::Reference
+        end
+      end
+    end
+
+    def self.media(dsl)
+      dsl.build do
+        collection :media, _("Installation media"), :numbered => false do
+          custom_field Fields::Reference
+        end
+      end
+    end
+
+    def self.config_templates(dsl)
+      dsl.build do
+        collection :config_templates, _("Templates"), :numbered => false do
+          custom_field Fields::Template
+        end
+      end
+    end
+
+    def self.domains(dsl)
+      dsl.build do
+        collection :domains, _("Domains"), :numbered => false do
+          custom_field Fields::Reference
+        end
+      end
+    end
+
+    def self.environments(dsl)
+      dsl.build do
+        collection :environments, _("Environments"), :numbered => false do
+          custom_field Fields::Reference
+        end
+      end
+    end
+
+    def self.hostgroups(dsl)
+      dsl.build do
+        collection :hostgroups, _("Hostgroups"), :numbered => false do
+          custom_field Fields::Reference
+        end
+      end
+    end
+
+    def self.subnets(dsl)
+      dsl.build do
+        collection :subnets, _("Subnets"), :numbered => false do
+          custom_field Fields::Reference, :details => :network_address
+        end
+      end
+    end
+
+
+    def self.parameters(dsl)
+      dsl.build do
+        collection :parameters, _("Parameters"), :numbered => false do
+          custom_field Fields::KeyValue
+        end
+      end
+    end
+
+    def self.puppetclasses(dsl)
+      dsl.build do
+        collection :puppetclasses, _("Puppetclasses"), :numbered => false do
+          custom_field Fields::Reference
+        end
+      end
+    end
+
+    def self.operating_systems(dsl)
+      dsl.build do
+        collection :operatingsystems, _("Operating systems"), :numbered => false do
+          custom_field Fields::Reference, :name_key => :fullname
+        end
+      end
+    end
+
+  end
+end

data/lib/hammer_cli_foreman/report.rb

@@ -20,14 +20,12 @@ module HammerCLIForeman
         end
       end
 
-      apipie_options
+      build_options
     end
 
 
     class InfoCommand < HammerCLIForeman::InfoCommand
 
-      identifiers :id
-
       output do
         field :id, _("Id")
         field :host_name, _("Host")
@@ -70,16 +68,15 @@ module HammerCLIForeman
         end
       end
 
-      apipie_options
+      build_options
     end
 
 
     class DeleteCommand < HammerCLIForeman::DeleteCommand
-      identifiers :id
       success_message _("Report has been deleted")
       failure_message _("Could not delete the report")
 
-      apipie_options
+      build_options
     end
 
 

data/lib/hammer_cli_foreman/searchables_option_builder.rb

@@ -0,0 +1,99 @@
+
+module HammerCLIForeman
+
+  class SearchablesOptionBuilder < HammerCLI::AbstractOptionBuilder
+
+    def initialize(resource, searchables)
+      @resource = resource
+      @searchables = searchables
+    end
+
+    def build(builder_params={})
+      @searchables.for(@resource).collect do |s|
+        option(
+          optionamize("--#{s.name}"),
+          s.name.upcase,
+          s.description
+        )
+      end
+    end
+  end
+
+
+  class DependentSearchablesOptionBuilder < HammerCLI::AbstractOptionBuilder
+
+    def initialize(dependent_resources, searchables)
+      @dependent_resources = dependent_resources.is_a?(Array) ? dependent_resources : [dependent_resources]
+      @searchables = searchables
+    end
+
+
+    def build(builder_params={})
+      @dependent_resources.collect do |res|
+        dependent_options(res)
+      end.flatten(1)
+    end
+
+    def dependent_options(resource)
+      options = []
+      searchables = @searchables.for(resource)
+      resource_name = resource.singular_name
+
+      unless searchables.empty?
+        first = searchables[0]
+        remaining = searchables[1..-1] || []
+
+        # First option is named by the resource
+        # Eg. --organization with accessor option_organization_name
+        options << option(
+          optionamize("--#{resource_name}"),
+          "#{resource_name}_#{first.name}".upcase,
+          " ",
+          :attribute_name => HammerCLI.option_accessor_name("#{resource_name}_#{first.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 << option(
+            optionamize("--#{resource_name}-#{s.name}"),
+            "#{resource_name}_#{s.name}".upcase,
+            " "
+          )
+        end
+      end
+
+      options << option(
+        optionamize("--#{resource_name}-id"),
+        "#{resource_name}_id".upcase,
+        " "
+      )
+      options
+    end
+
+  end
+
+  class SearchablesUpdateOptionBuilder < HammerCLI::AbstractOptionBuilder
+
+    def initialize(resource, searchables)
+      @resource = resource
+      @searchables = searchables
+    end
+
+    def build(builder_params={})
+
+      @searchables.for(@resource).collect do |s|
+        if s.editable?
+          option(
+            optionamize("--new-#{s.name}"),
+            "NEW_#{s.name.upcase}",
+            " "
+          )
+        end
+      end.compact
+    end
+
+  end
+
+
+end

data/lib/hammer_cli_foreman/smart_class_parameter.rb

@@ -1,6 +1,6 @@
 module HammerCLIForeman
 
-  class SmartClassParametersBriefList < HammerCLIForeman::ListCommand
+  class SmartClassParametersBriefList < HammerCLIForeman::AssociatedResourceListCommand
     resource :smart_class_parameters, :index
     command_name 'sc-params'
 
@@ -12,13 +12,18 @@ module HammerCLIForeman
       field :override, _("Override")
     end
 
-    def retrieve_data
+    def send_request
       res = super
       # FIXME: API returns doubled records, probably just if filtered by puppetclasses
       # it seems group by environment is missing
       # having the uniq to fix that
       HammerCLI::Output::RecordCollection.new(res.uniq, :meta => res.meta)
     end
+
+    def self.build_options(options={})
+      options[:without] ||= [:host_id, :hostgroup_id, :puppetclass_id, :environment_id]
+      super(options)
+    end
   end
 
   class SmartClassParametersList < SmartClassParametersBriefList
@@ -35,9 +40,11 @@ module HammerCLIForeman
 
     resource :smart_class_parameters
 
-    class ListCommand < HammerCLIForeman::SmartClassParametersList
-      command_name 'list'
-      apipie_options
+    class ListCommand < HammerCLIForeman::ListCommand
+
+      output SmartClassParametersList.output_definition
+
+      build_options
     end
 
     class InfoCommand < HammerCLIForeman::InfoCommand
@@ -46,8 +53,7 @@ module HammerCLIForeman
         field :description, _("Description")
         field :parameter_type, _("Type")
         field :required, _("Required")
-        field :_environments, _("Environments"), Fields::List
-        field :_environment_ids, _("Environment Ids"), Fields::List
+
         label _("Validator") do
           field :validator_type, _("Type")
           field :validator_rule, _("Rule")
@@ -63,18 +69,16 @@ module HammerCLIForeman
             end
           end
         end
-        field :created_at, _("Created at"), Fields::Date
-        field :updated_at, _("Updated at"), Fields::Date
+        HammerCLIForeman::References.environments(self)
+        HammerCLIForeman::References.timestamps(self)
       end
 
       def extend_data(res)
         res['override_value_order'] = res['override_value_order'].split("\n")
-        res['_environments'] = res['environments'].map { |e| e['name']}
-        res['_environment_ids'] = res['environments'].map { |e| e['id']}
         res
       end
 
-      apipie_options
+      build_options
     end
 
     class UpdateCommand < HammerCLIForeman::UpdateCommand
@@ -82,7 +86,7 @@ module HammerCLIForeman
       success_message _("Parameter updated")
       failure_message _("Could not update the parameter")
 
-      apipie_options :without => [:parameter_type, :validator_type, :id, :override, :required]
+      build_options :without => [:parameter_type, :validator_type, :override, :required]
 
       option "--override", "OVERRIDE", _("Override this parameter."),
         :format => HammerCLI::Options::Normalizers::Bool.new

data/lib/hammer_cli_foreman/smart_proxy.rb

@@ -6,72 +6,62 @@ module HammerCLIForeman
 
     class ListCommand < HammerCLIForeman::ListCommand
 
-      action :index
-
       #FIXME: search by unknown type returns 500 from the server, propper error handling should resove this
       output do
         field :id, _("Id")
         field :name, _("Name")
         field :url, _("URL")
+        field :_features, _( "Features"), Fields::List, :width => 25, :hide_blank => true
+      end
+
+      def extend_data(proxy)
+        proxy['_features'] = proxy['features'].map { |f| f['name'] } if proxy['features']
+        proxy
       end
 
-      apipie_options
+      build_options
     end
 
 
     class InfoCommand < HammerCLIForeman::InfoCommand
 
-      action :show
-
       output ListCommand.output_definition do
-        field :_features, _( "Features"),   Fields::List
-        field :created_at, _("Created at"), Fields::Date
-        field :updated_at, _("Updated at"), Fields::Date
-      end
-
-      def extend_data(proxy)
-        proxy['_features'] = proxy['features'].map { |f| f['name'] }
-        proxy
+        collection :features,  _("Features"), :numbered => false do
+          custom_field Fields::Reference
+        end
+        HammerCLIForeman::References.taxonomies(self)
+        HammerCLIForeman::References.timestamps(self)
       end
 
-      apipie_options
+      build_options
     end
 
 
     class CreateCommand < HammerCLIForeman::CreateCommand
-
-      action :create
-
       success_message _("Smart proxy created")
       failure_message _("Could not create the proxy")
 
-      apipie_options
+      build_options
     end
 
 
     class UpdateCommand < HammerCLIForeman::UpdateCommand
-
-      action :update
-
       success_message _("Smart proxy updated")
       failure_message _("Could not update the proxy")
 
-      apipie_options
+      build_options
     end
 
 
     class DeleteCommand < HammerCLIForeman::DeleteCommand
-
-      action :destroy
-
       success_message _("Smart proxy deleted")
       failure_message _("Could not delete the proxy")
 
-      apipie_options
+      build_options
     end
 
 
-    class ImportPuppetClassesCommand < HammerCLIForeman::WriteCommand
+    class ImportPuppetClassesCommand < HammerCLIForeman::Command
 
       action :import_puppetclasses
 
@@ -81,7 +71,7 @@ module HammerCLIForeman
 
       option "--dryrun", :flag, _("Do not run the import")
 
-      apipie_options :without => [:smart_proxy_id, :dryrun]
+      build_options :without => [:smart_proxy_id, :dryrun]
 
       def request_params
         opts = super

data/lib/hammer_cli_foreman/subnet.rb

@@ -13,7 +13,7 @@ module HammerCLIForeman
         field :mask, _("Mask")
       end
 
-      apipie_options
+      build_options
     end
 
 
@@ -21,22 +21,21 @@ module HammerCLIForeman
 
       output ListCommand.output_definition do
         field :priority, _("Priority")
-        field :dns_id, _("DNS id")
-        field :dns, _("DNS"), Fields::Server
+        field :dns, _("DNS"), Fields::Reference, :details => :url
         field :dns_primary, _("Primary DNS")
         field :dns_secondary, _("Secondary DNS")
-        field :domain_ids, _("Domain ids"), Fields::List
-        field :tftp, _("TFTP"), Fields::Server
-        field :tftp_id, _("TFTP id")
-        field :dhcp, _("DHCP"), Fields::Server
-        field :dhcp_id, _("DHCP id")
-        field :vlanid, _("vlan id")
+        field :tftp, _("TFTP"), Fields::Reference, :details => :url
+        field :dhcp, _("DHCP"), Fields::Reference, :details => :url
+
+        field :vlanid, _("VLAN ID")
         field :gateway, _("Gateway")
         field :from, _("From")
         field :to, _("To")
+        HammerCLIForeman::References.domains(self)
+        HammerCLIForeman::References.taxonomies(self)
       end
 
-      apipie_options
+      build_options
     end
 
 
@@ -45,7 +44,7 @@ module HammerCLIForeman
       success_message _("Subnet created")
       failure_message _("Could not create the subnet")
 
-      apipie_options
+      build_options
     end
 
 
@@ -54,7 +53,7 @@ module HammerCLIForeman
       success_message _("Subnet updated")
       failure_message _("Could not update the subnet")
 
-      apipie_options
+      build_options
     end
 
 
@@ -63,7 +62,7 @@ module HammerCLIForeman
       success_message _("Subnet deleted")
       failure_message _("Could not delete the subnet")
 
-      apipie_options
+      build_options
     end
 
     autoload_subcommands