hammer_cli_foreman 0.1.1 → 0.1.2

data/lib/hammer_cli_foreman/organization.rb

@@ -41,7 +41,9 @@ module HammerCLIForeman
       end
 
 
-      build_options
+      build_options do |o|
+        o.expand.primary(:organizations)
+      end
     end
 
 
@@ -75,7 +77,9 @@ module HammerCLIForeman
       success_message _("Organization deleted")
       failure_message _("Could not delete the organization")
 
-      build_options
+      build_options do |o|
+        o.expand.primary(:organizations)
+      end
     end
 
 
@@ -94,5 +98,5 @@ module HammerCLIForeman
 
 end
 
-HammerCLI::MainCommand.subcommand 'organization', _("Manipulate organizations."), HammerCLIForeman::Organization
+
 

data/lib/hammer_cli_foreman/param_filters.rb

@@ -0,0 +1,61 @@
+module HammerCLIForeman
+
+  class AbstractParamsFilter
+
+    def for_action(action)
+      filter(action.params)
+    end
+
+    def filter(params)
+      []
+    end
+
+  end
+
+  class ParamsFlattener < AbstractParamsFilter
+
+    def filter(params)
+      flatten_params(params)
+    end
+
+    private
+
+    def flatten_params(params)
+      result = params
+      params.each do |p|
+        result += flatten_params(p.params)
+      end
+      result
+    end
+
+  end
+
+  class IdParamsFilter < AbstractParamsFilter
+
+    def initialize(options={})
+      @required = !(options[:only_required] == false)
+    end
+
+    def filter(params)
+      params = ParamsFlattener.new.filter(params)
+      params = params.reject{ |p| !(p.name.end_with?("_id")) }
+      params = params.reject{ |p| !(p.required?) } if @required
+      params
+    end
+
+  end
+
+
+  class ParamsNameFilter < AbstractParamsFilter
+
+    def initialize(name)
+      @name = name
+    end
+
+    def filter(params)
+      ParamsFlattener.new.filter(params).select{ |p| p.name == @name }
+    end
+
+  end
+
+end

data/lib/hammer_cli_foreman/parameter.rb

@@ -7,7 +7,7 @@ module HammerCLIForeman
     class AbstractParameterCommand < HammerCLIForeman::Command
 
       def self.parameter_resource
-        HammerCLIForeman.foreman_resource(:parameters)
+        HammerCLIForeman.foreman_resource!(:parameters)
       end
 
       def parameter_resource
@@ -34,10 +34,12 @@ module HammerCLIForeman
         }
       end
 
-      def self.custom_option_builders
-        [
+      def self.create_option_builder
+        builder = super
+        builder.builders = [
           DependentSearchablesOptionBuilder.new(resource, searchables)
         ]
+        builder
       end
 
     end

data/lib/hammer_cli_foreman/partition_table.rb

@@ -77,4 +77,4 @@ module HammerCLIForeman
 
 end
 
-HammerCLI::MainCommand.subcommand 'partition-table', _("Manipulate partition tables."), HammerCLIForeman::PartitionTable
+

data/lib/hammer_cli_foreman/puppet_class.rb

@@ -57,5 +57,5 @@ module HammerCLIForeman
 
 end
 
-HammerCLI::MainCommand.subcommand 'puppet-class', _("Search puppet modules."), HammerCLIForeman::PuppetClass
+
 

data/lib/hammer_cli_foreman/references.rb

@@ -29,6 +29,14 @@ module HammerCLIForeman
       end
     end
 
+    def self.usergroups(dsl)
+      dsl.build do
+        collection :usergroups, _("User groups"), :numbered => false do
+          custom_field Fields::Reference
+        end
+      end
+    end
+
     def self.smart_proxies(dsl)
       dsl.build do
         collection :smart_proxies, _("Smart proxies"), :numbered => false do
@@ -118,5 +126,13 @@ module HammerCLIForeman
       end
     end
 
+    def self.roles(dsl)
+      dsl.build do
+        collection :roles, _("Roles"), :numbered => false do
+          custom_field Fields::Reference
+        end
+      end
+    end
+
   end
 end

data/lib/hammer_cli_foreman/report.rb

@@ -85,5 +85,5 @@ module HammerCLIForeman
 
 end
 
-HammerCLI::MainCommand.subcommand 'report', _("Browse and read reports."), HammerCLIForeman::Report
+
 

data/lib/hammer_cli_foreman/role.rb

@@ -0,0 +1,78 @@
+require 'hammer_cli_foreman/filter'
+
+module HammerCLIForeman
+
+  class Role < HammerCLIForeman::Command
+
+    resource :roles
+
+    class ListCommand < HammerCLIForeman::ListCommand
+      output do
+        field :id, _("Id")
+        field :name, _("Name")
+      end
+
+      build_options
+    end
+
+
+    class FiltersCommand < HammerCLIForeman::ListCommand
+      command_name "filters"
+      resource :filters, :index
+
+      option "--id", "ID", _("User role id")
+
+      output HammerCLIForeman::Filter::ListCommand.output_definition
+
+      def request_params
+        role_id = get_resource_id(HammerCLIForeman.foreman_resource(:roles))
+
+        params = super
+        params[:search] = "role_id = \"#{role_id}\""
+        params
+      end
+
+      def extend_data(filter)
+        filter['resource_type'] ||= _("(Miscellaneous)")
+        filter['search'] ||= _("none")
+        filter['permissions'] = filter.fetch('permissions', []).collect{|p| p["name"]}
+        filter
+      end
+
+      build_options do |o|
+        o.expand.primary(:roles)
+        o.without(:search)
+      end
+    end
+
+
+    class CreateCommand < HammerCLIForeman::CreateCommand
+      success_message _("User role [%<name>s] created")
+      failure_message _("Could not create the user role")
+
+      build_options
+    end
+
+
+    class UpdateCommand < HammerCLIForeman::UpdateCommand
+      success_message _("User role [%<name>s] updated")
+      failure_message _("Could not update the user role")
+
+      build_options
+    end
+
+
+    class DeleteCommand < HammerCLIForeman::DeleteCommand
+      success_message _("User role [%<name>s] deleted")
+      failure_message _("Could not delete the user roles")
+
+      build_options
+    end
+
+    autoload_subcommands
+  end
+
+end
+
+
+

data/lib/hammer_cli_foreman/smart_class_parameter.rb

@@ -104,6 +104,6 @@ module HammerCLIForeman
 
   end
 
-  HammerCLI::MainCommand.subcommand 'sc-param', _("Manipulate smart class parameters."), HammerCLIForeman::SmartClassParameter
+
 
 end

data/lib/hammer_cli_foreman/smart_proxy.rb

@@ -71,7 +71,10 @@ module HammerCLIForeman
 
       option "--dryrun", :flag, _("Do not run the import")
 
-      build_options :without => [:smart_proxy_id, :dryrun]
+      build_options do |o|
+        o.without(:smart_proxy_id, :dryrun)
+        o.expand.except(:smart_proxies)
+      end
 
       def request_params
         opts = super
@@ -80,9 +83,21 @@ module HammerCLIForeman
       end
     end
 
+
+    class RefreshFeaturesCommand < HammerCLIForeman::Command
+
+      action :refresh
+
+      command_name    "refresh-features"
+      success_message _("Smart proxy features were refreshed")
+      failure_message _("Refresh of smart proxy features failed")
+
+      build_options
+    end
+
     autoload_subcommands
   end
 
 end
 
-HammerCLI::MainCommand.subcommand 'proxy', _("Manipulate smart proxies."), HammerCLIForeman::SmartProxy
+

data/lib/hammer_cli_foreman/smart_variables.rb

@@ -0,0 +1,111 @@
+module HammerCLIForeman
+
+  class SmartVariableBriefList < HammerCLIForeman::AssociatedResourceListCommand
+    resource :smart_variables, :index
+
+    output do
+      field :id, _("Id")
+
+      field :parameter, _("Parameter")
+      field :default_value, _("Default Value")
+      field :override, _("Override")
+    end
+
+    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 SmartVariableList < SmartVariableBriefList
+
+    output do
+      from :puppetclass do
+        field :name, _("Puppet class")
+        field :id, _("Class Id"), Fields::Id
+      end
+    end
+  end
+
+  class SmartVariable < HammerCLIForeman::Command
+
+    resource :smart_variables
+
+    class ListCommand < HammerCLIForeman::ListCommand
+
+      output SmartVariablesList.output_definition
+
+      build_options
+    end
+
+    class InfoCommand < HammerCLIForeman::InfoCommand
+
+      output ListCommand.output_definition do
+        field :description, _("Description")
+        field :parameter_type, _("Type")
+        field :required, _("Required")
+
+        label _("Validator") do
+          field :validator_type, _("Type")
+          field :validator_rule, _("Rule")
+        end
+        label _("Override values") do
+          field :override_value_order, _("Order"), Fields::List
+          field :override_values_count, _("Count")
+          collection :override_values, "Values" do
+            label  _("Value") do
+              field :id, _('Id')
+              field :match, _('Match')
+              field :value, _('Value')
+            end
+          end
+        end
+        HammerCLIForeman::References.environments(self)
+        HammerCLIForeman::References.timestamps(self)
+      end
+
+      def extend_data(res)
+        res['override_value_order'] = res['override_value_order'].split("\n")
+        res
+      end
+
+      build_options
+    end
+
+    class UpdateCommand < HammerCLIForeman::UpdateCommand
+
+      success_message _("Parameter updated")
+      failure_message _("Could not update the parameter")
+
+      build_options :without => [:parameter_type, :validator_type, :override, :required]
+
+      option "--override", "OVERRIDE", _("Override this parameter."),
+        :format => HammerCLI::Options::Normalizers::Bool.new
+      option "--required", "REQUIRED", _("This parameter is required."),
+        :format => HammerCLI::Options::Normalizers::Bool.new
+      option "--parameter-type", "PARAMETER_TYPE", _("Type of the parameter."),
+        :format => HammerCLI::Options::Normalizers::Enum.new(
+            ['string', 'boolean', 'integer', 'real', 'array', 'hash', 'yaml', 'json'])
+      option "--validator-type", "VALIDATOR_TYPE", _("Type of the validator."),
+        :format => HammerCLI::Options::Normalizers::Enum.new(['regexp', 'list', ''])
+    end
+
+    class AddOverrideValueCommand < HammerCLIForeman::UpdateCommand
+
+    end
+
+    autoload_subcommands
+
+  end
+
+  HammerCLI::MainCommand.subcommand 'smart-variable', _("Manipulate smart variables."), HammerCLIForeman::SmartVariable
+
+end

data/lib/hammer_cli_foreman/subnet.rb

@@ -70,5 +70,5 @@ module HammerCLIForeman
 
 end
 
-HammerCLI::MainCommand.subcommand 'subnet', _("Manipulate subnets."), HammerCLIForeman::Subnet
+
 

data/lib/hammer_cli_foreman/template.rb

@@ -12,7 +12,7 @@ module HammerCLIForeman
 
       def option_template_kind_id
         kinds = HammerCLIForeman.collection_to_common_format(
-          HammerCLIForeman.foreman_resource(:template_kinds).call(:index))
+          HammerCLIForeman.foreman_resource!(:template_kinds).call(:index))
         table = kinds.inject({}){ |result, k| result.update(k["name"] => k["id"]) }
         table[option_type]
       end
@@ -108,7 +108,10 @@ module HammerCLIForeman
 
       include TemplateCreateUpdateCommons
 
-      build_options :without => [:template_combinations_attributes, :template, :snippet, :template_kind_id]
+      build_options do |o|
+        o.without(:template_combinations_attributes, :template, :snippet, :template_kind_id)
+        o.expand.except(:template_kinds)
+      end
     end
 
 
@@ -123,7 +126,10 @@ module HammerCLIForeman
 
       include TemplateCreateUpdateCommons
 
-      build_options :without => [:template_combinations_attributes, :template, :snippet, :template_kind_id]
+      build_options do |o|
+        o.without(:template_combinations_attributes, :template, :snippet, :template_kind_id)
+        o.expand.except(:template_kinds)
+      end
     end
 
 
@@ -145,5 +151,5 @@ module HammerCLIForeman
 
 end
 
-HammerCLI::MainCommand.subcommand 'template', _("Manipulate config templates."), HammerCLIForeman::Template
+