hammer_cli_foreman 0.1.0 → 0.1.1

data/lib/hammer_cli_foreman/location.rb

@@ -14,19 +14,33 @@ module HammerCLIForeman
         field :name, _("Name")
       end
 
-      apipie_options
+      build_options
     end
 
 
     class InfoCommand < HammerCLIForeman::InfoCommand
       include HammerCLIForeman::ResourceSupportedTest
 
+      option '--id', 'ID', _("Location numeric id to search by")
+
       output ListCommand.output_definition do
-        field :created_at, _("Created at"), Fields::Date
-        field :updated_at, _("Updated at"), Fields::Date
+        HammerCLIForeman::References.users(self)
+        HammerCLIForeman::References.smart_proxies(self)
+        HammerCLIForeman::References.subnets(self)
+        HammerCLIForeman::References.compute_resources(self)
+        HammerCLIForeman::References.media(self)
+        HammerCLIForeman::References.config_templates(self)
+        HammerCLIForeman::References.domains(self)
+        HammerCLIForeman::References.environments(self)
+        HammerCLIForeman::References.hostgroups(self)
+        HammerCLIForeman::References.parameters(self)
+        collection :organizations, _("Organizations"), :numbered => false, :hide_blank => true do
+          custom_field Fields::Reference
+        end
+        HammerCLIForeman::References.timestamps(self)
       end
 
-      apipie_options
+      build_options
     end
 
 
@@ -36,39 +50,43 @@ module HammerCLIForeman
       success_message _("Location created")
       failure_message _("Could not create the location")
 
-      apipie_options
+      build_options
     end
 
 
     class UpdateCommand < HammerCLIForeman::UpdateCommand
       include HammerCLIForeman::ResourceSupportedTest
 
+      option '--id', 'ID', _("Location numeric id to search by")
+
       success_message _("Location updated")
       failure_message _("Could not update the location")
 
-      apipie_options
+      build_options
     end
 
 
     class DeleteCommand < HammerCLIForeman::DeleteCommand
       include HammerCLIForeman::ResourceSupportedTest
 
+      option '--id', 'ID', _("Location numeric id to search by")
+
       success_message _("Location deleted")
       failure_message _("Could not delete the location")
 
-      apipie_options
+      build_options
     end
 
-    include HammerCLIForeman::AssociatingCommands::Hostgroup
-    include HammerCLIForeman::AssociatingCommands::Environment
-    include HammerCLIForeman::AssociatingCommands::Domain
-    include HammerCLIForeman::AssociatingCommands::Medium
-    include HammerCLIForeman::AssociatingCommands::Subnet
-    include HammerCLIForeman::AssociatingCommands::ComputeResource
-    include HammerCLIForeman::AssociatingCommands::SmartProxy
-    include HammerCLIForeman::AssociatingCommands::User
-    include HammerCLIForeman::AssociatingCommands::ConfigTemplate
-    include HammerCLIForeman::AssociatingCommands::Organization
+    HammerCLIForeman::AssociatingCommands::Hostgroup.extend_command(self)
+    HammerCLIForeman::AssociatingCommands::Environment.extend_command(self)
+    HammerCLIForeman::AssociatingCommands::Domain.extend_command(self)
+    HammerCLIForeman::AssociatingCommands::Medium.extend_command(self)
+    HammerCLIForeman::AssociatingCommands::Subnet.extend_command(self)
+    HammerCLIForeman::AssociatingCommands::ComputeResource.extend_command(self)
+    HammerCLIForeman::AssociatingCommands::SmartProxy.extend_command(self)
+    HammerCLIForeman::AssociatingCommands::User.extend_command(self)
+    HammerCLIForeman::AssociatingCommands::ConfigTemplate.extend_command(self)
+    HammerCLIForeman::AssociatingCommands::Organization.extend_command(self)
 
     autoload_subcommands
   end

data/lib/hammer_cli_foreman/media.rb

@@ -11,24 +11,19 @@ module HammerCLIForeman
         field :path, _("Path")
       end
 
-      apipie_options
+      build_options
     end
 
 
     class InfoCommand < HammerCLIForeman::InfoCommand
       output ListCommand.output_definition do
         field :os_family, _("OS Family")
-        field :operatingsystem_ids, _("OS IDs"), Fields::List
-        field :created_at, _("Created at"), Fields::Date
-        field :updated_at, _("Updated at"), Fields::Date
+        HammerCLIForeman::References.operating_systems(self)
+        HammerCLIForeman::References.taxonomies(self)
+        HammerCLIForeman::References.timestamps(self)
       end
 
-      def extend_data(res)
-        res['operatingsystem_ids'] = res['operatingsystems'].map { |e| e["id"] } rescue []
-        res
-      end
-
-      apipie_options
+      build_options
     end
 
 
@@ -37,31 +32,29 @@ module HammerCLIForeman
       success_message _("Installation medium created")
       failure_message _("Could not create the installation medium")
 
-      apipie_options
+      build_options
 
     end
 
 
     class UpdateCommand < HammerCLIForeman::UpdateCommand
-
       success_message _("Installation medium updated")
       failure_message _("Could not update the installation media")
 
-      apipie_options
+      build_options
 
     end
 
 
     class DeleteCommand < HammerCLIForeman::DeleteCommand
-
       success_message _("Installation medium deleted")
       failure_message _("Could not delete the installation media")
 
-      apipie_options
+      build_options
     end
 
 
-    include HammerCLIForeman::AssociatingCommands::OperatingSystem
+    HammerCLIForeman::AssociatingCommands::OperatingSystem.extend_command(self)
 
     autoload_subcommands
   end

data/lib/hammer_cli_foreman/model.rb

@@ -13,19 +13,17 @@ module HammerCLIForeman
         field :hardware_model, _("HW model")
       end
 
-      apipie_options
+      build_options
     end
 
 
     class InfoCommand < HammerCLIForeman::InfoCommand
-
       output ListCommand.output_definition do
         field :info, _("Info")
-        field :created_at, _("Created at"), Fields::Date
-        field :updated_at, _("Updated at"), Fields::Date
+        HammerCLIForeman::References.timestamps(self)
       end
 
-      apipie_options
+      build_options
     end
 
 
@@ -33,14 +31,14 @@ module HammerCLIForeman
       success_message _("Hardware model created")
       failure_message _("Could not create the hardware model")
 
-      apipie_options
+      build_options
     end
 
     class DeleteCommand < HammerCLIForeman::DeleteCommand
       success_message _("Hardware model deleted")
       failure_message _("Could not delete the hardware model")
 
-      apipie_options
+      build_options
     end
 
 
@@ -48,7 +46,7 @@ module HammerCLIForeman
       success_message _("Hardware model updated")
       failure_message _("Could not update the hardware model")
 
-      apipie_options
+      build_options
     end
 
 

data/lib/hammer_cli_foreman/operating_system.rb

@@ -8,52 +8,41 @@ module HammerCLIForeman
 
       output do
         field :id, _("Id")
-        field :_os_name, _("Name"), Fields::OSName
+        field :fullname, _("Full name")
         field :release_name, _("Release name")
         field :family, _("Family")
       end
 
-      def extend_data(os)
-        os["_os_name"] = Hash[os.select { |k, v| ["name", "major", "minor"].include? k }]
-        os
-      end
-
-      apipie_options
+      build_options
     end
 
 
     class InfoCommand < HammerCLIForeman::InfoCommand
-
-      identifiers :id
-
       output ListCommand.output_definition do
-        field :media_names, _("Installation media"), Fields::List
-        field :architecture_names, _("Architectures"), Fields::List
-        field :ptable_names, _("Partition tables"), Fields::List
-        field :config_template_names, _("Config templates"), Fields::List
-        collection :parameters, _("Parameters") do
-          field nil, nil, Fields::KeyValue
+        field :name, _("Name")
+        field :major, _("Major version")
+        field :minor, _("Minor version")
+        collection :ptables, _("Partition tables"), :numbered => false do
+          custom_field Fields::Reference
         end
+        collection :os_default_templates, _("Default templates"), :numbered => false do
+          custom_field Fields::Template, :id_key => :config_template_id, :name_key => :config_template_name
+        end
+        collection :architectures, _("Architectures"), :numbered => false do
+          custom_field Fields::Reference
+        end
+        HammerCLIForeman::References.media(self)
+        HammerCLIForeman::References.config_templates(self)
+        HammerCLIForeman::References.parameters(self)
       end
 
-      #FIXME: remove custom retrieve_data after the api has support for listing names
-      def extend_data(os)
-        os["_os_name"] = Hash[os.select { |k, v| ["name", "major", "minor"].include? k }]
-        os["media_names"] = os["media"].collect{|m| m["name"] } rescue []
-        os["architecture_names"] = os["architectures"].collect{|m| m["name"] } rescue []
-        os["ptable_names"] = os["ptables"].collect{|m| m["name"] } rescue []
-        os["config_template_names"] = os["config_templates"].collect{|m| m["name"] } rescue []
-        os["parameters"] = HammerCLIForeman::Parameter.get_parameters(resource_config, :operatingsystem, os)
-        os
-      end
-
-      apipie_options
+      build_options
     end
 
 
     class CreateCommand < HammerCLIForeman::CreateCommand
 
-      #FIXME: replace with apipie_options when they are added to the api docs
+      #FIXME: replace with build_options when they are added to the api docs
       option "--architecture-ids", "ARCH_IDS", _("set associated architectures"),
         :format => HammerCLI::Options::Normalizers::List.new
       option "--config-template-ids", "CONFIG_TPL_IDS", _("set associated templates"),
@@ -75,13 +64,12 @@ module HammerCLIForeman
         params
       end
 
-      apipie_options
+      build_options
     end
 
 
     class UpdateCommand < HammerCLIForeman::UpdateCommand
-
-      #FIXME: replace with apipie_options when they are added to the api docs
+      #FIXME: replace with build_options when they are added to the api docs
       option "--architecture-ids", "ARCH_IDS", _("set associated architectures"),
         :format => HammerCLI::Options::Normalizers::List.new
       option "--config-template-ids", "CONFIG_TPL_IDS", _("set associated templates"),
@@ -91,8 +79,6 @@ module HammerCLIForeman
       option "--ptable-ids", "PTABLE_IDS", _("set associated partition tables"),
         :format => HammerCLI::Options::Normalizers::List.new
 
-      identifiers :id
-
       success_message _("Operating system updated")
       failure_message _("Could not update the operating system")
 
@@ -105,72 +91,152 @@ module HammerCLIForeman
         params
       end
 
-      apipie_options
+      build_options
     end
 
 
     class DeleteCommand < HammerCLIForeman::DeleteCommand
-
-      identifiers :id
-
       success_message _("Operating system deleted")
       failure_message _("Could not delete the operating system")
 
-      apipie_options
+      build_options
     end
 
-    class SetParameterCommand < HammerCLIForeman::Parameter::SetCommand
-
-      resource :parameters
 
+    class SetParameterCommand < HammerCLIForeman::Parameter::SetCommand
       desc _("Create or update parameter for an operating system.")
 
-      #FIXME: add option --os-label when api supports it
-      option "--os-id", "OS_ID", _("id of the operating system the parameter is being set for")
-
       success_message_for :update, _("Operating system parameter updated")
       success_message_for :create, _("New operating system parameter created")
       failure_message _("Could not set operating system parameter")
 
       def validate_options
         super
-        validator.any(:option_os_id).required
+        validator.any(:option_operatingsystem_id).required
       end
 
-      def base_action_params
-        {
-          "operatingsystem_id" => option_os_id
-        }
-      end
+      build_options
     end
 
 
     class DeleteParameterCommand < HammerCLIForeman::Parameter::DeleteCommand
-
-      resource :parameters
-
       desc _("Delete parameter for an operating system.")
 
-      #FIXME: add option --os-label when api supports it
-      option "--os-id", "OS_ID", _("id of the operating system the parameter is being deleted for")
       success_message _("operating system parameter deleted")
 
       def validate_options
         super
-        validator.any(:option_os_id).required
+        validator.any(:option_operatingsystem_id).required
+      end
+
+      build_options
+    end
+
+
+    class SetOsDefaultTemplate < HammerCLIForeman::Command
+      command_name "set-default-template"
+      resource :os_default_templates
+
+      option "--id", "OS ID", _("operatingsystem id"), :required => true
+      option "--config-template-id", "TPL ID", _("config template id to be set"), :required => true
+
+
+      success_message _("[%{config_template_name}] was set as default %{template_kind_name} template")
+      failure_message _("Could not set the os default template")
+
+      def option_type_name
+        tpl = HammerCLIForeman.collection_to_common_format(
+          HammerCLIForeman.foreman_resource(:config_templates).call(:show, {"id" => option_config_template_id}))
+        tpl[0]["template_kind_name"]
+      end
+
+      def base_action_params
+        {"operatingsystem_id" => option_id}
+      end
+
+      def template_exist?(tpl_kind_name)
+        templates = HammerCLIForeman.collection_to_common_format(resource.call(:index, base_action_params))
+        templates.find { |p| p["template_kind_name"] == tpl_kind_name}
+      end
+
+      def update_default_template(tpl)
+        params = {
+          "id" => tpl["id"],
+          "os_default_template" => {
+            "config_template_id" => option_config_template_id,
+            "template_kind_id" => tpl["template_kind_id"]
+          }
+        }.merge base_action_params
+
+        HammerCLIForeman.record_to_common_format(resource.call(:update, params))
+      end
+
+      def create_default_template(tpl_kind_name)
+        params = {
+          "os_default_template" => {
+            "config_template_id" => option_config_template_id,
+            "template_kind_name" => tpl_kind_name
+          }
+        }.merge base_action_params
+
+        HammerCLIForeman.record_to_common_format(resource.call(:create, params))
+      end
+
+      def execute
+        tpl_kind_name = option_type_name
+        tpl = template_exist? tpl_kind_name
+        if tpl
+          update_default_template tpl
+          print_message(success_message, tpl) if success_message
+        else
+          tpl = create_default_template tpl_kind_name
+          print_message(success_message, tpl) if success_message
+        end
+        HammerCLI::EX_OK
+      end
+
+      build_options :without => [:template_kind_id, :type]
+    end
+
+
+    class DeleteOsDefaultTemplate < HammerCLIForeman::Command
+      command_name "delete-default-template"
+      resource :os_default_templates
+
+      option "--id", "OS ID", _("operatingsystem id"), :required => true
+      option "--type", "TPL TYPE", _("Type of the config template"), :required => true
+
+      success_message _("Default template deleted")
+      failure_message _("Could not delete the default template")
+
+      def execute
+        templates = HammerCLIForeman.collection_to_common_format(resource.call(:index, base_action_params))
+        tpl = templates.find { |p| p["template_kind_name"] == option_type }
+
+        if tpl.nil?
+          raise RuntimeError.new(_("Default template of type #{option_type} not found"))
+        end
+
+        params = {
+          "id" => tpl["id"]
+        }.merge base_action_params
+
+        HammerCLIForeman.record_to_common_format(resource.call(:destroy, params))
+        print_message success_message if success_message
+        HammerCLI::EX_OK
       end
 
       def base_action_params
-        {
-          "operatingsystem_id" => option_os_id
-        }
+        {"operatingsystem_id" => option_id}
       end
+
+      build_options
     end
 
 
-    include HammerCLIForeman::AssociatingCommands::Architecture
-    include HammerCLIForeman::AssociatingCommands::ConfigTemplate
-    include HammerCLIForeman::AssociatingCommands::PartitionTable
+    HammerCLIForeman::AssociatingCommands::Architecture.extend_command(self)
+    HammerCLIForeman::AssociatingCommands::ConfigTemplate.extend_command(self)
+    HammerCLIForeman::AssociatingCommands::PartitionTable.extend_command(self)
 
 
     autoload_subcommands