foreman_rh_cloud 12.2.1 → 12.2.2

data/test/factories/inventory_upload_factories.rb

@@ -1,87 +1,3 @@
-# redefine katello factories, as long as katello is not compatible with dynamic properties
-FactoryBot.define do
-  factory :katello_organization, :class => "Organization" do
-    type { "Organization" }
-    sequence(:name) { |n| "Organization#{n}" }
-    sequence(:label) { |n| "org#{n}" }
-    sequence(:id) { |n| n }
-
-    trait :acme_corporation do
-      name { "ACME_Corporation" }
-      type { "Organization" }
-      description { "This is the first Organization." }
-      label { "acme_corporation_label" }
-    end
-
-    trait :with_library do
-      association :library, :factory => :katello_library
-    end
-
-    factory :acme_corporation, :traits => [:acme_corporation]
-  end
-end
-
-FactoryBot.define do
-  factory :katello_content_view, :class => Katello::ContentView do
-    sequence(:name) { |n| "Database#{n}" }
-    description { "This content view is for database content" }
-    association :organization, :factory => :katello_organization
-
-    trait :composite do
-      composite { true }
-    end
-  end
-end
-
-FactoryBot.define do
-  factory :katello_content_view_environment, :class => Katello::ContentViewEnvironment do
-    sequence(:name) { |n| "name#{n}" }
-    sequence(:label) { |n| "label#{n}" }
-  end
-end
-
-FactoryBot.define do
-  factory :katello_k_t_environment, :aliases => [:katello_environment], :class => Katello::KTEnvironment do
-    sequence(:name) { |n| "Environment#{n}" }
-    sequence(:label) { |n| "environment#{n}" }
-    association :organization, :factory => :katello_organization
-  end
-end
-
-FactoryBot.define do
-  factory :katello_content_facets, :aliases => [:content_facet], :class => ::Katello::Host::ContentFacet do
-    sequence(:uuid) { |n| "uuid-#{n}" }
-  end
-end
-
-FactoryBot.define do
-  factory :katello_subscription_facets, :aliases => [:subscription_facet], :class => ::Katello::Host::SubscriptionFacet do
-    sequence(:uuid) { |n| "00000000-%<n>04d-%<r>04d-0000-000000000000" % { n: n, r: rand(500) } }
-    facts { { 'memory.memtotal' => "12 GB" } }
-  end
-end
-
-FactoryBot.define do
-  factory :katello_subscription, :class => Katello::Subscription do
-  end
-end
-
-FactoryBot.define do
-  factory :katello_pool, :class => Katello::Pool do
-    active { true }
-    end_date { Date.today + 1.year }
-    cp_id { 1 }
-
-    association :organization, :factory => :katello_organization
-
-    after(:build) do |pool, _evaluator|
-      pool.subscription.organization = pool.organization
-    end
-
-    association :subscription, :factory => :katello_subscription
-  end
-end
-
 FactoryBot.define do
   factory :katello_host_collection_host, :class => Katello::HostCollectionHosts do
     host_id { nil }
@@ -94,34 +10,10 @@ FactoryBot.define do
   end
 end
 
+# Fix Katello factories to return a valid UUID
 FactoryBot.modify do
-  factory :host do
-    transient do
-      content_view { nil }
-      lifecycle_environment { nil }
-      content_source { nil }
-      content_view_environments { [] }
-    end
-
-    trait :with_content do
-      association :content_facet, :factory => :content_facet, :strategy => :build
-
-      after(:build) do |host, evaluator|
-        if host.content_facet
-          if evaluator.content_view && evaluator.lifecycle_environment
-            host.content_facet.assign_single_environment(
-              content_view_id: evaluator.content_view.id,
-              lifecycle_environment_id: evaluator.lifecycle_environment.id
-            )
-          end
-          host.content_facet.content_source = evaluator.content_source if evaluator.content_source
-          host.content_facet.content_view_environments = evaluator.content_view_environments unless evaluator.content_view_environments.empty?
-        end
-      end
-    end
-
-    trait :with_subscription do
-      association :subscription_facet, :factory => :subscription_facet, :strategy => :build
-    end
+  factory :katello_subscription_facets, :aliases => [:subscription_facet], :class => ::Katello::Host::SubscriptionFacet do
+    sequence(:uuid) { |n| "00000000-%<n>04d-%<r>04d-0000-000000000000" % { n: n, r: rand(500) } }
+    facts { { 'memory.memtotal' => "12 GB" } }
   end
 end

data/test/jobs/inventory_scheduled_sync_test.rb

@@ -14,14 +14,14 @@ class InventoryScheduledSyncTest < ActiveSupport::TestCase
     ForemanTasks.sync_task(InventorySync::Async::InventoryScheduledSync)
   end
 
-  test 'Skips execution if with_local_advisor_engine? is true' do
-    ForemanRhCloud.stubs(:with_local_advisor_engine?).returns(true)
+  test 'Skips execution if with_iop_smart_proxy? is true' do
+    ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
 
     InventorySync::Async::InventoryScheduledSync.any_instance.expects(:plan_org_sync).never
 
     task = ForemanTasks.sync_task(InventorySync::Async::InventoryScheduledSync)
     status = task.output[:status].to_s
-    assert_match(/Foreman is configured with the use_local_advisor_engine option/, status)
+    assert_match(/Foreman is configured with a local IoP Smart Proxy/, status)
   end
 
   test 'Skips execution if auto upload is disabled' do

data/test/test_plugin_helper.rb

@@ -2,10 +2,16 @@
 require 'test_helper'
 
 # Add plugin to FactoryBot's paths
-FactoryBot.definition_file_paths << "#{ForemanTasks::Engine.root}/test/factories"
+katello_files = Dir.glob("#{Katello::Engine.root}/test/factories/**/*.rb")
+FactoryBot.definition_file_paths +=
+  katello_files
+  # skip foreman_task factories, since we already have those definitions in ForemanTasks
+  .reject { |f| f =~ /foreman_task|recurring_logic/ }
+  .map { |f| f.gsub('.rb', '') } # .rb extension is will be appended by factorybot
 FactoryBot.definition_file_paths << "#{ForemanRemoteExecution::Engine.root}/test/factories"
+FactoryBot.definition_file_paths << "#{ForemanTasks::Engine.root}/test/factories"
+
 FactoryBot.definition_file_paths << File.join(File.dirname(__FILE__), 'factories')
-# FactoryBot.definition_file_paths << "#{Katello::Engine.root}/test/factories"
 FactoryBot.reload
 
 begin

data/test/unit/rh_cloud_http_proxy_test.rb

@@ -4,7 +4,7 @@ class RhCloudHttpProxyTest < ActiveSupport::TestCase
   setup do
     @global_content_proxy_mock = 'http://global:content@localhost:80'
     @global_foreman_proxy_mock = 'http://global:foreman@localhost:80'
-    ForemanRhCloud.stubs(:with_local_advisor_engine?).returns(false)
+    ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(false)
   end
 
   test 'selects global content proxy' do
@@ -20,8 +20,8 @@ class RhCloudHttpProxyTest < ActiveSupport::TestCase
   end
 
   test 'returns empty string in on-prem setup' do
-    ForemanRhCloud.unstub(:with_local_advisor_engine?)
-    ForemanRhCloud.stubs(:with_local_advisor_engine?).returns(true)
+    ForemanRhCloud.unstub(:with_iop_smart_proxy?)
+    ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
 
     assert_empty ForemanRhCloud.proxy_setting
   end

data/test/unit/services/foreman_rh_cloud/cloud_request_forwarder_test.rb

@@ -22,7 +22,10 @@ class CloudRequestForwarderTest < ActiveSupport::TestCase
       :with_content,
       :with_hostgroup,
       :with_parameter,
-      content_view_environments: [make_cve(lifecycle_environment: env), make_cve(lifecycle_environment: env2)],
+      content_facet: FactoryBot.build(
+        :content_facet,
+        content_view_environments: [make_cve(lifecycle_environment: env), make_cve(lifecycle_environment: env2)]
+      ),
       organization: env.organization
     )
 

data/test/unit/slice_generator_test.rb

@@ -998,6 +998,39 @@ class SliceGeneratorTest < ActiveSupport::TestCase
     assert_not_nil actual_host['insights_id']
   end
 
+  test 'reports yum repos' do
+    ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
+    FactoryBot.create(:katello_content, cp_content_id: '1', organization: @host.organization, name: 'Test Content', label: 'test-content')
+    repo = FactoryBot.build(
+      :katello_repository,
+      :fedora_17_x86_64_dev,
+      :with_content_view,
+      product: FactoryBot.create(:katello_product, :redhat, :with_provider, organization: @host.organization)
+    )
+    # force-save the root repo to avoid validation errors
+    repo.root.save(validate: false)
+    repo.save(validate: false)
+    @host.content_facet.bound_repositories << repo
+    @host.content_facet.content_source = FactoryBot.create(:smart_proxy, :with_pulp3)
+    @host.save(validate: false)
+
+    batch = Host.where(id: @host.id).in_batches.first
+    generator = create_generator(batch)
+    json_str = generator.render
+    actual = JSON.parse(json_str.join("\n"))
+
+    assert_not_nil(actual_host = actual['hosts'].first)
+    assert_not_nil(actual_system_profile = actual_host['system_profile'])
+    assert_not_nil(actual_yum_repos = actual_system_profile['yum_repos'])
+    assert_equal 1, actual_yum_repos.count
+    assert_not_nil(actual_yum_repo = actual_yum_repos.first)
+    assert_equal 'Test Content', actual_yum_repo['name']
+    assert_equal 'test-content', actual_yum_repo['id']
+    assert_match(/fedora_17/, actual_yum_repo['base_url'])
+    assert_equal true, actual_yum_repo['enabled']
+    assert_equal true, actual_yum_repo['gpgcheck']
+  end
+
   private
 
   def create_generator(batch, name = '00000000-0000-0000-0000-000000000000')

data/test/unit/tags_generator_test.rb

@@ -26,7 +26,10 @@ class TagsGeneratorTest < ActiveSupport::TestCase
       organization: env.organization,
       location: @location2,
       hostgroup: @hostgroup2,
-      content_view_environments: [make_cve(lifecycle_environment: env), make_cve(lifecycle_environment: env2)]
+      content_facet: FactoryBot.build(
+        :content_facet,
+        content_view_environments: [make_cve(lifecycle_environment: env), make_cve(lifecycle_environment: env2)]
+      )
     )
 
     @host.organization.pools << FactoryBot.create(:katello_pool, account_number: '1234', cp_id: 1)

data/webpack/CVEsHostDetailsTab/CVEsHostDetailsTab.js

@@ -8,7 +8,7 @@ const CVEsHostDetailsTab = ({ systemId }) => {
   const scope = 'vulnerability';
   const module = './SystemDetailTable';
   return (
-    <div className="rh-cloud-insights-vulnerability-host-details-component">
+    <div className="rh-cloud-insights-vulnerability-host-details-component vulnerability">
       <ScalprumComponent scope={scope} module={module} systemId={systemId} />
     </div>
   );

data/webpack/CveDetailsPage/CveDetailsPage.js

@@ -10,7 +10,7 @@ const CveDetailsPage = () => {
 
   return (
     <ScalprumProvider {...providerOptions}>
-      <div className="rh-cloud-cve-details-page">
+      <div className="rh-cloud-cve-details-page vulnerability">
         <ScalprumComponent scope={scope} module={module} cveId={cveId} />
       </div>
     </ScalprumProvider>

data/webpack/CveDetailsPage/CveDetailsPage.test.js

@@ -18,9 +18,7 @@ jest.mock('@scalprum/react-core', () => ({
 describe('CveDetailsPage component', () => {
   it('renders the container with correct class', () => {
     const { container } = render(<CveDetailsPage />);
-    expect(
-      container.querySelector('.rh-cloud-cve-details-page')
-    ).toBeTruthy();
+    expect(container.querySelector('.rh-cloud-cve-details-page')).toBeTruthy();
   });
 
   it('passes cveId from URL params to ScalprumComponent', () => {

data/webpack/ForemanRhCloudPages.js

@@ -8,6 +8,7 @@ import InsightsCloudSync from './InsightsCloudSync';
 import IopRecommendationDetails from './IopRecommendationDetails/IopRecommendationDetails';
 import InsightsHostDetailsTab from './InsightsHostDetailsTab';
 import CveDetailsPage from './CveDetailsPage';
+import './common/styles.scss';
 
 const pages = [
   { name: 'ForemanInventoryUpload', type: ForemanInventoryUpload },

data/webpack/InsightsCloudSync/Components/RemediationModal/RemediationHelpers.js

@@ -3,7 +3,15 @@ import React from 'react';
 import { orderBy } from 'lodash';
 import Resolutions from './Resolutions';
 
-export const modifyRows = (remediations, setResolutions, setHostsIds) => {
+export const getResolutionId = (selectedResolution, id) =>
+  `${id}_${selectedResolution}`;
+
+export const modifyRows = (
+  remediations,
+  setResolutions,
+  setHostsIds,
+  isIop
+) => {
   if (remediations.length === 0) return [];
 
   const resolutionToSubmit = [];
@@ -15,9 +23,22 @@ export const modifyRows = (remediations, setResolutions, setHostsIds) => {
   ).map(({ id, host_id, hostname, title, resolutions, reboot }) => {
     hostsIdsToSubmit.add(host_id);
     const selectedResolution = resolutions[0]?.id;
+    /* eslint-disable spellcheck/spell-checker */
+
+    // For IoP: {
+    //  hit_id: "c7c6727e-2966-4f7c-87f1-20ef14db7a2d",
+    //  rule_id: "hardening_ssh_client_alive|OPENSSH_HARDENING_CLIENT_ALIVE",
+    //  resolution_type: "less_secure",
+    //  resolution_id:"hardening_ssh_client_alive|OPENSSH_HARDENING_CLIENT_ALIVE_less_secure",
+    // }
+    // for Hosted, hit_id and rule_id will be Foreman database IDs
+
+    /* eslint-enable spellcheck/spell-checker */
     resolutionToSubmit.push({
-      hit_id: id,
-      resolution_id: selectedResolution /** defaults to the first resolution if many */,
+      hit_id: isIop ? host_id : id,
+      rule_id: id,
+      resolution_type: selectedResolution /** defaults to the first resolution if many */,
+      resolution_id: getResolutionId(selectedResolution, id),
     });
     return {
       cells: [
@@ -25,10 +46,11 @@ export const modifyRows = (remediations, setResolutions, setHostsIds) => {
         title,
         <div>
           <Resolutions
-            hit_id={id}
+            hit_id={isIop ? host_id : id}
             resolutions={resolutions}
             setResolutions={setResolutions}
             selectedResolution={selectedResolution}
+            isIop={isIop}
           />
         </div>,
         reboot,

data/webpack/InsightsCloudSync/Components/RemediationModal/RemediationModal.js

@@ -1,5 +1,6 @@
 /* eslint-disable react-hooks/exhaustive-deps */
 import React, { useEffect } from 'react';
+import Immutable from 'seamless-immutable';
 import PropTypes from 'prop-types';
 import {
   Table,
@@ -7,7 +8,7 @@ import {
   TableBody,
 } from '@patternfly/react-table/deprecated';
 import { Modal, ModalVariant, Button } from '@patternfly/react-core';
-import { isEmpty } from 'lodash';
+import { isEmpty, noop } from 'lodash';
 import { STATUS } from 'foremanReact/constants';
 import { translate as __ } from 'foremanReact/common/I18n';
 import { columns } from './RemediationTableConstants';
@@ -15,8 +16,48 @@ import { modifyRows } from './RemediationHelpers';
 import ModalFooter from './RemediationModalFooter';
 import TableEmptyState from '../../../common/table/EmptyState';
 import './RemediationModal.scss';
+import { useAdvisorEngineConfig } from '../../../common/Hooks/ConfigHooks';
+
+/* eslint-disable spellcheck/spell-checker */
+
+// Sample iopData:
+// const iopTestData = Immutable([
+//   {
+//     hostid: 'c7c6727e-2966-4f7c-87f1-20ef14db7a2d',
+//     host_name: 'advisor-test.local',
+//     rulename: 'hardening_cryptopol_krb5|NO_CPOL_KRB5',
+//     resolutions: [
+//       {
+//         description: 'Remove manual crypto-policies',
+//         id: 'fix',
+//         needs_reboot: true,
+//         resolution_risk: 1,
+//       },
+//     ],
+//     rebootable: true,
+//     description: 'Decreased security: krb5 crypto-policies overridden',
+//   },
+//   {
+//     hostid: 'c7c6727e-2966-4f7c-87f1-20ef14db7a2d',
+//     host_name: 'advisor-test.local',
+//     rulename: 'hardening_logging_auditd|HARDENING_LOGGING_5_AUDITD',
+//     resolutions: [
+//       {
+//         description: 'Install and enable auditd',
+//         id: 'fix',
+//         needs_reboot: false,
+//         resolution_risk: 1,
+//       },
+//     ],
+//     rebootable: false,
+//     description: 'Decreased security: auditd not running',
+//   },
+// ]);
+
+/* eslint-enable spellcheck/spell-checker */
 
 const RemediationModal = ({
+  iopData,
   selectedIds,
   fetchRemediations,
   remediations,
@@ -24,31 +65,49 @@ const RemediationModal = ({
   error,
   isAllSelected,
   query,
+  isDisabled,
 }) => {
-  const [rows, setRows] = React.useState([]);
+  const iopRows = Immutable(iopData ?? []).map(recommendation => ({
+    id: recommendation.rulename,
+    host_id: recommendation.hostid,
+    hostname: recommendation.host_name,
+    title: recommendation.description,
+    resolutions: recommendation.resolutions ?? [],
+    reboot: recommendation.rebootable,
+  }));
+
   const [open, setOpen] = React.useState(false);
   const [resolutions, setResolutions] = React.useState([]);
   const [hostsIds, setHostsIds] = React.useState([]);
+  const [rows, setRows] = React.useState([]);
   const toggleModal = () => setOpen(prevValue => !prevValue);
 
+  const isIop = useAdvisorEngineConfig();
   useEffect(() => {
-    if (open) fetchRemediations({ selectedIds, isAllSelected, query });
+    // only fetch for Hosted. IoP provides via props.
+    if (!isIop && open)
+      fetchRemediations({ selectedIds, isAllSelected, query });
   }, [open]);
 
   useEffect(() => {
-    const modifiedRows =
-      status === STATUS.PENDING
-        ? []
-        : modifyRows(remediations, setResolutions, setHostsIds);
+    let modifiedRows;
+    if (isIop) {
+      modifiedRows = modifyRows(iopRows, setResolutions, setHostsIds, true);
+    } else {
+      modifiedRows =
+        status === STATUS.PENDING
+          ? []
+          : modifyRows(remediations, setResolutions, setHostsIds, false);
+    }
     setRows(modifiedRows);
-  }, [remediations, status]);
+  }, [remediations, status, iopData, isIop]);
 
   return (
     <React.Fragment>
       <Button
         ouiaId="button-remediate"
         variant="primary"
-        isDisabled={isEmpty(selectedIds)}
+        isDisabled={isDisabled || isEmpty(selectedIds)}
         onClick={() => {
           toggleModal();
         }}
@@ -68,6 +127,7 @@ const RemediationModal = ({
             toggleModal={toggleModal}
             resolutions={resolutions}
             hostsIds={hostsIds}
+            isIop={isIop}
           />
         }
       >
@@ -92,22 +152,36 @@ const RemediationModal = ({
 };
 
 RemediationModal.propTypes = {
-  selectedIds: PropTypes.object,
-  fetchRemediations: PropTypes.func.isRequired,
+  iopData: PropTypes.arrayOf(
+    PropTypes.shape({
+      hostid: PropTypes.string,
+      host_name: PropTypes.string,
+      rulename: PropTypes.string,
+      resolutions: PropTypes.array,
+      rebootable: PropTypes.bool,
+      description: PropTypes.string,
+    })
+  ),
+  selectedIds: PropTypes.shape({}),
+  fetchRemediations: PropTypes.func,
   remediations: PropTypes.array,
   status: PropTypes.string,
   error: PropTypes.string,
   isAllSelected: PropTypes.bool,
   query: PropTypes.string,
+  isDisabled: PropTypes.bool,
 };
 
 RemediationModal.defaultProps = {
+  iopData: null,
   selectedIds: {},
+  fetchRemediations: noop,
   remediations: [],
   status: null,
   error: null,
   isAllSelected: false,
   query: null,
+  isDisabled: false,
 };
 
 export default RemediationModal;

data/webpack/InsightsCloudSync/Components/RemediationModal/RemediationModalFooter.js

@@ -2,18 +2,44 @@ import React from 'react';
 import PropTypes from 'prop-types';
 import { Button } from '@patternfly/react-core';
 import { translate as __ } from 'foremanReact/common/I18n';
+import { useBulkSelect } from 'foremanReact/components/PF4/TableIndexPage/Table/TableHooks';
 import { JOB_INVOCATION_PATH } from './RemediationTableConstants';
 
-const ModalFooter = ({ toggleModal, resolutions, hostsIds }) => {
+const ModalFooter = ({ toggleModal, resolutions, hostsIds, isIop }) => {
   let token = document.querySelector('meta[name="csrf-token"]');
   token = token?.content || '';
+
+  const [jobInProgress, setJobInProgress] = React.useState(false);
+  const formRef = React.useRef(null);
+
+  const { fetchBulkParams } = useBulkSelect({
+    initialArry: hostsIds,
+    idColumn: 'insights_uuid',
+  });
+
+  const handleSubmit = e => {
+    e.preventDefault();
+    setJobInProgress(true);
+
+    setTimeout(() => {
+      // eslint-disable-next-line no-unused-expressions
+      formRef.current?.submit?.();
+    }, 100);
+  };
   return (
-    <form action={JOB_INVOCATION_PATH} method="post">
+    <form
+      action={JOB_INVOCATION_PATH}
+      method="post"
+      ref={formRef}
+      onSubmit={handleSubmit}
+    >
       <Button
         type="submit"
         ouiaId="button-confirm"
         key="confirm"
         variant="primary"
+        isDisabled={jobInProgress}
+        isLoading={jobInProgress}
       >
         {__('Remediate')}
       </Button>
@@ -32,9 +58,15 @@ const ModalFooter = ({ toggleModal, resolutions, hostsIds }) => {
         name="inputs[hit_remediation_pairs]"
         value={JSON.stringify(resolutions)}
       />
-      {hostsIds.map(id => (
-        <input type="hidden" name="host_ids[]" key={id} value={id} />
-      ))}
+      {!isIop &&
+        hostsIds.map(id => (
+          <input type="hidden" name="host_ids[]" key={id} value={id} />
+        ))}
+      {isIop && (
+        <>
+          <input type="hidden" name="search" value={fetchBulkParams()} />
+        </>
+      )}
     </form>
   );
 };
@@ -43,11 +75,13 @@ ModalFooter.propTypes = {
   toggleModal: PropTypes.func.isRequired,
   resolutions: PropTypes.array,
   hostsIds: PropTypes.array,
+  isIop: PropTypes.bool,
 };
 
 ModalFooter.defaultProps = {
   resolutions: [],
   hostsIds: [],
+  isIop: false,
 };
 
 export default ModalFooter;

data/webpack/InsightsCloudSync/Components/RemediationModal/Resolutions.js

@@ -2,12 +2,14 @@
 import React from 'react';
 import PropTypes from 'prop-types';
 import { Radio } from '@patternfly/react-core';
+import { getResolutionId } from './RemediationHelpers';
 
 const Resolutions = ({
   resolutions,
   setResolutions,
   selectedResolution,
   hit_id,
+  isIop,
 }) => {
   const [checkedID, setCheckedID] = React.useState(selectedResolution);
 
@@ -27,6 +29,15 @@ const Resolutions = ({
               stateRes.map(res => {
                 if (hit_id === res.hit_id) {
                   setCheckedID(resolution_id);
+                  if (isIop)
+                    return {
+                      ...res,
+                      resolution_id: getResolutionId(
+                        resolution_id,
+                        res.rule_id
+                      ),
+                      resolution_type: resolution_id,
+                    };
                   return { ...res, resolution_id };
                 }
                 return res;
@@ -45,12 +56,14 @@ Resolutions.propTypes = {
   resolutions: PropTypes.array,
   hit_id: PropTypes.number,
   selectedResolution: PropTypes.number,
+  isIop: PropTypes.bool,
 };
 
 Resolutions.defaultProps = {
   resolutions: [],
   hit_id: null,
   selectedResolution: null,
+  isIop: false,
 };
 
 export default Resolutions;

data/webpack/InsightsCloudSync/InsightsCloudSync.js

@@ -67,13 +67,15 @@ export const generateRuleUrl = ruleId =>
   foremanUrl(`/foreman_rh_cloud/recommendations/${ruleId}`);
 
 const IopRecommendationsPage = props => (
-  <ScalprumComponent
-    scope={scope}
-    module={module}
-    IopRemediationModal={RemediationModal}
-    generateRuleUrl={generateRuleUrl}
-    {...props}
-  />
+  <div className="advisor">
+    <ScalprumComponent
+      scope={scope}
+      module={module}
+      IopRemediationModal={RemediationModal}
+      generateRuleUrl={generateRuleUrl}
+      {...props}
+    />
+  </div>
 );
 
 const IopRecommendationsPageWrapped = props => (