hammer_cli_katello 1.1.3 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2d5fafd24b632ade65c6e11166ef3e01c7da7cd29c68fd5930a074c0abee1d18
-  data.tar.gz: fb8a68f817c2bb3f6bb693b7b5ed2b9b5bb482c9fe7590afc3a6420a4c7dafac
+  metadata.gz: 0a00c7f5dd067ab0cc8203da1989be46411dc15ed1079442b38bf27e4c1c5eef
+  data.tar.gz: 4088650b9e19e79031e912cfc1f3c49ec11ef448e41c5c97cb41820fe5474bcc
 SHA512:
-  metadata.gz: eaafa419371c904704b7ee5b9c776e829d35ad6560d55cc7fa118c5a076dda9617321e2838a1608dfd14067c8a6ea1aecd91d61e4cd2d0e1b5a04841322f8dd1
-  data.tar.gz: 31ac0a523c9d27c1e84de086dfc79e2868d05a235065b513a4797539f485a988f96c2cbc829192fcb0733f0f3241e9460311ebc1cb50eb84b7b1019cb339ceb5
+  metadata.gz: 0b638de29d5f9ebf39fa1962ec41d316f23579c64f9b0bd387bc3bd0323d447368fb9dacf283ea8e10d898e1d2bd4e6956c99f435080f38c835092c154e85f03
+  data.tar.gz: 8c59ab30460634b1098d74933284052ee98492897a04d17358166597fb2d53e2412e521228486b5df37c8d8cb2f254565eb79ccb3cc29e5dfd8cfc9b42b4e399

data/lib/hammer_cli_katello/command_extensions/content_source.rb

@@ -0,0 +1,32 @@
+module HammerCLIKatello
+  module CommandExtensions
+    class ContentSource < HammerCLI::CommandExtensions
+      option_family associate: 'content_source' do
+        child '--content-source', 'CONTENT_SOURCE_NAME', _('Content Source name'),
+              attribute_name: :option_content_source,
+              referenced_resource: :smart_proxy
+      end
+
+      request_params do |params, cmd_obj|
+        begin
+          resource_name = cmd_obj.resource.singular_name
+          if cmd_obj.option_content_source && !cmd_obj.option_content_source_id
+            resource_hash = if resource_name == 'hostgroup'
+                              params[resource_name]
+                            else
+                              params[resource_name]['content_facet_attributes']
+                            end
+
+            proxy_options = {
+              HammerCLI.option_accessor_name('name') => cmd_obj.option_content_source
+            }
+            resource_hash['content_source_id'] = cmd_obj.resolver.smart_proxy_id(proxy_options)
+          end
+        rescue HammerCLIForeman::ResolverError => e
+          e.message.gsub!('smart_proxy', _('Content Source'))
+          raise e
+        end
+      end
+    end
+  end
+end

data/lib/hammer_cli_katello/command_extensions/kickstart_repository.rb

@@ -0,0 +1,50 @@
+module HammerCLIKatello
+  module CommandExtensions
+    class KickstartRepository < HammerCLI::CommandExtensions
+      option_family associate: 'kickstart_repository' do
+        child '--kickstart-repository', 'REPOSITORY_NAME', _('Kickstart repository name'),
+              attribute_name: :option_kickstart_repository
+      end
+
+      request_params do |params, cmd_obj|
+        resource_name = cmd_obj.resource.singular_name
+        if cmd_obj.option_kickstart_repository && !cmd_obj.option_kickstart_repository_id
+          resource_hash = if resource_name == 'hostgroup'
+                            params[resource_name]
+                          else
+                            params[resource_name]['content_facet_attributes']
+                          end
+
+          resource_hash ||= {}
+
+          env_id = resource_hash['lifecycle_environment_id']
+          cv_id = resource_hash['content_view_id']
+
+          raise _('Please provide --lifecycle-environment-id') unless env_id
+
+          raise _('Please provide --content-view-id') unless cv_id
+          # rubocop:disable LineLength
+          resource_hash['kickstart_repository_id'] = HammerCLIKatello::CommandExtensions::KickstartRepository.fetch_repo_id(
+            cv_id, env_id, cmd_obj.option_kickstart_repository
+          )
+          # rubocop:enable LineLength
+        end
+      end
+
+      def self.fetch_repo_id(cv_id, env_id, repo_name)
+        repo_resource = HammerCLIForeman.foreman_resource(:repositories)
+        index_options = {
+          content_view_id: cv_id,
+          environment_id: env_id,
+          name: repo_name
+        }
+        repos = repo_resource.call(:index, index_options)['results']
+        if repos.empty?
+          raise _("No such repository with name %{name}, in lifecycle environment"\
+                    " %{environment_id} and content view %{content_view_id}" % index_options)
+        end
+        repos.first['id']
+      end
+    end
+  end
+end

data/lib/hammer_cli_katello/command_extensions/lifecycle_environment.rb

@@ -13,6 +13,18 @@ module HammerCLIKatello
                attribute_name: :option_environment_id
       end
 
+      # Add explicitly defined options since option builder won't be
+      # able to create options automatically in case there is missing resource
+      # in API docs or if the resource name is different
+      # (e.g. environment instead of lifecycle_environment)
+      # This can happen if API docs contain a param which cannot be mapped
+      # via param_name to resource_name mapping
+      option_family associate: 'lifecycle_environment' do
+        child '--lifecycle-environment', 'ENVIRONMENT_NAME',
+              _('Lifecycle environment name to search by'),
+              attribute_name: :option_environment_name
+      end
+
       option_sources do |sources, command|
         sources.find_by_name('IdResolution').insert_relative(
           :after,

data/lib/hammer_cli_katello/command_extensions.rb

@@ -3,3 +3,5 @@
 require 'hammer_cli_katello/command_extensions/lifecycle_environment'
 require 'hammer_cli_katello/command_extensions/lifecycle_environments'
 require 'hammer_cli_katello/command_extensions/ping'
+require 'hammer_cli_katello/command_extensions/content_source'
+require 'hammer_cli_katello/command_extensions/kickstart_repository'

data/lib/hammer_cli_katello/commands.rb

@@ -36,7 +36,7 @@ module HammerCLIKatello
       end
 
       def resource_name_mapping
-        HammerCLIKatello::RESOURCE_NAME_MAPPING.dup
+        super.merge(HammerCLIKatello::RESOURCE_NAME_MAPPING.dup)
       end
     end
   end

data/lib/hammer_cli_katello/content_export_helper.rb

@@ -162,14 +162,18 @@ module HammerCLIKatello
                  :required => false
 
       base.build_options do |o|
-        o.expand(:all).including(:content_views, :organizations, :environments)
+        o.expand(:all).including(:content_views, :organizations, :lifecycle_environments)
         o.without(:environment_ids, :environment_id)
       end
 
       base.validate_options do
         unless option(:option_id).exist?
           any(:option_id, :option_content_view_name, :option_content_view_id).required
-          any(:option_version, :option_environment_id, :option_environment_name).required
+          any(
+            :option_version,
+            :option_lifecycle_environment_id,
+            :option_lifecycle_environment_name
+          ).required
           unless option(:option_content_view_id).exist?
             any(:option_organization_id, :option_organization_name, \
                                 :option_organization_label).required

data/lib/hammer_cli_katello/content_units.rb

@@ -0,0 +1,66 @@
+module HammerCLIKatello
+  class ContentUnitsCommand < HammerCLIKatello::Command
+    resource :generic_content_units
+
+    class ListCommand < HammerCLIKatello::ListCommand
+      extend RepositoryScopedToProduct
+
+      validate_repo_name_requires_product_options(:option_repository_name)
+
+      output do
+        field :id, _("Id")
+        field :name, _("Name")
+        field :version, _("Version")
+        field :filename, _("Filename"), Fields::Field, hide_blank: true
+      end
+
+      validate_options :before, 'IdResolution' do
+        organization_options = [:option_organization_id, :option_organization_name,
+                                :option_organization_label]
+        if any(:option_product_name, :option_content_view_name).exist?
+          any(*organization_options).required
+        end
+      end
+
+      build_options do |o|
+        o.expand.including(:products, :organizations, :content_views)
+      end
+    end
+
+    class InfoCommand < HammerCLIKatello::InfoCommand
+      output do
+        field :id, _("Id")
+        field :name, _("Name")
+        field :version, _("Version")
+        field :filename, _("Filename"), Fields::Field, hide_blank: true
+      end
+
+      validate_options :before, 'IdResolution' do
+        organization_options = [:option_organization_id, :option_organization_name,
+                                :option_organization_label]
+        product_options = [:option_product_id, :option_product_name]
+        repository_options = [:option_repository_id, :option_repository_name]
+        content_view_version_options = [:option_content_view_version_id,
+                                        :option_content_view_version_version]
+
+        if option(:option_product_name).exist?
+          any(*organization_options).required
+        end
+
+        if option(:option_repository_name).exist?
+          any(*product_options).required
+        end
+
+        if option(:option_name).exist?
+          any(*(repository_options + content_view_version_options)).required
+        end
+      end
+
+      build_options do |o|
+        o.expand.including(:products, :organizations, :content_views, :content_view_versions)
+      end
+    end
+
+    autoload_subcommands
+  end
+end

data/lib/hammer_cli_katello/content_view.rb

@@ -197,6 +197,17 @@ module HammerCLIKatello
       failure_message _("Could not publish the content view")
 
       build_options
+
+      option_family associate: 'lifecycle_environment' do
+        child '--lifecycle-environments', 'LIFECYCLE_ENVIRONMENT_NAMES',
+              _('Names for Lifecycle Environment'),
+              format: HammerCLI::Options::Normalizers::List.new,
+              attribute_name: :option_environment_names
+      end
+
+      extend_with(
+        HammerCLIKatello::CommandExtensions::LifecycleEnvironments.new(only: :option_sources)
+      )
     end
 
     class RemoveFromEnvironmentCommand < HammerCLIKatello::SingleResourceCommand

data/lib/hammer_cli_katello/content_view_component.rb

@@ -84,16 +84,16 @@ module HammerCLIKatello
       include OrganizationOptions
       include CompositeContentViewNameResolvable
       output do
-        field :id, _("Id")
+        field :content_view_id, _("Content View Id")
         field :content_view_name, _("Name")
         field :version, _("Version")
+        field :id, _("Component Id")
         field :current_version, _("Current Version")
         field :version_id, _("Version Id")
       end
 
       def extend_data(mod)
         if mod['latest']
-          mod['content_view_name'] = mod["content_view"]["name"]
           mod['version'] = _("Latest")
           if mod['content_view_version']
             mod['current_version'] = mod['content_view_version']['version']
@@ -102,10 +102,11 @@ module HammerCLIKatello
             mod['current_version'] = _("No Published Version")
           end
         else
-          mod['content_view_name'] = mod["content_view"]["name"]
           mod['version'] = mod['content_view_version']['version']
           mod['version_id'] = mod['content_view_version']['id']
         end
+        mod['content_view_name'] = mod["content_view"]["name"]
+        mod['content_view_id'] = mod["content_view"]["id"]
         mod
       end
 

data/lib/hammer_cli_katello/content_view_version.rb

@@ -60,7 +60,7 @@ module HammerCLIKatello
       end
 
       build_options do |o|
-        o.expand(:all).including(:environments, :content_views, :organizations)
+        o.expand(:all).including(:lifecycle_environments, :content_views, :organizations)
       end
 
       extend_with(HammerCLIKatello::CommandExtensions::LifecycleEnvironment.new)
@@ -143,7 +143,7 @@ module HammerCLIKatello
       failure_message _("Could not delete the content view")
 
       build_options do |o|
-        o.expand(:all).including(:environments, :content_views, :organizations)
+        o.expand(:all).including(:lifecycle_environments, :content_views, :organizations)
       end
 
       extend_with(HammerCLIKatello::CommandExtensions::LifecycleEnvironment.new)
@@ -157,7 +157,7 @@ module HammerCLIKatello
       failure_message _("Could not update the content view version")
 
       build_options do |o|
-        o.expand(:all).including(:environments, :content_views, :organizations)
+        o.expand(:all).including(:lifecycle_environments, :content_views, :organizations)
       end
 
       extend_with(HammerCLIKatello::CommandExtensions::LifecycleEnvironment.new)

data/lib/hammer_cli_katello/foreman_search_options_creators.rb

@@ -48,18 +48,6 @@ module HammerCLIKatello
       create_search_options_without_katello_api(options, api.resource(:ptables), mode)
     end
 
-    def create_puppet_ca_proxies_search_options(options, mode = nil)
-      create_search_options_without_katello_api(options, api.resource(:puppet_ca_proxies), mode)
-    end
-
-    def create_puppet_proxies_search_options(options, mode = nil)
-      create_search_options_without_katello_api(options, api.resource(:puppet_proxies), mode)
-    end
-
-    def create_puppetclasses_search_options(options, mode = nil)
-      create_search_options_without_katello_api(options, api.resource(:puppetclasses), mode)
-    end
-
     def create_subnets_search_options(options, mode = nil)
       create_search_options_without_katello_api(options, api.resource(:subnets), mode)
     end

data/lib/hammer_cli_katello/host_collection.rb

@@ -50,6 +50,12 @@ module HammerCLIKatello
       success_message _("Host collection created.")
       failure_message _("Could not create the host collection")
 
+      option "--unlimited-hosts", :flag, "Set hosts max to unlimited"
+
+      validate_options :before, 'IdResolution' do
+        all(:option_unlimited_hosts, :option_max_hosts).rejected
+      end
+
       build_options
     end
 

data/lib/hammer_cli_katello/host_errata.rb

@@ -11,7 +11,7 @@ module HammerCLIKatello
       success_message _("Errata is being applied with task %{id}.")
       failure_message _("Could not apply errata")
 
-      build_options
+      build_options(without: ['ids'])
 
       def execute
         warn "This command uses katello agent and will be removed in favor of remote execution " \
@@ -46,10 +46,9 @@ module HammerCLIKatello
     end
 
     class RecalculateCommand < HammerCLIKatello::SingleResourceCommand
-      include HammerCLIForemanTasks::Async
       resource :host_errata, :applicability
       command_name "recalculate"
-      success_message _("Errata recalculated with task %{id}.")
+      success_message _("Errata recalculation started.")
       failure_message _("Could not recalculate errata")
 
       build_options

data/lib/hammer_cli_katello/host_extensions.rb

@@ -3,23 +3,25 @@ require 'hammer_cli_katello/host_errata'
 require 'hammer_cli_katello/host_subscription'
 require 'hammer_cli_katello/host_package'
 require 'hammer_cli_katello/host_package_group'
-require 'hammer_cli_katello/host_kickstart_repository_options'
-require 'hammer_cli_katello/host_content_source_options'
 require 'hammer_cli_katello/host_traces'
 
 module HammerCLIKatello
   module HostExtensions
     ::HammerCLIForeman::Host::CreateCommand.instance_eval do
       include HammerCLIKatello::ResolverCommons
-      include HammerCLIKatello::HostContentSourceOptions
-      include ::HammerCLIKatello::HostKickstartRepositoryOptions
     end
+    ::HammerCLIForeman::Host::CreateCommand.extend_with(
+      HammerCLIKatello::CommandExtensions::ContentSource.new,
+      HammerCLIKatello::CommandExtensions::KickstartRepository.new
+    )
 
     ::HammerCLIForeman::Host::UpdateCommand.instance_eval do
       include HammerCLIKatello::ResolverCommons
-      include HammerCLIKatello::HostContentSourceOptions
-      include ::HammerCLIKatello::HostKickstartRepositoryOptions
     end
+    ::HammerCLIForeman::Host::UpdateCommand.extend_with(
+      HammerCLIKatello::CommandExtensions::ContentSource.new,
+      HammerCLIKatello::CommandExtensions::KickstartRepository.new
+    )
 
     ::HammerCLIForeman::Host::ListCommand.instance_eval do
       output do

data/lib/hammer_cli_katello/hostgroup_extensions.rb

@@ -1,16 +1,19 @@
 require 'hammer_cli_foreman/hostgroup'
-require 'hammer_cli_katello/host_kickstart_repository_options'
-require 'hammer_cli_katello/host_content_source_options'
 
 module HammerCLIKatello
   module QueryOrganizationOptions
     def self.included(base)
-      base.option '--query-organization-id', 'ORGANIZATION_ID',
-        _('Organization ID to search by'), attribute_name: :option_organization_id
-      base.option '--query-organization', 'ORGANIZATION_NAME',
-        _('Organization name to search by'), attribute_name: :option_organization_name
-      base.option '--query-organization-label', 'ORGANIZATION_LABEL',
-        _('Organization label to search by'), attribute_name: :option_organization_label
+      base.option_family do
+        parent '--query-organization-id', 'ORGANIZATION_ID',
+               _('Organization ID to search by'),
+               attribute_name: :option_organization_id
+        child '--query-organization', 'ORGANIZATION_NAME',
+              _('Organization name to search by'),
+              attribute_name: :option_organization_name
+        child '--query-organization-label', 'ORGANIZATION_LABEL',
+              _('Organization label to search by'),
+              attribute_name: :option_organization_label
+      end
 
       base.validate_options :before, 'IdResolution' do
         organization_options = [
@@ -30,17 +33,21 @@ module HammerCLIKatello
       include HammerCLIKatello::ResolverCommons
       include HammerCLIKatello::ContentViewNameResolvable
       include HammerCLIKatello::QueryOrganizationOptions
-      include HammerCLIKatello::HostContentSourceOptions
-      include HammerCLIKatello::HostKickstartRepositoryOptions
     end
+    ::HammerCLIForeman::Hostgroup::CreateCommand.extend_with(
+      HammerCLIKatello::CommandExtensions::ContentSource.new,
+      HammerCLIKatello::CommandExtensions::KickstartRepository.new
+    )
 
     ::HammerCLIForeman::Hostgroup::UpdateCommand.instance_eval do
       include HammerCLIKatello::ResolverCommons
       include HammerCLIKatello::ContentViewNameResolvable
       include HammerCLIKatello::QueryOrganizationOptions
-      include HammerCLIKatello::HostContentSourceOptions
-      include HammerCLIKatello::HostKickstartRepositoryOptions
     end
+    ::HammerCLIForeman::Hostgroup::UpdateCommand.extend_with(
+      HammerCLIKatello::CommandExtensions::ContentSource.new,
+      HammerCLIKatello::CommandExtensions::KickstartRepository.new
+    )
 
     ::HammerCLIForeman::Hostgroup::InfoCommand.instance_eval do
       output do

data/lib/hammer_cli_katello/id_resolver.rb

@@ -74,12 +74,15 @@ module HammerCLIKatello
         return options['option_lifecycle_environment_ids']
       end
 
-      key_names = HammerCLI.option_accessor_name 'lifecycle_environment_names'
+      names = options[HammerCLI.option_accessor_name('lifecycle_environment_names')] ||
+              options[HammerCLI.option_accessor_name('environment_names')] ||
+              options[HammerCLI.option_accessor_name('names')]
       key_organization_id = HammerCLI.option_accessor_name 'organization_id'
       options[key_organization_id] ||= organization_id(scoped_options('organization', options))
 
+      options['option_lifecycle_environment_names'] = names
       find_resources(:lifecycle_environments, options)
-        .select { |repo| options[key_names].include? repo['name'] }.map { |repo| repo['id'] }
+        .select { |repo| names.include? repo['name'] }.map { |repo| repo['id'] }
     end
 
     def repository_id(options)

data/lib/hammer_cli_katello/option_sources/lifecycle_environment_params.rb

@@ -6,15 +6,52 @@ module HammerCLIKatello
       end
 
       def get_options(_defined_options, result)
+        legacy_option_id(result)
+        legacy_option_ids(result)
+        ensure_option_id(result)
+        ensure_option_ids(result)
+        result
+      end
+
+      private
+
+      def lifecycle_environment_resource_name
+        (HammerCLIForeman.param_to_resource('environment') ||
+          HammerCLIForeman.param_to_resource('lifecycle_environment')).singular_name
+      end
+
+      def legacy_option_id(result)
         if result['option_environment_name'] && result['option_environment_id'].nil?
           result['option_environment_id'] = @command.resolver.lifecycle_environment_id(
-            @command.resolver.scoped_options('environment', result, :single))
+            @command.resolver.scoped_options(lifecycle_environment_resource_name, result, :single))
         end
+      end
+
+      def legacy_option_ids(result)
         if result['option_environment_names'] && result['option_environment_ids'].nil?
           result['option_environment_ids'] = @command.resolver.lifecycle_environment_ids(
-            @command.resolver.scoped_options('environment', result, :multi))
+            @command.resolver.scoped_options(lifecycle_environment_resource_name, result, :multi))
+        end
+      end
+
+      def ensure_option_id(result)
+        if result['option_environment_id'].nil? && result['option_lifecycle_environment_name']
+          id = @command.resolver.lifecycle_environment_id(
+            @command.resolver.scoped_options(lifecycle_environment_resource_name, result, :single)
+          )
+          result['option_environment_id'] = id
+          result['option_lifecycle_environment_id'] = id
+        end
+      end
+
+      def ensure_option_ids(result)
+        if result['option_environment_ids'].nil? && result['option_lifecycle_environment_names']
+          ids = @command.resolver.lifecycle_environment_ids(
+            @command.resolver.scoped_options(lifecycle_environment_resource_name, result, :multi)
+          )
+          result['option_environment_ids'] = ids
+          result['option_lifecycle_environment_ids'] = ids
         end
-        result
       end
     end
   end

data/lib/hammer_cli_katello/organization.rb

@@ -20,9 +20,17 @@ module HammerCLIKatello
       output do
         field :label, _("Label")
         field :description, _("Description")
-        field :redhat_repository_url, _("Red Hat Repository URL")
         field :simple_content_access_label, _("Simple Content Access")
         field :service_levels, _("Service Levels"), Fields::List
+        from :cdn_configuration do
+          label "CDN configuration", hide_blank: true do
+            field :url, _("URL"), Fields::Field, hide_blank: true
+            field :upstream_organization_label, _("Upstream Organization"),
+                                               Fields::Field, hide_blank: true
+            field :username, _("Username"), Fields::Field, hide_blank: true
+            field :ssl_ca_credential_id, _("SSL CA Credential ID"), Fields::Field, hide_blank: true
+          end
+        end
       end
 
       def extend_data(data)
@@ -44,7 +52,9 @@ module HammerCLIKatello
       success_message _("Organization updated.")
       failure_message _("Could not update the organization")
 
-      build_options
+      build_options do |o|
+        o.expand(:all).except(:environments)
+      end
     end
 
     class CreateCommand < HammerCLIForeman::Organization::CreateCommand
@@ -54,7 +64,9 @@ module HammerCLIKatello
       success_message _("Organization created.")
       failure_message _("Could not create the organization")
 
-      build_options
+      build_options do |o|
+        o.expand(:all).except(:environments)
+      end
     end
 
     class DeleteCommand < HammerCLIForeman::Organization::DeleteCommand
@@ -68,6 +80,19 @@ module HammerCLIKatello
       build_options
     end
 
+    class ConfigureCdnCommand < HammerCLIKatello::SingleResourceCommand
+      include HammerCLIKatello::ResolverCommons
+      resource :organizations, :cdn_configuration
+      command_name "configure-cdn"
+      success_message _("Updated CDN configuration.")
+      failure_message _("Could not update CDN configuration.")
+
+      build_options do |o|
+        o.expand(:all).except(:organizations, :locations)
+        o.without(:organization_id, :location_id)
+      end
+    end
+
     autoload_subcommands
   end
 end

data/lib/hammer_cli_katello/package.rb

@@ -3,7 +3,6 @@ module HammerCLIKatello
     resource :packages
 
     class ListCommand < HammerCLIKatello::ListCommand
-      extend_with(HammerCLIKatello::CommandExtensions::LifecycleEnvironment.new)
       include LifecycleEnvironmentNameMapping
 
       output do
@@ -40,6 +39,8 @@ module HammerCLIKatello
       build_options do |o|
         o.expand.including(:products, :content_views)
       end
+
+      extend_with(HammerCLIKatello::CommandExtensions::LifecycleEnvironment.new)
     end
 
     class InfoCommand < HammerCLIKatello::InfoCommand

data/lib/hammer_cli_katello/ping.rb

@@ -51,6 +51,13 @@ module HammerCLIKatello
           end
         end
 
+        label "pulp3_content", :hide_blank => true do
+          from "pulp3_content" do
+            field "status", _("Status"), Fields::Field, :hide_blank => true
+            field "_response", _("Server Response"), Fields::Field, :hide_blank => true
+          end
+        end
+
         label "foreman_tasks" do
           from "foreman_tasks" do
             field "status", _("Status")