katello 4.0.3 → 4.1.0.rc1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bce7f44a7f2b8e7a0eda1ca43fc7835ff63a4e13f37dc989b34e5902a1aed53a
-  data.tar.gz: 9347b0be58ab615b998c47b00ec3db414e21a0802072c1abd1b0d09ebe45b46d
+  metadata.gz: cc20921c082e7725e89028befc8b8e9469b5088304795b5cbeb4cfc9bc343866
+  data.tar.gz: aee4945d14f0e66a96d516eb3e47ee0e93fc02d9bb7384e28f5d360646a73a88
 SHA512:
-  metadata.gz: 302db8a69576d6ede5e7b76a2a68d1ec255490bfe6d08748f503b43b3b57d2930ab872e7cb5f2bc26330fd08781650eab35e05bf1adab85505090f471886cc32
-  data.tar.gz: ca2ec5c259ed700896a7509da48bf7ccb814e273e9391210d96473c7ae37563904a1291062e2b04adb1afe1bbf8f50394d4d05edc199a5843148612e8a5c57f3
+  metadata.gz: b868b400fb3a0be4e7db0755700b45b5388e4c433a447dfe335c7bc156f40b5dc557fce7553aa79fa2eaa61bb83e0ceae3c27cbb7deb2458efb15f7ac890df22
+  data.tar.gz: 550c55ae8bec58857cfa5b75d33579f9c9de56013a58f273487fdccdf9378890f2d47ac257fdf8b2b29a89fd648cdd8e668725358b169dcf35f0277d7758e0a3

data/app/assets/javascripts/katello/hosts/host_and_hostgroup_edit.js

@@ -9,12 +9,6 @@ $(document).on('ContentLoad', function(){
     $("#hostgroup_lifecycle_environment_id").change(KT.hosts.environmentChanged);
     $("#host_lifecycle_environment_id").change(KT.hosts.environmentChanged);
 
-    $("#hostgroup_content_view_id").change(KT.hosts.contentViewSelected);
-    $("#host_content_view_id").change(KT.hosts.contentViewSelected);
-    $("#reset_puppet_environment").click(function() {
-        KT.hosts.getPuppetEnvironmentSelect().data('content_puppet_match', 'true');
-        KT.hosts.setDefaultPuppetEnvironment(KT.hosts.getSelectedContentView(), KT.hosts.getSelectedEnvironment());
-    });
     KT.hosts.update_media_enablement();
     KT.hosts.set_media_selection_bindings();
 });
@@ -68,33 +62,6 @@ KT.hosts.signalContentViewFetch = function(fetching) {
     }
 };
 
-
-KT.hosts.contentViewSelected = function() {
-    if (KT.hosts.getPuppetEnvironmentSelect().data('content_puppet_match')) {
-        KT.hosts.setDefaultPuppetEnvironment(KT.hosts.getSelectedContentView(), KT.hosts.getSelectedEnvironment());
-    }
-};
-
-KT.hosts.setDefaultPuppetEnvironment = function(view_id, env_id) {
-    if (view_id && env_id) {
-        $.get('/hosts/puppet_environment_for_content_view', {content_view_id: view_id, lifecycle_environment_id: env_id}, function (data) {
-            var select = KT.hosts.getPuppetEnvironmentSelect();
-            if (data !== null) {
-                select.val(data.id);
-                select.trigger('change');
-            }
-        })
-    }
-};
-
-KT.hosts.getPuppetEnvironmentSelect = function() {
-    var select = $("#host_environment_id").first();
-    if(select.length === 0) {
-        select = $("#hostgroup_environment_id").first();
-    }
-    return select;
-};
-
 KT.hosts.getContentViewSelect2 = function() {
     var select = $("#s2id_host_content_view_id").first();
     if(select.length === 0) {

data/app/assets/stylesheets/katello/contents.scss

@@ -322,7 +322,7 @@ $container_width: ($static_width/2) - 36;
         position: relative;
         text-align: left;
         bottom: 18px;
-        left: 30px;
+        left: 10px;
       }
       .progress {
         display: inline-block;

data/app/controllers/katello/api/registry/registry_proxies_controller.rb

@@ -93,15 +93,6 @@ module Katello
       true
     end
 
-    # Reduce visible repos to include lifecycle env permissions
-    # http://projects.theforeman.org/issues/22914
-    # Also include repositories in lifecycle environments with registry_unauthenticated_pull=true
-    def readable_repositories
-      table_name = Repository.table_name
-      in_unauth_environments = Repository.joins(:environment).where("#{Katello::KTEnvironment.table_name}.registry_unauthenticated_pull" => true).select(:id)
-      Repository.readable.or(Repository.joins(:root).where("#{table_name}.id in (?)", in_unauth_environments))
-    end
-
     def find_readable_repository
       return nil unless params[:repository]
       repository = Repository.docker_type.find_by(container_repository_name: params[:repository])
@@ -113,7 +104,10 @@ module Katello
 
     def require_user_authorization?(repository = @repository)
       !(params['action'] == 'token' && params['scope'].blank? && params['account'].blank?) &&
-        (!repository || (!repository.environment.registry_unauthenticated_pull && !ssl_client_authorized?(repository.organization.label)))
+        (!repository ||
+          (!repository.archive? &&
+            !repository.environment.registry_unauthenticated_pull &&
+            !ssl_client_authorized?(repository.organization.label)))
     end
 
     def ssl_client_authorized?(org_label)
@@ -483,7 +477,7 @@ module Katello
       return nil unless scope
 
       scopes = scope.split(':')
-      scopes[2] == 'pull' ? Repository.docker_type.find_by_container_repository_name(scopes[1]) : nil
+      scopes[2] == 'pull' ? Repository.docker_type.non_archived.find_by_container_repository_name(scopes[1]) : nil
     end
 
     def disable_strong_params

data/app/controllers/katello/api/rhsm/candlepin_proxies_controller.rb

@@ -242,15 +242,8 @@ module Katello
     #description N_("This service is available for unauthenticated users")
     def server_status
       candlepin_response = Resources::Candlepin::CandlepinPing.ping
-      status = { :managerCapabilities => candlepin_response['managerCapabilities'] + ["combined_reporting"],
-                 :result => candlepin_response['result'],
-                 :rulesSource => candlepin_response['rulesSource'],
-                 :rulesVersion => candlepin_response['rulesVersion'],
-                 :standalone => candlepin_response['standalone'],
-                 :timeUTC => Time.now.getutc,
-                 :version => Katello::VERSION }
-
-      render :json => status
+      candlepin_response[:managerCapabilities] << 'combined_reporting'
+      render :json => candlepin_response
     end
 
     #api :PUT, "/consumers/:id", N_("Update consumer information")

data/app/controllers/katello/api/v2/activation_keys_controller.rb

@@ -6,7 +6,7 @@ module Katello
     before_action :find_environment, :only => [:index, :create, :update]
     before_action :find_optional_organization, :only => [:index, :create, :show]
     before_action :find_content_view, :only => [:index]
-    before_action :find_authorized_katello_resource, :only => [:show, :update, :destroy, :available_releases, :product_content,
+    before_action :find_authorized_katello_resource, :only => [:show, :update, :destroy, :available_releases,
                                                                :available_host_collections, :add_host_collections, :remove_host_collections,
                                                                :content_override, :add_subscriptions, :remove_subscriptions,
                                                                :subscriptions]
@@ -141,29 +141,6 @@ module Katello
       respond_for_index :collection => response
     end
 
-    api :GET, "/activation_keys/:id/product_content", N_("Show content available for an activation key")
-    param :id, String, :desc => N_("ID of the activation key"), :required => true
-    param :content_access_mode_all, :bool, :desc => N_("Get all content available, not just that provided by subscriptions")
-    param :content_access_mode_env, :bool, :desc => N_("Limit content to just that available in the activation key's content view version")
-    def product_content
-      content_access_mode_all = ::Foreman::Cast.to_bool(params[:content_access_mode_all])
-      content_access_mode_env = ::Foreman::Cast.to_bool(params[:content_access_mode_env])
-
-      content_finder = ProductContentFinder.new(
-          :match_subscription => !content_access_mode_all,
-          :match_environment => content_access_mode_env,
-          :consumable => @activation_key
-      )
-
-      content = content_finder.presenter_with_overrides(@activation_key.content_overrides)
-      response = {
-        :results => content,
-        :total => content.size,
-        :subtotal => content.size
-      }
-      respond_for_index :collection => response
-    end
-
     api :POST, "/activation_keys/:id/host_collections"
     param :id, :number, :desc => N_("ID of the activation key"), :required => true
     param :host_collection_ids, Array, :required => true, :desc => N_("List of host collection IDs to associate with activation key")
@@ -248,6 +225,15 @@ module Katello
       respond_for_show(:resource => @activation_key)
     end
 
+    api :GET, "/activation_keys/:id/product_content", N_("Show content available for an activation key")
+    param :id, String, :desc => N_("ID of the activation key"), :required => true
+    param :content_access_mode_all, :bool, :desc => N_("Get all content available, not just that provided by subscriptions")
+    param :content_access_mode_env, :bool, :desc => N_("Limit content to just that available in the activation key's content view version")
+    def product_content
+      # note this is just there as a place holder for apipie.
+      # The routing would automatically redirect it to repository_sets#index
+    end
+
     def index_relation
       activation_keys = ActivationKey.readable
       activation_keys = activation_keys.where(:name => params[:name]) if params[:name]

data/app/controllers/katello/api/v2/api_controller.rb

@@ -39,7 +39,18 @@ module Katello
     end
 
     def deprecate_katello_agent
-      ::Foreman::Deprecation.api_deprecation_warning("Katello-agent is deprecated and will be removed in a future release.")
+      check_katello_agent_not_disabled
+      ::Foreman::Deprecation.api_deprecation_warning("This action uses katello-agent, which is deprecated and will be removed in #{katello_agent_removal_release}.")
+    end
+
+    def check_katello_agent_not_disabled
+      unless ::Katello.with_katello_agent?
+        fail HttpErrors::BadRequest, _("This action uses katello-agent, which is currently disabled. Use remote execution instead.")
+      end
+    end
+
+    def katello_agent_removal_release
+      self.class.katello_agent_removal_release
     end
 
     def full_result_response(collection)
@@ -75,7 +86,8 @@ module Katello
       end
 
       total = scoped_search_total(query, group)
-
+      # If you get an ArgumentError here (wrong number of arguments given)
+      # it's probably because query is an array and not an ActiveRecord::Relation
       query = query.select(:id) if query.respond_to?(:select)
       query = resource.search_for(*search_options).where("#{resource.table_name}.id" => query)
 
@@ -207,7 +219,7 @@ module Katello
     end
 
     def find_host_with_subscriptions(id, permission)
-      @host = resource_finder(::Host::Managed.authorized(permission, ::Host::Managed), id)
+      @host = resource_finder(::Host::Managed.authorized(permission, ::Host::Managed), id&.to_i)
       fail HttpErrors::BadRequest, _("Host has not been registered with subscription-manager") if @host.subscription_facet.nil?
     end
 

data/app/controllers/katello/api/v2/content_credentials_controller.rb

@@ -49,7 +49,7 @@ module Katello
 
       content = nil
       if filepath
-        content = File.open(filepath, 'rb') { |file| file.read }
+        content = File.read(filepath, mode: 'rb')
       else
         content = params[:content]
       end

data/app/controllers/katello/api/v2/content_export_incrementals_controller.rb

@@ -1,6 +1,5 @@
 module Katello
   class Api::V2::ContentExportIncrementalsController < Api::V2::ApiController
-    before_action :fail_if_not_pulp3, :only => [:version, :library]
     before_action :find_exportable_organization, :only => [:library]
     before_action :find_exportable_content_view_version, :only => [:version]
     before_action :find_library_export_view, :only => [:library]
@@ -16,7 +15,7 @@ module Katello
     param :fail_on_missing_content, :bool, :desc => N_("Fails if any of the repositories belonging to this version"\
                                                          " are unexportable. False by default."), :required => false
     def version
-      tasks = async_task(::Actions::Pulp3::Orchestration::ContentViewVersion::Export,
+      tasks = async_task(Actions::Katello::ContentViewVersion::Export,
                           content_view_version: @version,
                           destination_server: params[:destination_server],
                           chunk_size: params[:chunk_size_mb],
@@ -88,11 +87,5 @@ module Katello
         throw_resource_not_found(name: 'organization', id: params[:organization_id])
       end
     end
-
-    def fail_if_not_pulp3
-      unless SmartProxy.pulp_primary.pulp3_repository_type_support?(Katello::Repository::YUM_TYPE)
-        fail HttpErrors::BadRequest, _("Invalid usage for Pulp 2 repositories. Use export for Yum repositories")
-      end
-    end
   end
 end

data/app/controllers/katello/api/v2/content_exports_controller.rb

@@ -1,6 +1,5 @@
 module Katello
   class Api::V2::ContentExportsController < Api::V2::ApiController
-    before_action :fail_if_not_pulp3, :only => [:version, :library]
     before_action :find_exportable_organization, :only => [:library]
     before_action :find_exportable_content_view_version, :only => [:version]
 
@@ -27,12 +26,6 @@ module Katello
                                        :collection => scoped_search(history, 'id', 'asc', resource_class: ContentViewVersionExportHistory))
     end
 
-    api :GET, "/content_exports/api_status", N_("true if the export api is pulp3 ready and usable. This API is intended for use by hammer-cli only.")
-    def api_status
-      ::Foreman::Deprecation.api_deprecation_warning("/content_exports/api_status is being deprecated and will be removed in Katello 4.1.")
-      render json: { api_usable: SmartProxy.pulp_primary.pulp3_repository_type_support?(Katello::Repository::YUM_TYPE) }, status: :ok
-    end
-
     api :POST, "/content_exports/version", N_("Performs a full-export of a content view version.")
     param :id, :number, :desc => N_("Content view version identifier"), :required => true
     param :destination_server, String, :desc => N_("Destination Server name"), :required => false
@@ -41,7 +34,7 @@ module Katello
     param :fail_on_missing_content, :bool, :desc => N_("Fails if any of the repositories belonging to this version"\
                                                          " are unexportable. False by default."), :required => false
     def version
-      tasks = async_task(::Actions::Pulp3::Orchestration::ContentViewVersion::Export,
+      tasks = async_task(::Actions::Katello::ContentViewVersion::Export,
                           content_view_version: @version,
                           destination_server: params[:destination_server],
                           chunk_size: params[:chunk_size_mb],
@@ -78,11 +71,5 @@ module Katello
         throw_resource_not_found(name: 'organization', id: params[:organization_id])
       end
     end
-
-    def fail_if_not_pulp3
-      unless SmartProxy.pulp_primary.pulp3_repository_type_support?(Katello::Repository::YUM_TYPE)
-        fail HttpErrors::BadRequest, _("Invalid usage for Pulp 2 repositories. Use export for Yum repositories")
-      end
-    end
   end
 end

data/app/controllers/katello/api/v2/content_imports_controller.rb

@@ -1,20 +1,39 @@
 module Katello
   class Api::V2::ContentImportsController < Api::V2::ApiController
-    before_action :find_publishable_content_view, :only => [:version]
+    before_action :find_organization, :only => [:version]
+    before_action :check_authorized, :only => [:version]
+
     before_action :find_importable_organization, :only => [:library]
     before_action :find_default_content_view, :only => [:library]
 
+    api :GET, "/content_imports", N_("List import histories")
+    param :content_view_version_id, :number, :desc => N_("Content view version identifier"), :required => false
+    param :content_view_id, :number, :desc => N_("Content view identifier"), :required => false
+    param :organization_id, :number, :desc => N_("Organization identifier"), :required => false
+    param :id, :number, :desc => N_("Content view version import history identifier"), :required => false
+    param :type, ::Katello::ContentViewVersionExportHistory::EXPORT_TYPES,
+                                  :desc => N_("Import Types"),
+                                  :required => false
+    param_group :search, Api::V2::ApiController
+    add_scoped_search_description_for(ContentViewVersionImportHistory)
+    def index
+      history = ContentViewVersionImportHistory.readable
+      history = history.where(:id => params[:id]) unless params[:id].blank?
+      history = history.where(:content_view_version_id => params[:content_view_version_id]) unless params[:content_view_version_id].blank?
+      history = history.with_organization_id(params[:organization_id]) unless params[:organization_id].blank?
+      history = history.with_content_view_id(params[:content_view_id]) unless params[:content_view_id].blank?
+      history = history.where(:import_type => params[:type]) unless params[:type].blank?
+      respond_with_template_collection("index", 'content_view_version_import_histories',
+                                       :collection => scoped_search(history, 'id', 'asc', resource_class: ContentViewVersionImportHistory))
+    end
+
     api :POST, "/content_imports/version", N_("Import a content view version")
-    param :content_view_id, :number, :desc => N_("Content view identifier"), :required => true
+    param :organization_id, :number, :desc => N_("Organization identifier"), :required => true
     param :path, String, :desc => N_("Directory containing the exported Content View Version"), :required => true
     param :metadata, Hash, :desc => N_("Metadata taken from the upstream export history for this Content View Version"), :required => true
     def version
-      if @view.default?
-        fail HttpErrors::BadRequest, _("Cannot use this end point for importing to library. "\
-                                       "If you intented to upload to library, use the library endpoint.")
-      end
-
-      task = async_task(::Actions::Katello::ContentViewVersion::Import, @view, path: params[:path], metadata: metadata_params.to_h)
+      task = async_task(::Actions::Katello::ContentViewVersion::Import, organization: @organization,
+                                                  path: params[:path], metadata: metadata_params.to_h)
       respond_for_async :resource => task
     end
 
@@ -29,9 +48,8 @@ module Katello
 
     private
 
-    def find_publishable_content_view
-      @view = ContentView.publishable.find(params[:content_view_id])
-      throw_resource_not_found(name: 'content_view', id: params[:content_view_id]) if @view.blank?
+    def check_authorized
+      fail HttpErrors::Forbidden, _("Action unauthorized to be performed in this organization.") unless ContentView.importable?
     end
 
     def find_default_content_view
@@ -47,12 +65,17 @@ module Katello
     def metadata_params
       params.require(:metadata).permit(
         :organization,
-        :content_view,
-        :repository_mapping,
+        :repositories,
+        :products,
         :toc,
-        content_view_version: [:major, :minor]
+        :incremental,
+        gpg_keys: {},
+        content_view: [:name, :label, :description],
+        content_view_version: [:major, :minor, :description],
+        from_content_view_version: [:major, :minor]
       ).tap do |nested|
-        nested[:repository_mapping] = params[:metadata].require(:repository_mapping).permit!
+        nested[:repositories] = params[:metadata].require(:repositories).permit!
+        nested[:products] = params[:metadata].require(:products).permit!
       end
     end
   end

data/app/controllers/katello/api/v2/content_uploads_controller.rb

@@ -10,7 +10,7 @@ module Katello
     param :repository_id, :number, :required => true, :desc => N_("repository id")
     param :size, :number, :required => true, :desc => N_("Size of file to upload")
     param :checksum, String, :required => false, :desc => N_("Checksum of file to upload")
-    param :content_type, RepositoryTypeManager.uploadable_content_types.map(&:label), :required => false, :desc => N_("content type ('deb', 'docker_manifest', 'file', 'ostree', 'puppet_module', 'rpm', 'srpm')")
+    param :content_type, RepositoryTypeManager.uploadable_content_types.map(&:label), :required => false, :desc => N_("content type ('deb', 'docker_manifest', 'file', 'ostree', 'rpm', 'srpm')")
     def create
       content_type = params[:content_type] || ::Katello::RepositoryTypeManager.find(@repository.content_type).default_managed_content_type.label
       unit_type_id = SmartProxy.pulp_primary.content_service(content_type).content_type

data/app/controllers/katello/api/v2/content_view_components_controller.rb

@@ -1,5 +1,6 @@
 module Katello
   class Api::V2::ContentViewComponentsController < Api::V2::ApiController
+    include Katello::Concerns::FilteredAutoCompleteSearch
     before_action :find_composite_content_view, :only => [:show, :index]
     before_action :find_composite_content_view_for_edit, :only => [:add_components, :remove_components, :update]
     before_action :find_authorized_katello_resource, :only => [:show, :update]
@@ -10,7 +11,12 @@ module Katello
         N_("List components attached to this content view")
     param :composite_content_view_id, :number, :desc => N_("composite content view identifier"), :required => true
     def index
-      respond :collection => index_response
+      sort_by, sort_order, options = sort_options
+      respond(:collection => scoped_search(index_relation, sort_by, sort_order, options))
+    end
+
+    def index_relation
+      @view.content_view_components.readable
     end
 
     def index_response
@@ -114,5 +120,20 @@ module Katello
       attrs = [:latest, :content_view_version_id, :content_view_id]
       params.require(:content_view_component).permit(*attrs)
     end
+
+    def default_sort
+      %w(label asc)
+    end
+
+    def sort_options
+      case default_sort
+      when Array
+        [default_sort[0], default_sort[1], {}]
+      when Proc
+        [nil, nil, { :custom_sort => default_sort }]
+      else
+        fail "Unsupported default_sort type"
+      end
+    end
   end
 end

data/app/controllers/katello/api/v2/content_view_filter_rules_controller.rb

@@ -1,7 +1,8 @@
 module Katello
   class Api::V2::ContentViewFilterRulesController < Api::V2::ApiController
+    include Katello::Concerns::FilteredAutoCompleteSearch
     before_action :find_filter
-    before_action :find_rule, :except => [:index, :create]
+    before_action :find_rule, :except => [:index, :create, :auto_complete_search]
 
     api :GET, "/content_view_filters/:content_view_filter_id/rules", N_("List filter rules")
     param :content_view_filter_id, :number, :desc => N_("filter identifier"), :required => true
@@ -19,6 +20,10 @@ module Katello
       query
     end
 
+    def resource_class
+      ContentViewFilter.rule_class_for(@filter)
+    end
+
     api :POST, "/content_view_filters/:content_view_filter_id/rules",
         N_("Create a filter rule. The parameters included should be based upon the filter type.")
     param :content_view_filter_id, :number, :desc => N_("filter identifier"), :required => true

data/app/controllers/katello/api/v2/content_view_versions_controller.rb

@@ -3,23 +3,20 @@ module Katello
     include Concerns::Api::V2::BulkHostsExtensions
     include Katello::Concerns::FilteredAutoCompleteSearch
 
-    before_action :find_authorized_katello_resource, :only => [:show, :update, :promote, :destroy, :export, :republish_repositories]
-    before_action :find_content_view_from_version, :only => [:show, :update, :promote, :destroy, :export, :republish_repositories]
+    before_action :find_authorized_katello_resource, :only => [:show, :update, :promote, :destroy, :republish_repositories]
+    before_action :find_content_view_from_version, :only => [:show, :update, :promote, :destroy, :republish_repositories]
     before_action :find_optional_readable_content_view, :only => [:index]
-    before_action :find_publishable_content_view, :only => [:import]
 
     before_action :find_environment, :only => [:index]
     before_action :find_environments, :only => [:promote]
     before_action :validate_promotable, :only => [:promote]
     before_action :authorize_destroy, :only => [:destroy]
     before_action :find_version_environments, :only => [:incremental_update]
-    before_action :find_puppet_module, :only => [:index]
 
     api :GET, "/content_view_versions", N_("List content view versions")
     api :GET, "/content_views/:content_view_id/content_view_versions", N_("List content view versions")
     param :content_view_id, :number, :desc => N_("Content view identifier"), :required => false
     param :environment_id, :number, :desc => N_("Filter versions by environment"), :required => false
-    param :puppet_module_id, :number, :desc => N_("Filter versions by puppet module"), :required => false
     param :version, String, :desc => N_("Filter versions by version number"), :required => false
     param :composite_version_id, :number, :desc => N_("Filter versions that are components in the specified composite version"), :required => false
     param :organization_id, :number, :desc => N_("Organization identifier")
@@ -41,7 +38,6 @@ module Katello
       versions = versions.where(:content_view_id => @view.id) if @view
       versions = versions.for_version(version_number) if version_number
       versions = versions.in_environment(@environment) if @environment
-      versions = versions.with_puppet_module(@puppet_module) if @puppet_module
       versions = versions.component_of(params[:composite_version_id]) if params[:composite_version_id]
       versions
     end
@@ -85,49 +81,6 @@ module Katello
       respond_for_async :resource => task
     end
 
-    api :POST, "/content_view_versions/:id/export", N_("Export a content view version.  Relevant only for Pulp 2 repositories.")
-    param :id, :number, :desc => N_("Content view version identifier"), :required => true
-    param :export_to_iso, :bool, :desc => N_("Export to ISO format."), :required => false
-    param :iso_mb_size, :number, :desc => N_("maximum size of each ISO in MB. Relevant only for Pulp 2 repositories"), :required => false
-    param :since, Date, :desc => N_("Optional date of last export (ex: 2010-01-01T12:00:00Z). Relevant only for Pulp 2 repositories"), :required => false
-    def export
-      if SmartProxy.pulp_primary.pulp3_repository_type_support?(Katello::Repository::YUM_TYPE)
-        fail HttpErrors::BadRequest, _("Invalid usage for Pulp 3 repositories. "\
-                                       "Use hammer content-export for Yum repositories")
-      end
-      ::Foreman::Deprecation.api_deprecation_warning("Pulp 2 export is deprecated and will not work as of Katello 4.0. To export using Pulp 3, use hammer content-export version.")
-      if params[:export_to_iso].blank? && params[:iso_mb_size].present?
-        fail HttpErrors::BadRequest, _("ISO export must be enabled when specifying ISO size")
-      end
-
-      if (repos = @content_view_version.content_view.on_demand_repositories).any?
-        fail HttpErrors::BadRequest, _("This content view has on demand repositories that cannot be exported: %{repos}" % {repos: repos.pluck(:label).join(", ")})
-      end
-
-      if params[:since].present?
-        begin
-          params[:since].to_datetime
-        rescue
-          raise HttpErrors::BadRequest, _("Invalid date provided.")
-        end
-      end
-      task = async_task(::Actions::Katello::ContentViewVersion::Export, @content_view_version,
-                        ::Foreman::Cast.to_bool(params[:export_to_iso]),
-                        params[:since].try(:to_datetime),
-                        params[:iso_mb_size])
-
-      respond_for_async :resource => task
-    end
-
-    api :POST, "/content_view_versions/import", N_("Import a content view version")
-    param :content_view_id, :number, :desc => N_("Content view identifier"), :required => true
-    param :path, String, :desc => N_("Directory containing the exported Content View Version"), :required => true
-    param :metadata, Hash, :desc => N_("Metadata taken from the upstream export history for this Content View Version"), :required => true
-    def import
-      task = async_task(::Actions::Katello::ContentViewVersion::Import, @view, path: params[:path], metadata: metadata_params&.to_h)
-      respond_for_async :resource => task
-    end
-
     api :DELETE, "/content_view_versions/:id", N_("Remove content view version")
     param :id, :number, :desc => N_("Content view version identifier"), :required => true
     def destroy
@@ -147,7 +100,6 @@ module Katello
       param :errata_ids, Array, :desc => "Errata ids to copy into the new versions"
       param :package_ids, Array, :desc => "Package ids to copy into the new versions"
       param :deb_ids, Array, :desc => "Deb Package ids to copy into the new versions"
-      param :puppet_module_ids, Array, :desc => "Puppet Module ids to copy into the new versions"
     end
     param :update_hosts, Hash, :desc => N_("After generating the incremental update, apply the changes to the specified hosts.  Only Errata are supported currently.") do
       param :included, Hash, :required => true, :action_aware => true do
@@ -281,11 +233,6 @@ module Katello
       @environment = KTEnvironment.find(params[:environment_id])
     end
 
-    def find_puppet_module
-      return unless params.key?(:puppet_module_id)
-      @puppet_module = PuppetModule.with_identifiers([params[:puppet_module_id]]).first
-    end
-
     def validate_content(content)
       if content[:errata_ids]
         errata = Erratum.with_identifiers(content[:errata_ids])
@@ -303,10 +250,6 @@ module Katello
       if content[:deb_ids]
         fail _("deb_ids is not an array") unless content[:deb_ids].is_a?(Array)
       end
-
-      if content[:puppet_module_ids]
-        fail _("puppet_module_ids is not an array") unless content[:puppet_module_ids].is_a?(Array)
-      end
     end
 
     def find_environments