foreman_rh_cloud 12.2.0 → 12.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 32fdc4ecba1d5949bce50e394de7cdd097697902c8ba47f47b14e26c96e44881
-  data.tar.gz: 0de08119aa42cce06a73dd7bce46d730e023d40b7f54b621cb6a3cef50fb0b63
+  metadata.gz: b3bc2f9bf0700b7f22d4bee9c0b1f02b10d7b91a441bfa125f38cf2b3b00f655
+  data.tar.gz: 21a3c48d1720f8ffbea0dd00fbf3b660dc6adea2a097254fd4c9a6c85791f49b
 SHA512:
-  metadata.gz: 03ad7ffd1b43db38bdd6762d01b1bd88e7404702a43cd7be626a72892e73f498c990eab3a4cc1037fe63393cdc87d460cd1b87fa7770749e87b9e9a2bf949418
-  data.tar.gz: 403935258065a6cc0755b0d71ed9c3fa6a8652c59264c85da213aa21471d93af5efcde6a829f0e1606d949474b41ab6eaa7b9c573b0ecc74393dce786795f01f
+  metadata.gz: 12b3933a12e68f827cd1a3130be3c4f0907eaa2235c2ac7850a734c7b8fc9e89780d2c87287874d6067337a3679c331d151cf695b5d4e3ad24647434e0a96714
+  data.tar.gz: 278421740888ba600cfd295c43e8eefb649053a4153430f74ff8729e32312f18d93723db1193de8bdc7cb82c96e5c4307405455671035407c978b8d659ab136a

data/app/controllers/api/v2/rh_cloud/advisor_engine_config_controller.rb

@@ -7,7 +7,7 @@ module Api
         api :GET, "/rh_cloud/advisor_engine_config", N_("Show if system is configured to use local iop-advisor-engine.")
         def show
           render json: {
-            use_local_advisor_engine: ForemanRhCloud.with_local_advisor_engine?,
+            use_iop_mode: ForemanRhCloud.with_iop_smart_proxy?,
           }, status: :ok
         end
       end

data/app/controllers/api/v2/rh_cloud/cloud_request_controller.rb

@@ -1,7 +1,10 @@
 module Api::V2::RhCloud
   class CloudRequestController < ::Api::V2::BaseController
+    include ForemanRhCloud::IopSmartProxyAccess
     layout false
 
+    before_action :require_non_iop_smart_proxy, only: [:update]
+
     KNOWN_DIRECTIVES = {
       'playbook-sat' => :handle_run_playbook_request,
       'foreman_rh_cloud' => :handle_run_playbook_request,

data/app/controllers/api/v2/rh_cloud/inventory_controller.rb

@@ -5,6 +5,9 @@ module Api
         include ::Api::Version2
         include InventoryUpload::ReportActions
         include InventoryUpload::TaskActions
+        include ForemanRhCloud::IopSmartProxyAccess
+
+        before_action :require_non_iop_smart_proxy, only: [:enable_cloud_connector]
 
         api :GET, "/organizations/:organization_id/rh_cloud/report", N_("Download latest report")
         param :organization_id, Integer, required: true, desc: N_("Set the current organization context for the request")

data/app/controllers/concerns/foreman_rh_cloud/iop_smart_proxy_access.rb

@@ -0,0 +1,28 @@
+module ForemanRhCloud
+  module IopSmartProxyAccess
+    extend ActiveSupport::Concern
+
+    private
+
+    def require_non_iop_smart_proxy
+      return unless ForemanRhCloud.with_iop_smart_proxy?
+
+      handle_iop_access_error('This feature is not available when using IoP Smart Proxy')
+    end
+
+    def require_iop_smart_proxy
+      return if ForemanRhCloud.with_iop_smart_proxy?
+
+      handle_iop_access_error('This feature is not available without IoP Smart Proxy')
+    end
+
+    def handle_iop_access_error(message)
+      if api_request?
+        not_found(message)
+      else
+        error(message)
+        redirect_back(fallback_location: main_app.root_path)
+      end
+    end
+  end
+end

data/app/controllers/concerns/insights_cloud/package_profile_upload_extensions.rb

@@ -10,7 +10,7 @@ module InsightsCloud
     end
 
     def generate_host_report
-      return unless ForemanRhCloud.with_local_advisor_engine?
+      return unless ForemanRhCloud.with_iop_smart_proxy?
 
       logger.debug("Generating host-specific report for host #{@host.name}")
 

data/app/controllers/foreman_inventory_upload/uploads_controller.rb

@@ -1,6 +1,9 @@
 module ForemanInventoryUpload
   class UploadsController < ::ApplicationController
     include InventoryUpload::ReportActions
+    include ForemanRhCloud::IopSmartProxyAccess
+
+    before_action :require_non_iop_smart_proxy, only: [:enable_cloud_connector]
 
     def last
       label = ForemanInventoryUpload::Async::UploadReportJob.output_label(params[:organization_id])

data/app/controllers/foreman_rh_cloud/foreman_rh_cloud_controller.rb

@@ -0,0 +1,22 @@
+module ForemanRhCloud
+  class ForemanRhCloudController < ApplicationController
+    include ForemanRhCloud::IopSmartProxyAccess
+    layout 'layouts/react_application'
+    skip_before_action :authorize, :only => :index
+
+    before_action :require_iop_smart_proxy, only: [:recommendations]
+
+    def index
+      response.headers['X-Request-Path'] = request.path
+      render("react/index", formats: [:html])
+    end
+
+    def inventory_upload
+      index
+    end
+
+    def recommendations
+      index
+    end
+  end
+end

data/app/controllers/insights_cloud/api/machine_telemetries_controller.rb

@@ -12,6 +12,7 @@ module InsightsCloud::Api
     skip_after_action :log_response_body, :only => [:forward_request]
     skip_before_action :check_media_type, :only => [:forward_request]
     after_action :update_host_insights_status, only: [:forward_request]
+    after_action :update_host_facet, only: [:forward_request], if: -> { ForemanRhCloud.with_iop_smart_proxy? }
 
     # The method that "proxies" requests over to Cloud
     def forward_request
@@ -117,14 +118,27 @@ module InsightsCloud::Api
     end
 
     def update_host_insights_status
-      return unless request.path == '/redhat_access/r/insights/platform/ingress/v1/upload' ||
-                    request.path.include?('/redhat_access/r/insights/uploads/')
-
-      return unless @cloud_response.code.to_s.start_with?('2')
-
+      return unless upload_success?
       # create insights status if it wasn't there in the first place and refresh its reporting date
       @host.get_status(InsightsClientReportStatus).refresh!
       @host.refresh_global_status!
     end
+
+    def update_host_facet
+      return unless upload_success?
+
+      # in IoP case, the hosts are identified by the sub-man ID, and we can assume they already
+      # exist in the local inventory. This will also handle facet creation for new hosts.
+      return if @host.insights
+
+      insights_facet = @host.build_insights(uuid: @host.subscription_facet.uuid)
+      insights_facet.save
+    end
+
+    def upload_success?
+      @cloud_response&.code&.to_s&.start_with?('2') &&
+        (request.path == '/redhat_access/r/insights/platform/ingress/v1/upload' ||
+          request.path.include?('/redhat_access/r/insights/uploads/'))
+    end
   end
 end

data/app/controllers/insights_cloud/ui_requests_controller.rb

@@ -75,6 +75,17 @@ module InsightsCloud
       res.headers[new_header] = header_content
     end
 
+    def translate_insights_host
+      facet = InsightsFacet.find_by(uuid: params[:uuid])
+      if facet.present?
+        Rails.logger.debug "Found InsightsFacet #{params[:uuid]}"
+        redirect_to host_details_page_path(facet.host_id)
+      else
+        Rails.logger.error "Could not find InsightsFacet for #{params[:uuid]}"
+        redirect_to '/page-not-found'
+      end
+    end
+
     private
 
     def ensure_org

data/app/services/foreman_rh_cloud/cert_auth.rb

@@ -11,7 +11,7 @@ module ForemanRhCloud
 
     def execute_cloud_request(params)
       organization = params.delete(:organization)
-      certs = ForemanRhCloud.with_local_advisor_engine? ? foreman_certificate : candlepin_id_cert(organization)
+      certs = ForemanRhCloud.with_iop_smart_proxy? ? foreman_certificate : candlepin_id_cert(organization)
       final_params = {
         ssl_client_cert: OpenSSL::X509::Certificate.new(certs[:cert]),
         ssl_client_key: OpenSSL::PKey.read(certs[:key]),

data/app/services/foreman_rh_cloud/cloud_request.rb

@@ -8,7 +8,7 @@ module ForemanRhCloud
         proxy: ForemanRhCloud.transformed_http_proxy_string,
       }.deep_merge(params)
 
-      if ForemanRhCloud.with_local_advisor_engine?
+      if ForemanRhCloud.with_iop_smart_proxy?
         final_params[:ssl_ca_file] ||= ForemanRhCloud.ca_cert
       end
 

data/app/services/foreman_rh_cloud/cloud_request_forwarder.rb

@@ -37,7 +37,7 @@ module ForemanRhCloud
       requested_url = original_request.original_fullpath.end_with?('/') ? original_request.path + '/' : original_request.path
       params = path_params(requested_url)
 
-      if ForemanRhCloud.with_local_advisor_engine?
+      if ForemanRhCloud.with_iop_smart_proxy?
         params[:ssl_ca_file] = ForemanRhCloud.ca_cert
       end
 

data/app/services/foreman_rh_cloud/hit_remediations_retriever.rb

@@ -2,24 +2,34 @@ module ForemanRhCloud
   class HitRemediationsRetriever < RemediationsRetriever
     def initialize(hit_remediation_pairs, logger: Logger.new(IO::NULL))
       super(logger: logger)
-      @hit_remediation_pairs = hit_remediation_pairs
+      @is_iop = ForemanRhCloud.with_iop_smart_proxy?
+      @hit_remediation_pairs = (hit_remediation_pairs || []).map(&:with_indifferent_access)
       logger.debug("Querying playbook for #{hit_remediation_pairs}")
     end
 
     private
 
     def hit_ids
+      # hit_ids are host ids
       @hit_remediation_pairs.map { |pair| pair["hit_id"] }
     end
 
     def remediation_ids
+      # In IoP, these are Insights rule IDs. For Hosted, they are Foreman database IDs
       @hit_remediation_pairs.map { |pair| pair["resolution_id"] }
     end
 
     def hits
-      @hits ||= Hash[
-        InsightsHit.joins(:insights_facet).where(id: hit_ids).pluck(:id, 'insights_facets.uuid')
-      ]
+      if @is_iop
+        # Return the hit_id unaltered. With IoP, host ids are already translated
+        @hits = Hash[@hit_remediation_pairs.map { |pair| [pair[:hit_id], pair[:hit_id]] }]
+      else
+        # Return a hash which maps Foreman host ID to Insights-flavored ID
+        @hits ||= Hash[
+          InsightsHit.joins(:insights_facet).where(id: hit_ids).pluck(:id, 'insights_facets.uuid')
+        ]
+      end
+      @hits
     end
 
     def pairs_by_remediation_id
@@ -27,11 +37,18 @@ module ForemanRhCloud
     end
 
     def remediations
-      @remediations ||= Hash[
-        InsightsResolution.where(id: remediation_ids).pluck(:id, :resolution_type, :rule_id).map do |id, resolution_type, rule_id|
-          [id, { resolution_type: resolution_type, rule_id: rule_id }]
-        end
-      ]
+      if @is_iop
+        @remediations = Hash[
+          @hit_remediation_pairs.map { |pair| [pair['resolution_id'], { resolution_type: pair['resolution_type'], rule_id: pair['rule_id'] }] }
+        ]
+      else
+        @remediations ||= Hash[
+          InsightsResolution.where(id: remediation_ids).pluck(:id, :resolution_type, :rule_id).map do |id, resolution_type, rule_id|
+            [id, { resolution_type: resolution_type, rule_id: rule_id }]
+          end
+        ]
+      end
+      @remediations
     end
 
     def playbook_request
@@ -65,7 +82,7 @@ module ForemanRhCloud
     end
 
     def organization
-      InsightsHit.find(@hit_remediation_pairs.first['hit_id']).host.organization
+      @is_iop ? Organization.current : InsightsHit.find(@hit_remediation_pairs.first['hit_id']).host.organization
     end
   end
 end

data/app/services/foreman_rh_cloud/url_remediations_retriever.rb

@@ -28,7 +28,7 @@ module ForemanRhCloud
     end
 
     def payload
-      @payload.to_json
+      @payload.present? ? @payload.to_json : @payload # don't run .to_json if @payload is ''
     end
 
     def method

data/app/views/api/v2/hosts/insights/base.rabl

@@ -6,6 +6,6 @@ end
 node :insights_hits_count do |facet|
   facet.hits&.count
 end
-node :use_local_advisor_engine do |_facet|
-  ForemanRhCloud.with_local_advisor_engine?
+node :use_iop_mode do |_facet|
+  ForemanRhCloud.with_iop_smart_proxy?
 end

data/app/views/api/v2/hosts/insights/single.rabl

@@ -1,5 +1,5 @@
 node :vulnerability do
   {
-    enabled: ForemanRhCloud.with_local_advisor_engine?,
+    enabled: ForemanRhCloud.with_iop_smart_proxy?,
   }
 end

data/config/routes.rb

@@ -9,9 +9,7 @@ Rails.application.routes.draw do
     get 'settings', to: 'uploads_settings#index'
     post 'setting', to: 'uploads_settings#set_advanced_setting'
 
-    unless ForemanRhCloud.with_local_advisor_engine?
-      post 'cloud_connector', to: 'uploads#enable_cloud_connector'
-    end
+    post 'cloud_connector', to: 'uploads#enable_cloud_connector'
 
     resources :tasks, only: [:create, :show]
 
@@ -35,13 +33,9 @@ Rails.application.routes.draw do
   end
 
   namespace :foreman_rh_cloud do
-    unless ForemanRhCloud.with_local_advisor_engine?
-      get 'inventory_upload', to: '/react#index'
-    end
-    if ForemanRhCloud.with_local_advisor_engine?
-      get 'recommendations', to: '/react#index'
-      get 'recommendations/:rule_id', to: '/react#index'
-    end
+    get 'inventory_upload', to: 'foreman_rh_cloud#inventory_upload'
+    get 'recommendations', to: 'foreman_rh_cloud#recommendations'
+    get 'recommendations/:rule_id', to: 'foreman_rh_cloud#recommendations'
     get 'insights_cloud', to: '/react#index' # Uses foreman's react controller
     get 'insights_vulnerability', to: '/react#index'
     get 'insights_vulnerability/:cve_id', to: '/react#index'
@@ -61,6 +55,8 @@ Rails.application.routes.draw do
     match '/api/lightspeed/*path', to: 'machine_telemetries#forward_request', via: :all
   end
 
+  get '/insights_hosts/:uuid', to: 'insights_cloud/ui_requests#translate_insights_host'
+
   # API routes
 
   namespace :api, :defaults => { :format => 'json' } do
@@ -77,10 +73,8 @@ Rails.application.routes.draw do
       end
 
       namespace 'rh_cloud' do
-        unless ForemanRhCloud.with_local_advisor_engine?
-          post 'enable_connector', to: 'inventory#enable_cloud_connector'
-          post 'cloud_request', to: 'cloud_request#update'
-        end
+        post 'enable_connector', to: 'inventory#enable_cloud_connector'
+        post 'cloud_request', to: 'cloud_request#update'
         get 'advisor_engine_config', to: 'advisor_engine_config#show'
       end
 

data/lib/foreman_inventory_upload/async/generate_all_reports_job.rb

@@ -13,7 +13,7 @@ module ForemanInventoryUpload
           return
         end
 
-        if ForemanRhCloud.with_local_advisor_engine?
+        if ForemanRhCloud.with_iop_smart_proxy?
           plan_self # so that 'run' runs
         else
           after_delay do
@@ -34,7 +34,7 @@ module ForemanInventoryUpload
       end
 
       def run
-        output[:status] = _('The scheduled process is disabled because this Foreman is configured with the use_local_advisor_engine option.') if ForemanRhCloud.with_local_advisor_engine?
+        output[:status] = _('The scheduled process is disabled because this Foreman is configured with a local IoP Smart Proxy.') if ForemanRhCloud.with_iop_smart_proxy?
       end
 
       def rescue_strategy_for_self

data/lib/foreman_inventory_upload/async/upload_report_job.rb

@@ -60,7 +60,7 @@ module ForemanInventoryUpload
       end
 
       def certificate
-        ForemanRhCloud.with_local_advisor_engine? ? foreman_certificate : manifest_certificate
+        ForemanRhCloud.with_iop_smart_proxy? ? foreman_certificate : manifest_certificate
       end
 
       def manifest_certificate

data/lib/foreman_inventory_upload/generators/slice.rb

@@ -213,6 +213,8 @@ module ForemanInventoryUpload
               first = false
             end
           end
+
+          report_yum_repos(host)
         end
         @stream.simple_field('satellite_managed', true, :last)
       end
@@ -248,6 +250,28 @@ module ForemanInventoryUpload
       def os_release_value(name, version, codename)
         "#{name} #{version} (#{codename})"
       end
+
+      def report_yum_repos(host)
+        return unless host&.content_facet&.bound_repositories&.any?
+        return unless ForemanRhCloud.with_iop_smart_proxy?
+
+        @stream.array_field('yum_repos') do
+          host.content_facet.bound_repositories.each_with_index do |repo, index|
+            report_yum_repo(host.content_source.load_balancer_pulp_content_url, repo)
+            @stream.comma unless index == host.content_facet.bound_repositories.count - 1
+          end
+        end
+      end
+
+      def report_yum_repo(url_prefix, repo)
+        @stream.object do
+          @stream.simple_field('name', repo.content.name)
+          @stream.simple_field('id', repo.content.label)
+          @stream.simple_field('base_url', "#{url_prefix}/#{repo.generate_repo_path(repo.content.content_url)}")
+          @stream.simple_field('enabled', true)
+          @stream.simple_field('gpgcheck', true, :last)
+        end
+      end
     end
   end
 end

data/lib/foreman_rh_cloud/engine.rb

@@ -33,134 +33,7 @@ module ForemanRhCloud
 
     initializer 'foreman_rh_cloud.register_plugin', :before => :finisher_hook do |app|
       app.reloader.to_prepare do
-        Foreman::Plugin.register :foreman_rh_cloud do
-          requires_foreman '>= 3.13'
-          register_gettext
-
-          apipie_documented_controllers ["#{ForemanRhCloud::Engine.root}/app/controllers/api/v2/**/*.rb"]
-
-          settings do
-            category(:rh_cloud, N_('Insights')) do
-              setting('allow_auto_inventory_upload', type: :boolean, description: N_('Enable automatic upload of your host inventory to the Red Hat cloud'), default: true, full_name: N_('Automatic inventory upload'))
-              setting('allow_auto_insights_sync', type: :boolean, description: N_('Enable automatic synchronization of Insights recommendations from the Red Hat cloud'), default: true, full_name: N_('Synchronize recommendations Automatically'))
-              setting('allow_auto_insights_mismatch_delete', type: :boolean, description: N_('Enable automatic deletion of mismatched host records from the Red Hat cloud'), default: false, full_name: N_('Automatic mismatch deletion'))
-              setting('obfuscate_inventory_hostnames', type: :boolean, description: N_('Obfuscate host names sent to the Red Hat cloud. (If insights_minimal_data_collection is set to true, this setting is ignored because host names are not included in the report.)'), default: false, full_name: N_('Obfuscate host names'))
-              setting('obfuscate_inventory_ips', type: :boolean, description: N_('Obfuscate ipv4 addresses sent to the Red Hat cloud.  (If insights_minimal_data_collection is set to true, this setting is ignored because host IPv4 addresses are not included in the report.)'), default: false, full_name: N_('Obfuscate host ipv4 addresses.'))
-              setting('exclude_installed_packages', type: :boolean, description: N_('Exclude installed packages from being uploaded to the Red Hat cloud. (If insights_minimal_data_collection is set to true, this setting is ignored and installed packages are always excluded.)'), default: false, full_name: N_("Exclude installed packages"))
-              setting('include_parameter_tags', type: :boolean, description: N_('Should import include parameter tags from Foreman?'), default: false, full_name: N_('Include parameters in insights-client reports'))
-              setting('rhc_instance_id', type: :string, description: N_('RHC daemon id'), default: nil, full_name: N_('ID of the RHC(Yggdrasil) daemon'))
-              setting('insights_minimal_data_collection', type: :boolean, default: false, full_name: N_('Minimal data collection'), description: N_('Only include the minimum required data in inventory reports for uploading to Red Hat cloud. When this is true, installed packages are excluded from the report regardless of the exclude_installed_packages setting, and host names and IPv4 addresses are excluded from the report regardless of obfuscation settings.'))
-            end
-          end
-
-          # Add permissions
-          security_block :foreman_rh_cloud do
-            permission(
-              :generate_foreman_rh_cloud,
-              'foreman_inventory_upload/reports': [:generate],
-              'foreman_inventory_upload/tasks': [:create],
-              'api/v2/rh_cloud/inventory': [:get_hosts, :remove_hosts, :sync_inventory_status, :download_file, :generate_report, :enable_cloud_connector],
-              'foreman_inventory_upload/uploads': [:enable_cloud_connector],
-              'foreman_inventory_upload/uploads_settings': [:set_advanced_setting],
-              'foreman_inventory_upload/missing_hosts': [:remove_hosts],
-              'insights_cloud/settings': [:update],
-              'insights_cloud/tasks': [:create]
-            )
-            permission(
-              :view_foreman_rh_cloud,
-              'foreman_inventory_upload/accounts': [:index],
-              'foreman_inventory_upload/reports': [:last],
-              'foreman_inventory_upload/uploads': [:auto_upload, :show_auto_upload, :download_file, :last],
-              'foreman_inventory_upload/tasks': [:show],
-              'foreman_inventory_upload/cloud_status': [:index],
-              'foreman_inventory_upload/uploads_settings': [:index],
-              'foreman_inventory_upload/missing_hosts': [:index],
-              'api/v2/rh_cloud/advisor_engine_config': [:show],
-              'react': [:index]
-            )
-            permission(
-              :view_insights_hits,
-              {
-                '/foreman_rh_cloud/insights_cloud': [:index], # for bookmarks and later for showing the page
-                'insights_cloud/hits': [:index, :show, :auto_complete_search, :resolutions],
-                'insights_cloud/settings': [:index, :show],
-                'insights_cloud/ui_requests': [:forward_request],
-                'react': [:index],
-              },
-              :resource_type => ::InsightsHit.name
-            )
-            permission(
-              :dispatch_cloud_requests,
-              'api/v2/rh_cloud/cloud_request': [:update]
-            )
-            permission(
-              :control_organization_insights,
-              'insights_cloud/settings': [:set_org_parameter]
-            )
-          end
-
-          plugin_permissions = [:view_foreman_rh_cloud, :generate_foreman_rh_cloud, :view_insights_hits, :dispatch_cloud_requests, :control_organization_insights]
-
-          role 'ForemanRhCloud', plugin_permissions, 'Role granting permissions to view the hosts inventory,
-                                                      generate a report, upload it to the cloud and download it locally'
-
-          add_permissions_to_default_roles Role::ORG_ADMIN => plugin_permissions,
-            Role::MANAGER => plugin_permissions,
-            Role::SYSTEM_ADMIN => plugin_permissions
-
-          # Adding a top-level menu item
-          sub_menu :top_menu, :insights_menu, caption: N_('Insights'), icon: 'fa fa-cloud', after: :hosts_menu do
-            menu :top_menu,
-              :inventory_upload,
-              caption: N_('Inventory Upload'),
-              url: '/foreman_rh_cloud/inventory_upload',
-              url_hash: { controller: :react, action: :index },
-              parent: :insights_menu
-            menu :top_menu, :insights_hits, caption: N_('Recommendations'), url: '/foreman_rh_cloud/insights_cloud', url_hash: { controller: :react, action: :index }, parent: :insights_menu
-            menu :top_menu,
-              :insights_vulnerability,
-              caption: N_('Vulnerability'),
-              url: '/foreman_rh_cloud/insights_vulnerability',
-              url_hash: { controller: :react, action: :index },
-              parent: :insights_menu,
-              if: -> { ForemanRhCloud.with_local_advisor_engine? }
-          end
-
-          register_facet InsightsFacet, :insights do
-            configure_host do
-              api_view :list => 'api/v2/hosts/insights/insights', :single => 'api/v2/hosts/insights/single'
-              set_dependent_action :destroy
-            end
-          end
-
-          register_global_js_file 'global'
-
-          register_custom_status InventorySync::InventoryStatus
-          register_custom_status InsightsClientReportStatus
-
-          describe_host do
-            overview_buttons_provider :insights_host_overview_buttons
-          end
-
-          extend_page 'hosts/show' do |context|
-            context.add_pagelet :main_tabs,
-              partial: 'hosts/insights_tab',
-              name: _('Insights'),
-              id: 'insights',
-              onlyif: proc { |host| host.insights }
-          end
-
-          extend_page 'hosts/_list' do |context|
-            context.with_profile :cloud, _('RH Cloud'), default: true do
-              add_pagelet :hosts_table_column_header, key: :insights_recommendations_count, label: _('Recommendations'), sortable: true, width: '12%', class: 'hidden-xs ellipsis', priority: 100,
-                          export_data: CsvExporter::ExportDefinition.new(:insights_recommendations_count, callback: ->(host) { host&.insights_hits&.count })
-              add_pagelet :hosts_table_column_content, key: :insights_recommendations_count, callback: ->(host) { hits_counts_cell(host) }, class: 'hidden-xs ellipsis text-center', priority: 100
-            end
-          end
-
-          extend_template_helpers ForemanRhCloud::TemplateRendererHelper
-          allowed_template_helpers :remediations_playbook, :download_rh_playbook
-        end
+        ForemanRhCloud::Plugin.register
 
         ::Katello::UINotifications::Subscriptions::ManifestImportSuccess.include ForemanInventoryUpload::Notifications::ManifestImportSuccessNotificationOverride if defined?(Katello)
 
@@ -248,8 +121,12 @@ module ForemanRhCloud
     end
   end
 
-  def self.with_local_advisor_engine?
-    SETTINGS.dig(:foreman_rh_cloud, :use_local_advisor_engine) || false
+  def self.with_iop_smart_proxy?
+    SmartProxy.with_features('iop').exists?
+  end
+
+  def self.iop_smart_proxy
+    SmartProxy.with_features('iop').first
   end
 
   def self.ca_cert
@@ -264,6 +141,6 @@ module ForemanRhCloud
   end
 
   def self.on_prem_smart_proxy_features
-    ['Insights']
+    ['iop']
   end
 end