katello 3.16.2 → 3.17.0.rc1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7e3320c3167ac19ebd8119f1d4e58f0dcc141a4a18e4bccc98f1a4965e09262d
-  data.tar.gz: a9d5092691dca107610bfae636df54566cc07bf9fdf724d4c2e96312c3fdca9b
+  metadata.gz: bf008f8880960ad2a6cf9b615c0e0801fb38cce104e67001f4d17178cca43ab7
+  data.tar.gz: d2cda17cc4664479ac34f7abd33c08f14ef81f038abb0cfe3d43c9542624c48b
 SHA512:
-  metadata.gz: 263362175e3deffd8bb979fd7464447b66524ccf5156a2a9836ae92b8ccd615b443202e3072309cc8d2d870ff899ddb4256887a5385c0fe860eb903b4b6a7066
-  data.tar.gz: af6e7d83da1cb8ac22dd279156e076cc42e605fb4f5b968d9e027f19903c3a819ebec6c8149f4a2f16ef43a9b3d5d34bb7ed4359c9b6b86e041ce0180c486a6a
+  metadata.gz: 27f052edff77986139ef6a82d0cd318c95b90a601a9f42178038896589c1042b1e18d030320173ae2c5a21ac75bfe09b13e7376525574f7dbff9d33b4867b8fa
+  data.tar.gz: 22f05c56710ee8951e1d15d30e5b2d471ab07f17287c51c01b7958e751442197fe89b8d262eb89584e612bba939aad7e8d7ddd407d55937d31a04d00e1df8e10

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

@@ -1,10 +1,3 @@
-/**
- * = require "katello/jquery.loadmask.css"
- * = require "katello/jquery-ui-1.8.11.custom.css"
- * = require "katello/jquery.treeTable.css"
- * = require "katello/ui.spinner.css"
- */
-
 @import "katello/overrides";
 @import "katello/katello_base";
 @import "katello/look";
@@ -18,6 +11,9 @@
 @import "katello/generic";
 @import "katello/notifications";
 
+@import "katello/jquery.treeTable";
+@import "katello/jquery-ui-1.8.11.custom.css";
+
 body {
   background-color: white;
 }

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

@@ -480,7 +480,7 @@ module Katello
         else
           User.consumer? || ::User.current.can?(:view_organizations, self)
         end
-      when "rhsm_proxy_owner_servicelevels_path", "rhsm_proxy_owner_system_purpose_path"
+      when "rhsm_proxy_owner_servicelevels_path"
         (User.consumer? || ::User.current.can?(:view_organizations, self))
       when "rhsm_proxy_consumer_accessible_content_path", "rhsm_proxy_consumer_certificates_path",
            "rhsm_proxy_consumer_releases_path", "rhsm_proxy_certificate_serials_path",

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

@@ -1,4 +1,5 @@
 module Katello
+  # rubocop:disable Metrics/ClassLength
   class Api::V2::ActivationKeysController < Api::V2::ApiController
     include Katello::Concerns::FilteredAutoCompleteSearch
     include Katello::Concerns::Api::V2::ContentOverridesController
@@ -11,6 +12,7 @@ module Katello
                                                   :content_override, :add_subscriptions, :remove_subscriptions,
                                                   :subscriptions]
     before_action :authorize
+    before_action :verify_simple_content_access_disabled, :only => [:add_subscriptions]
 
     wrap_parameters :include => (ActivationKey.attribute_names + %w(host_collection_ids service_level auto_attach purpose_role purpose_usage purpose_addons content_view_environment))
 
@@ -348,5 +350,11 @@ module Katello
 
       key_params
     end
+
+    def verify_simple_content_access_disabled
+      if @activation_key.organization.simple_content_access?
+        fail HttpErrors::BadRequest, _("The specified organization is in Simple Content Access mode. Attaching subscriptions is disabled")
+      end
+    end
   end
 end

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

@@ -39,7 +39,9 @@ module Katello
       respond_for_index :collection => @collection
     end
 
-    def index_response
+    def index_response(reload_host = false)
+      # Host needs to be reloaded because of lazy accessor
+      @host.reload if reload_host
       entitlements = @host.subscription_facet.candlepin_consumer.entitlements
       subscriptions = entitlements.map { |entitlement| ::Katello::HostSubscriptionPresenter.new(entitlement) }
       full_result_response(subscriptions)
@@ -53,7 +55,7 @@ module Katello
       end
 
       sync_task(::Actions::Katello::Host::AutoAttachSubscriptions, @host)
-      respond_for_index(:collection => index_response, :template => "index")
+      respond_for_index(:collection => index_response(true), :template => "index")
     end
 
     api :DELETE, "/hosts/:host_id/subscriptions/", N_("Unregister the host as a subscription consumer")
@@ -124,7 +126,7 @@ module Katello
       end
 
       sync_task(::Actions::Katello::Host::RemoveSubscriptions, @host, pool_id_quantities.values)
-      respond_for_index(:collection => index_response, :template => "index")
+      respond_for_index(:collection => index_response(true), :template => "index")
     end
 
     api :PUT, "/hosts/:host_id/subscriptions/add_subscriptions", N_("Add a subscription to a host")
@@ -134,12 +136,16 @@ module Katello
       param :quantity, :number, :desc => N_("Quantity of this subscriptions to add"), :required => true
     end
     def add_subscriptions
+      if @host.organization.simple_content_access?
+        fail ::Katello::HttpErrors::BadRequest, _("This host's organization is in Simple Content Access mode. Attaching subscriptions is disabled.")
+      end
+
       pools_with_quantities = params.require(:subscriptions).map do |sub_params|
         PoolWithQuantities.new(Pool.with_identifier(sub_params['id']), sub_params['quantity'].to_i)
       end
 
       sync_task(::Actions::Katello::Host::AttachSubscriptions, @host, pools_with_quantities)
-      respond_for_index(:collection => index_response, :template => "index")
+      respond_for_index(:collection => index_response(true), :template => "index")
     end
 
     api :PUT, "/hosts/:host_id/subscriptions/content_override", N_("Set content overrides for the host")

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

@@ -1,6 +1,6 @@
 module Katello
   class Api::V2::HostTracerController < Api::V2::ApiController
-    before_action :find_host
+    before_action :find_host, :only => :index
 
     resource_description do
       api_version 'v2'
@@ -14,18 +14,43 @@ module Katello
       respond_for_index(:collection => collection)
     end
 
-    api :PUT, "/hosts/:host_id/traces/resolve", N_("Resolve Traces")
-    param :host_id, :number, :required => true, :desc => N_("ID of the host")
+    api :PUT, "/traces/resolve", N_("Resolve Traces")
     param :trace_ids, Array, :required => true, :desc => N_("Array of Trace IDs")
     def resolve
-      traces = @host.host_traces.resolvable.where(id: params[:trace_ids])
-      fail HttpErrors::BadRequest, _("The requested traces were not found for this host") if traces.empty?
+      traces = Katello::HostTracer.resolvable.where(id: params[:trace_ids])
+
+      traces.each do |trace|
+        if trace.reboot_required?
+          trace.helper = 'reboot'
+        end
+      end
+
+      traces_by_host_id = traces.group_by(&:host_id)
+      traces_by_helper = traces.group_by(&:helper)
+
+      composers = []
+
+      if traces_by_host_id.size < traces_by_helper.size
+        traces_by_host_id.each do |host_id, trace|
+          needed_traces = trace.map(&:helper).join(',')
+          joined_helpers = { :helper => needed_traces }
+          composers << JobInvocationComposer.for_feature(:katello_service_restart, [host_id], joined_helpers)
+        end
+      else
+        traces_by_helper.each do |helper, trace|
+          helpers = { :helper => helper }
+          composers << JobInvocationComposer.for_feature(:katello_service_restart, trace.map(&:host_id), helpers)
+        end
+      end
 
-      result = Katello::HostTraceManager.resolve_traces(traces)
+      job_invocations = []
 
-      task = ForemanTasks::Task.find(result.first.task_id)
+      composers.each do |composer|
+        composer.trigger
+        job_invocations << composer.job_invocation
+      end
 
-      respond_for_async(resource: task)
+      render json: job_invocations
     end
 
     protected

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

@@ -6,13 +6,15 @@ module Katello
     before_action :find_host_collections, :only => [:bulk_add_host_collections, :bulk_remove_host_collections]
     before_action :find_environment, :only => [:environment_content_view]
     before_action :find_content_view, :only => [:environment_content_view]
-    before_action :find_editable_hosts, :except => [:destroy_hosts, :applicable_errata, :installable_errata, :resolve_traces]
+    before_action :find_editable_hosts, :except => [:destroy_hosts, :applicable_errata, :installable_errata]
     before_action :find_deletable_hosts, :only => [:destroy_hosts]
     before_action :find_readable_hosts, :only => [:applicable_errata, :installable_errata, :available_incremental_updates]
     before_action :find_errata, :only => [:available_incremental_updates]
+    before_action :find_organization, :only => [:add_subscriptions]
     before_action :deprecate_katello_agent, :only => [:install_content, :update_content, :remove_content]
 
     before_action :validate_content_action, :only => [:install_content, :update_content, :remove_content]
+    before_action :validate_organization, :only => [:add_subscriptions]
 
     # disable *_count fields on erratum rabl, since they perform N+1 queries
     before_action :disable_erratum_hosts_count
@@ -173,6 +175,10 @@ module Katello
       param :quantity, :number, :desc => N_("Quantity of this subscriptions to add"), :required => true
     end
     def add_subscriptions
+      if @organization.simple_content_access?
+        fail HttpErrors::BadRequest, _("The specified organization is in Simple Content Access mode. Attaching subscriptions is disabled")
+      end
+
       pools_with_quantities = params.require(:subscriptions).map do |sub_params|
         PoolWithQuantities.new(Pool.find(sub_params['id']), sub_params['quantity'])
       end
@@ -230,16 +236,6 @@ module Katello
       respond_for_index(:collection => collection, :template => '../../../api/v2/host_tracer/index')
     end
 
-    api :PUT, "/hosts/bulk/resolve_traces", N_("Resolve traces for one or more hosts")
-    param_group :bulk_params
-    param :trace_ids, Array, :required => true, :desc => N_("Array of Trace IDs")
-    def resolve_traces
-      traces = Katello::HostTracer.resolvable.where(id: params[:trace_ids])
-      result = Katello::HostTraceManager.resolve_traces(traces)
-
-      render json: result
-    end
-
     api :POST, "/hosts/bulk/available_incremental_updates", N_("Given a set of hosts and errata, lists the content view versions" \
                                                                  " and environments that need updating.")
     param_group :bulk_params
@@ -305,6 +301,10 @@ module Katello
       find_bulk_hosts(:destroy_hosts, params)
     end
 
+    def validate_organization
+      fail HttpErrors::BadRequest, _("Organization ID is required") if @organization.blank?
+    end
+
     def validate_host_collection_membership_limit
       max_hosts_exceeded = []
       host_ids = @hosts.map(&:id)

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

@@ -50,21 +50,6 @@ module Katello
       respond_for_async :resource => task
     end
 
-    api :PUT, "/products/bulk/verify_checksum", N_("Verify checksum for one or more products")
-    param :ids, Array, :desc => N_("List of product ids"), :required => true
-    def verify_checksum_products
-      repairable_products = @products.syncable
-      repairable_roots = RootRepository.where(:product_id => repairable_products).
-        where(:content_type => ::Katello::Repository::YUM_TYPE).has_url.select { |r| r.library_instance }.uniq
-
-      repairable_repositories = Katello::Repository.where(:root_id => repairable_roots)
-      task = async_task(::Actions::BulkAction,
-                        ::Actions::Katello::Repository::VerifyChecksum,
-                        repairable_repositories)
-
-      respond_for_async :resource => task
-    end
-
     api :PUT, "/products/bulk/http_proxy", N_("Update the HTTP proxy configuration on the repositories of one or more products.")
     param :ids, Array, :desc => N_("List of product ids"), :required => true
     param :http_proxy_policy, ::Katello::RootRepository::HTTP_PROXY_POLICIES, :desc => N_("policy for HTTP proxy for content sync")

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

@@ -15,7 +15,7 @@ module Katello
     before_action :find_organization_from_product, :only => [:create]
     before_action :find_repository, :only => [:show, :update, :destroy, :sync, :export,
                                               :remove_content, :upload_content, :republish,
-                                              :import_uploads, :gpg_key_content, :verify_checksum]
+                                              :import_uploads, :gpg_key_content]
     before_action :find_content, :only => :remove_content
     before_action :find_organization_from_repo, :only => [:update]
     before_action :error_on_rh_product, :only => [:create]
@@ -57,6 +57,7 @@ module Katello
       param :ansible_collection_requirements, String, :desc => N_("Contents of requirement yaml file to sync from URL")
       param :http_proxy_policy, ::Katello::RootRepository::HTTP_PROXY_POLICIES, :desc => N_("policies for HTTP proxy for content sync")
       param :http_proxy_id, :number, :desc => N_("ID of a HTTP Proxy")
+      param :auto_enabled, :bool, :desc => N_("if true, the repositories will be automatically enabled on a registered host subscribed to this product. Default: true")
     end
 
     def_param_group :repo_create do
@@ -294,15 +295,6 @@ module Katello
       raise HttpErrors::BadRequest, e.message
     end
 
-    api :POST, "/repositories/:id/verify_checksum", N_("Verify checksum of repository contents")
-    param :id, :number, :required => true, :desc => N_("repository ID")
-    def verify_checksum
-      task = async_task(::Actions::Katello::Repository::VerifyChecksum, @repository)
-      respond_for_async :resource => task
-    rescue Errors::InvalidActionOptionError => e
-      raise HttpErrors::BadRequest, e.message
-    end
-
     api :POST, "/repositories/:id/export", N_("Export a repository")
     param :id, :number, :desc => N_("Repository identifier"), :required => true
     param :export_to_iso, :bool, :desc => N_("Export to ISO format"), :required => false
@@ -336,6 +328,7 @@ module Katello
     api :PUT, "/repositories/:id", N_("Update a repository")
     param :id, :number, :required => true, :desc => N_("repository ID")
     param :name, String, :required => false
+    param :description, String, :desc => N_("description of the repository"), :required => false
     param_group :repo
     def update
       repo_params = repository_params
@@ -472,7 +465,7 @@ module Katello
 
     def repository_params
       keys = [:download_policy, :mirror_on_sync, :arch, :verify_ssl_on_sync, :upstream_password, :upstream_username,
-              :ostree_upstream_sync_depth, :ostree_upstream_sync_policy,
+              :ostree_upstream_sync_depth, :ostree_upstream_sync_policy, :auto_enabled,
               :deb_releases, :deb_components, :deb_architectures, :description, :http_proxy_policy, :http_proxy_id,
               {:ignorable_content => []}
              ]
@@ -523,6 +516,7 @@ module Katello
       root.ansible_collection_requirements = repo_params[:ansible_collection_requirements] if root.ansible_collection?
       root.http_proxy_policy = repo_params[:http_proxy_policy] if repo_params.key?(:http_proxy_policy)
       root.http_proxy_id = repo_params[:http_proxy_id] if repo_params.key?(:http_proxy_id)
+      root.auto_enabled = repo_params[:auto_enabled] if repo_params.key?(:auto_enabled)
 
       if root.ostree?
         root.ostree_upstream_sync_policy = repo_params[:ostree_upstream_sync_policy]

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

@@ -77,6 +77,22 @@ module Katello
       render json: { status: 'OK' }
     end
 
+    api :PUT, "/organizations/:organization_id/simple_content_access/enable",
+      N_("Enable simple content access for a manifest")
+    param :organization_id, :number, :desc => N_("Organization ID"), :required => true
+    def enable_simple_content_access
+      task = async_task(::Actions::Katello::Organization::SimpleContentAccess::Enable, params[:organization_id])
+      respond_for_async :resource => task
+    end
+
+    api :PUT, "/organizations/:organization_id/simple_content_access/disable",
+      N_("Disable simple content access for a manifest")
+    param :organization_id, :number, :desc => N_("Organization ID"), :required => true
+    def disable_simple_content_access
+      task = async_task(::Actions::Katello::Organization::SimpleContentAccess::Disable, params[:organization_id])
+      respond_for_async :resource => task
+    end
+
     private
 
     def update_params

data/app/controllers/katello/concerns/api/v2/bulk_hosts_extensions.rb

@@ -5,19 +5,19 @@ module Katello
 
       def find_bulk_hosts(permission, bulk_params, restrict_to = nil)
         #works on a structure of param_group bulk_params and transforms it into a list of systems
-        organization = find_organization
+        find_organization
         bulk_params[:included] ||= {}
         bulk_params[:excluded] ||= {}
         @hosts = []
 
         unless bulk_params[:included][:ids].blank?
           @hosts = ::Host::Managed.authorized(permission).where(:id => bulk_params[:included][:ids])
-          @hosts = @hosts.where(:organization_id => organization.id) if organization
+          @hosts = @hosts.where(:organization_id => @organization.id) if @organization
         end
 
         if bulk_params[:included][:search]
           search_hosts = ::Host::Managed.authorized(permission)
-          search_hosts = search_hosts.where(:organization_id => organization_id) if params[:organization_id]
+          search_hosts = search_hosts.where(:organization_id => @organization.id) if @organization
           search_hosts = search_hosts.search_for(bulk_params[:included][:search])
           if @hosts.any?
             @hosts = ::Host.where(id: @hosts).or(::Host.where(id: search_hosts))
@@ -38,7 +38,7 @@ module Katello
       end
 
       def find_organization
-        Organization.find_by_id(params[:organization_id])
+        @organization ||= Organization.find_by_id(params[:organization_id])
       end
     end
   end

data/app/controllers/katello/concerns/api/v2/hostgroups_controller_extensions.rb

@@ -20,7 +20,7 @@ module Katello
         include ApiPieExtensions
 
         def create
-          @hostgroup = Hostgroup.new(hostgroup_params)
+          @hostgroup = ::Hostgroup.new(hostgroup_params)
           process_response @hostgroup.save
         end
 

data/app/controllers/katello/concerns/hosts_controller_extensions.rb

@@ -19,17 +19,11 @@ module Katello
         prepend Overrides
 
         def update_multiple_taxonomies(type)
-          if type == :organization
-            new_org_id = params.dig(type, 'id')
-
-            if new_org_id
-              registered_host = @hosts.where.not(organization_id: new_org_id).joins(:subscription_facet).first
-              if registered_host
-                error _("Unregister host %s before assigning an organization") % registered_host.name
-                redirect_back_or_to hosts_path
-                return
-              end
-            end
+          registered_host = @hosts.detect { |host| host.subscription_facet }
+          unless registered_host.nil?
+            error _("Unregister host %s before assigning an organization") % registered_host.name
+            redirect_back_or_to hosts_path
+            return
           end
 
           super

data/app/helpers/katello/concerns/dashboard_helper_extensions.rb

@@ -28,6 +28,16 @@ module Katello
       def unsubscribed_hypervisor_count
         host_query.search_for("subscription_status = unsubscribed_hypervisor").size
       end
+
+      def removed_widgets
+        widgets = super
+
+        if Organization.current&.simple_content_access?
+          widgets.reject! { |widget| ::Widget.singleton_class::SUBSCRIPTION_TEMPLATES.include? widget[:template] }
+        end
+
+        widgets
+      end
     end
   end
 end

data/app/helpers/katello/hosts_and_hostgroups_helper.rb

@@ -15,7 +15,7 @@ module Katello
     end
 
     def organizations(host)
-      if host.is_a?(Hostgroup)
+      if host.is_a?(::Hostgroup)
         host.organizations
       else
         host.organization ? [host.organization] : []
@@ -29,8 +29,7 @@ module Katello
 
     def host_hostgroup_kickstart_repository_id(host)
       return if host.blank?
-      return host.kickstart_repository_id if host.is_a?(Hostgroup)
-      host.content_facet.kickstart_repository_id if host.try(:content_facet).present?
+      host.content_facet&.kickstart_repository_id
     end
 
     def kickstart_repository_id(host, options = {})
@@ -78,13 +77,13 @@ module Katello
 
     def accessible_lifecycle_environments(org, host)
       selected = host.lifecycle_environment
-      envs = org.kt_environments.readable
+      envs = org.kt_environments.readable.order(:name)
       envs |= [selected] if selected.present? && org == selected.organization
       envs
     end
 
     def accessible_content_proxies(obj)
-      list = accessible_resource_records(:smart_proxy).with_content.to_a
+      list = accessible_resource_records(:smart_proxy).with_content.order(:name).to_a
       current = obj.content_source
       list |= [current] if current.present?
       list
@@ -159,7 +158,7 @@ module Katello
 
       views = []
       if lifecycle_environment
-        views = Katello::ContentView.in_environment(lifecycle_environment).readable
+        views = Katello::ContentView.in_environment(lifecycle_environment).readable.order(:name)
         views |= [content_view] if content_view.present? && content_view.in_environment?(lifecycle_environment)
       elsif content_view
         views = [content_view]
@@ -205,7 +204,7 @@ module Katello
 
         return [] unless new_host.operatingsystem.is_a?(Redhat)
 
-        if (host.is_a? Hostgroup)
+        if (host.is_a? ::Hostgroup)
           new_host.content_facet = ::Katello::Host::ContentFacet.new(:lifecycle_environment_id => host.inherited_lifecycle_environment_id,
                                                           :content_view_id => host.inherited_content_view_id,
                                                           :content_source_id => host.inherited_content_source_id)

data/app/lib/actions/candlepin/product/content_add.rb

@@ -6,11 +6,12 @@ module Actions
           param :product_id
           param :content_id
           param :owner
+          param :enabled
         end
 
         def run
           output[:response] = ::Katello::Resources::Candlepin::Product.
-              add_content(input[:owner], input[:product_id], input[:content_id], true)
+              add_content(input[:owner], input[:product_id], input[:content_id], input[:enabled])
         end
       end
     end

data/app/lib/actions/candlepin/product/content_update_enablement.rb

@@ -0,0 +1,18 @@
+module Actions
+  module Candlepin
+    module Product
+      class ContentUpdateEnablement < Candlepin::Abstract
+        input_format do
+          param :content_enablements
+          param :owner
+          param :product_id
+        end
+
+        def run
+          output[:response] = ::Katello::Resources::Candlepin::Product.
+              update_enabled(input[:owner], input[:product_id], input[:content_enablements])
+        end
+      end
+    end
+  end
+end

data/app/lib/actions/katello/applicability/hosts/bulk_generate.rb

@@ -9,12 +9,8 @@ module Actions
 
           def run
             input[:host_ids].each do |host_id|
-              content_facet = ::Katello::Host::ContentFacet.find_by_host_id(host_id)
-              if content_facet.present?
-                content_facet.calculate_and_import_applicability
-              else
-                Rails.logger.warn(_("Content Facet for host with id %s is non-existent. Skipping applicability calculation.") % host_id)
-              end
+              content_facet = ::Host.find(host_id).content_facet
+              content_facet.calculate_and_import_applicability
             end
           end
 

data/app/lib/actions/katello/capsule_content/refresh_repos.rb

@@ -37,7 +37,7 @@ module Actions
           current_repos_on_capsule = smart_proxy_service.current_repositories(environment, content_view)
           current_repos_on_capsule_ids = current_repos_on_capsule.pluck(:id)
 
-          list_of_repos_to_sync = smart_proxy_helper.combined_repos_available_to_capsule(environment, content_view, repository)
+          list_of_repos_to_sync = smart_proxy_helper.repos_available_to_capsule(environment, content_view, repository)
           list_of_repos_to_sync.each do |repo|
             if repo.is_a?(Katello::ContentViewPuppetEnvironment)
               repo = repo.nonpersisted_repository

data/app/lib/actions/katello/capsule_content/sync.rb

@@ -31,7 +31,7 @@ module Actions
           fail _("Action not allowed for the default smart proxy.") if smart_proxy.pulp_master?
 
           smart_proxy_helper = ::Katello::SmartProxyHelper.new(smart_proxy)
-          repositories = smart_proxy_helper.combined_repos_available_to_capsule(environment, content_view, repository)
+          repositories = smart_proxy_helper.repos_available_to_capsule(environment, content_view, repository)
 
           smart_proxy.ping_pulp if repositories.any? { |repo| !smart_proxy.pulp3_support?(repo) }
 

data/app/lib/actions/katello/capsule_content/sync_capsule.rb

@@ -10,12 +10,11 @@ module Actions
           repository = options[:repository]
           skip_metadata_check = options.fetch(:skip_metadata_check, false)
 
+          smart_proxy_helper = ::Katello::SmartProxyHelper.new(smart_proxy)
           sequence do
-            repos = repos_to_sync(smart_proxy, environment, content_view, repository)
-
-            repos.in_groups_of(Setting[:foreman_proxy_content_batch_size], false) do |repo_batch|
+            smart_proxy_helper.repos_available_to_capsule(environment, content_view, repository).in_groups_of(Setting[:foreman_proxy_content_batch_size], false) do |repos|
               concurrence do
-                repo_batch.each do |repo|
+                repos.each do |repo|
                   plan_pulp_action([Actions::Pulp::Orchestration::Repository::SmartProxySync,
                                     Actions::Pulp3::CapsuleContent::Sync],
                                      repo, smart_proxy,
@@ -34,19 +33,6 @@ module Actions
           end
         end
 
-        def repos_to_sync(smart_proxy, environment, content_view, repository)
-          smart_proxy_helper = ::Katello::SmartProxyHelper.new(smart_proxy)
-          smart_proxy_helper.lifecycle_environment_check(environment, repository)
-
-          if repository
-            [repository]
-          else
-            repositories = smart_proxy_helper.repositories_available_to_capsule(environment, content_view).by_rpm_count
-            puppet_envs = smart_proxy_helper.puppet_environments_available_to_capsule(environment, content_view)
-            repositories + puppet_envs
-          end
-        end
-
         def resource_locks
           :link
         end

data/app/lib/actions/katello/content_view_version/incremental_update.rb

@@ -266,9 +266,8 @@ module Actions
           history.status = ::Katello::ContentViewHistory::SUCCESSFUL
           history.save!
 
-          cvv_yum_repos = version.repositories.yum_type
-          unless cvv_yum_repos.empty? || SmartProxy.pulp_master.pulp3_support?(cvv_yum_repos.first)
-            cvv_yum_repos.each do |repo|
+          unless SmartProxy.pulp_master.pulp3_support?(version.repositories.first)
+            version.repositories.each do |repo|
               SmartProxy.pulp_master.pulp_api.extensions.send(:module_default).
                 copy(repo.library_instance.pulp_id,
                 repo.pulp_id)

data/app/lib/actions/katello/organization/simple_content_access/disable.rb

@@ -0,0 +1,17 @@
+module Actions
+  module Katello
+    module Organization
+      module SimpleContentAccess
+        class Disable < Toggle
+          def content_access_mode_value
+            SIMPLE_CONTENT_ACCESS_DISABLED_VALUE
+          end
+
+          def humanized_name
+            N_("Disable Simple Content Access")
+          end
+        end
+      end
+    end
+  end
+end

data/app/lib/actions/katello/organization/simple_content_access/enable.rb

@@ -0,0 +1,17 @@
+module Actions
+  module Katello
+    module Organization
+      module SimpleContentAccess
+        class Enable < Toggle
+          def content_access_mode_value
+            SIMPLE_CONTENT_ACCESS_ENABLED_VALUE
+          end
+
+          def humanized_name
+            N_("Enable Simple Content Access")
+          end
+        end
+      end
+    end
+  end
+end