foreman_rh_cloud 12.2.17 → 13.0.0

data/test/jobs/insights_resolutions_sync_test.rb

@@ -2,7 +2,7 @@ require 'test_plugin_helper'
 require 'foreman_tasks/test_helpers'
 
 class InsightsResolutionsSyncTest < ActiveSupport::TestCase
-  include Dynflow::Testing::Factories
+  include ForemanTasks::TestHelpers::WithInThreadExecutor
   include MockCerts
 
   setup do
@@ -76,8 +76,7 @@ class InsightsResolutionsSyncTest < ActiveSupport::TestCase
     Katello::UpstreamConnectionChecker.any_instance.stubs(:can_connect?).returns(true)
     InsightsCloud::Async::InsightsResolutionsSync.any_instance.stubs(:query_insights_resolutions).returns(@resolutions)
 
-    action = create_and_plan_action(InsightsCloud::Async::InsightsResolutionsSync)
-    run_action(action)
+    ForemanTasks.sync_task(InsightsCloud::Async::InsightsResolutionsSync)
     @rule.reload
 
     assert_equal 5, InsightsResolution.all.count
@@ -89,8 +88,7 @@ class InsightsResolutionsSyncTest < ActiveSupport::TestCase
     Katello::UpstreamConnectionChecker.any_instance.stubs(:can_connect?).returns(false)
     InsightsCloud::Async::InsightsResolutionsSync.any_instance.expects(:query_insights_resolutions).never
 
-    action = create_and_plan_action(InsightsCloud::Async::InsightsResolutionsSync)
-    run_action(action)
+    ForemanTasks.sync_task(InsightsCloud::Async::InsightsResolutionsSync)
 
     assert_equal 0, InsightsResolution.all.count
   end
@@ -100,8 +98,7 @@ class InsightsResolutionsSyncTest < ActiveSupport::TestCase
     Katello::UpstreamConnectionChecker.any_instance.stubs(:can_connect?).returns(true)
     InsightsCloud::Async::InsightsResolutionsSync.any_instance.expects(:query_insights_resolutions).never
 
-    action = create_and_plan_action(InsightsCloud::Async::InsightsResolutionsSync)
-    run_action(action)
+    ForemanTasks.sync_task(InsightsCloud::Async::InsightsResolutionsSync)
 
     assert_equal 0, InsightsResolution.all.count
   end
@@ -111,8 +108,7 @@ class InsightsResolutionsSyncTest < ActiveSupport::TestCase
     InsightsCloud::Async::InsightsResolutionsSync.any_instance.expects(:query_insights_resolutions).never
     InsightsRule.all.delete_all
 
-    action = create_and_plan_action(InsightsCloud::Async::InsightsResolutionsSync)
-    run_action(action)
+    ForemanTasks.sync_task(InsightsCloud::Async::InsightsResolutionsSync)
 
     assert_equal 0, InsightsResolution.all.count
   end

data/test/jobs/insights_rules_sync_test.rb

@@ -2,7 +2,7 @@ require 'test_plugin_helper'
 require 'foreman_tasks/test_helpers'
 
 class InsightsRulesSyncTest < ActiveSupport::TestCase
-  include Dynflow::Testing::Factories
+  include ForemanTasks::TestHelpers::WithInThreadExecutor
 
   setup do
     rules_json = <<-'RULES_JSON'
@@ -119,8 +119,7 @@ class InsightsRulesSyncTest < ActiveSupport::TestCase
     # do not cleanup unused rules for tests
     InsightsCloud::Async::InsightsRulesSync.any_instance.stubs(:cleanup_rules)
 
-    action = create_and_plan_action(InsightsCloud::Async::InsightsRulesSync, Organization.all)
-    run_action(action)
+    ForemanTasks.sync_task(InsightsCloud::Async::InsightsRulesSync, Organization.all)
     @hit.reload
 
     assert_equal 2, InsightsRule.all.count
@@ -202,8 +201,7 @@ class InsightsRulesSyncTest < ActiveSupport::TestCase
     # do not cleanup unused rules for tests
     InsightsCloud::Async::InsightsRulesSync.any_instance.stubs(:cleanup_rules)
 
-    action = create_and_plan_action(InsightsCloud::Async::InsightsRulesSync, Organization.all)
-    run_action(action)
+    ForemanTasks.sync_task(InsightsCloud::Async::InsightsRulesSync, Organization.all)
 
     assert_equal 3, InsightsRule.all.count
   end

data/test/jobs/inventory_full_sync_test.rb

@@ -2,7 +2,7 @@ require 'test_plugin_helper'
 require 'foreman_tasks/test_helpers'
 
 class InventoryFullSyncTest < ActiveSupport::TestCase
-  include Dynflow::Testing::Factories
+  include ForemanTasks::TestHelpers::WithInThreadExecutor
   include MockCerts
   include KatelloCVEHelper
 
@@ -265,8 +265,7 @@ class InventoryFullSyncTest < ActiveSupport::TestCase
 
     InventorySync::Async::InventoryFullSync.any_instance.expects(:query_inventory).returns(@inventory)
 
-    action = create_and_plan_action(InventorySync::Async::InventoryFullSync, @host2.organization)
-    run_action(action)
+    ForemanTasks.sync_task(InventorySync::Async::InventoryFullSync, @host2.organization)
 
     @host2.reload
 
@@ -281,8 +280,7 @@ class InventoryFullSyncTest < ActiveSupport::TestCase
     InventorySync::Async::InventoryFullSync.any_instance.expects(:query_inventory).returns(@inventory)
     FactoryBot.create(:fact_value, fact_name: fact_names['virt::uuid'], value: '1234', host: @host2)
 
-    action = create_and_plan_action(InventorySync::Async::InventoryFullSync, @host1.organization)
-    run_action(action)
+    ForemanTasks.sync_task(InventorySync::Async::InventoryFullSync, @host1.organization)
     @host2.reload
 
     assert_equal InventorySync::InventoryStatus::DISCONNECT, InventorySync::InventoryStatus.where(host_id: @host1.id).first.status
@@ -293,8 +291,7 @@ class InventoryFullSyncTest < ActiveSupport::TestCase
 
     InventorySync::Async::InventoryFullSync.any_instance.expects(:plan_self).never
 
-    action = create_and_plan_action(InventorySync::Async::InventoryFullSync, @host1.organization)
-    run_action(action)
+    ForemanTasks.sync_task(InventorySync::Async::InventoryFullSync, @host1.organization)
   end
 
   test 'Should skip hosts that are not returned in query' do
@@ -307,8 +304,7 @@ class InventoryFullSyncTest < ActiveSupport::TestCase
     InventorySync::Async::InventoryFullSync.any_instance.expects(:affected_host_ids).returns([@host1.id, @host2.id])
     FactoryBot.create(:fact_value, fact_name: fact_names['virt::uuid'], value: '1234', host: @host2)
 
-    action = create_and_plan_action(InventorySync::Async::InventoryFullSync, @host1.organization)
-    run_action(action)
+    ForemanTasks.sync_task(InventorySync::Async::InventoryFullSync, @host1.organization)
     @host2.reload
 
     assert_nil InventorySync::InventoryStatus.where(host_id: @host3.id).first

data/test/jobs/inventory_hosts_sync_test.rb

@@ -2,7 +2,7 @@ require 'test_plugin_helper'
 require 'foreman_tasks/test_helpers'
 
 class InventoryHostsSyncTest < ActiveSupport::TestCase
-  include Dynflow::Testing::Factories
+  include ForemanTasks::TestHelpers::WithInThreadExecutor
   include MockCerts
   include KatelloCVEHelper
 
@@ -302,8 +302,7 @@ class InventoryHostsSyncTest < ActiveSupport::TestCase
 
     @host2.build_insights.save
 
-    action = create_and_plan_action(InventorySync::Async::InventoryHostsSync, [@host1.organization, @host2.organization])
-    run_action(action)
+    ForemanTasks.sync_task(InventorySync::Async::InventoryHostsSync, [@host1.organization, @host2.organization])
 
     @host2.reload
 
@@ -318,8 +317,7 @@ class InventoryHostsSyncTest < ActiveSupport::TestCase
       InventorySync::Async::InventoryHostsSync.any_instance.stubs(:candlepin_id_cert)
     end
 
-    action = create_and_plan_action(InventorySync::Async::InventoryHostsSync, [@host1.organization, @host2.organization])
-    run_action(action)
+    ForemanTasks.sync_task(InventorySync::Async::InventoryHostsSync, [@host1.organization, @host2.organization])
 
     @host2.reload
 
@@ -338,8 +336,7 @@ class InventoryHostsSyncTest < ActiveSupport::TestCase
 
     assert_nil @host2.insights
 
-    action = create_and_plan_action(InventorySync::Async::InventoryHostsSync, [@host1.organization, @host2.organization])
-    run_action(action)
+    ForemanTasks.sync_task(InventorySync::Async::InventoryHostsSync, [@host1.organization, @host2.organization])
 
     @host2.reload
 
@@ -356,8 +353,7 @@ class InventoryHostsSyncTest < ActiveSupport::TestCase
       InventorySync::Async::InventoryHostsSync.any_instance.stubs(:candlepin_id_cert)
     end
 
-    action = create_and_plan_action(InventorySync::Async::InventoryHostsSync, [org])
-    run_action(action)
+    ForemanTasks.sync_task(InventorySync::Async::InventoryHostsSync, [org])
 
     assert_equal 3, InsightsMissingHost.count
   end
@@ -374,8 +370,7 @@ class InventoryHostsSyncTest < ActiveSupport::TestCase
       InventorySync::Async::InventoryHostsSync.any_instance.stubs(:candlepin_id_cert)
     end
 
-    action = create_and_plan_action(InventorySync::Async::InventoryHostsSync, [org])
-    run_action(action)
+    ForemanTasks.sync_task(InventorySync::Async::InventoryHostsSync, [org])
 
     assert_equal 3, InsightsMissingHost.count
   end

data/test/jobs/inventory_scheduled_sync_test.rb

@@ -2,7 +2,7 @@ require 'test_plugin_helper'
 require 'foreman_tasks/test_helpers'
 
 class InventoryScheduledSyncTest < ActiveSupport::TestCase
-  include Dynflow::Testing::Factories
+  include ForemanTasks::TestHelpers::WithInThreadExecutor
 
   test 'Schedules an execution if auto upload is enabled' do
     Setting[:allow_auto_inventory_upload] = true
@@ -11,8 +11,7 @@ class InventoryScheduledSyncTest < ActiveSupport::TestCase
     InventorySync::Async::InventoryScheduledSync.any_instance.expects(:plan_org_sync).times(Organization.unscoped.count)
     InventorySync::Async::InventoryScheduledSync.any_instance.expects(:plan_remove_insights_hosts).times(Organization.unscoped.count)
 
-    action = create_and_plan_action(InventorySync::Async::InventoryScheduledSync)
-    run_action(action)
+    ForemanTasks.sync_task(InventorySync::Async::InventoryScheduledSync)
   end
 
   test 'Skips execution if with_iop_smart_proxy? is true' do
@@ -20,9 +19,8 @@ class InventoryScheduledSyncTest < ActiveSupport::TestCase
 
     InventorySync::Async::InventoryScheduledSync.any_instance.expects(:plan_org_sync).never
 
-    action = create_and_plan_action(InventorySync::Async::InventoryScheduledSync)
-    action = run_action(action)
-    status = action.output[:status].to_s
+    task = ForemanTasks.sync_task(InventorySync::Async::InventoryScheduledSync)
+    status = task.output[:status].to_s
     assert_match(/Foreman is configured with a local IoP Smart Proxy/, status)
   end
 
@@ -31,8 +29,7 @@ class InventoryScheduledSyncTest < ActiveSupport::TestCase
 
     InventorySync::Async::InventoryScheduledSync.any_instance.expects(:plan_org_sync).never
 
-    action = create_and_plan_action(InventorySync::Async::InventoryScheduledSync)
-    run_action(action)
+    ForemanTasks.sync_task(InventorySync::Async::InventoryScheduledSync)
   end
 
   test 'Skips mismatch deletion if the setting is disabled' do
@@ -42,7 +39,6 @@ class InventoryScheduledSyncTest < ActiveSupport::TestCase
     InventorySync::Async::InventoryScheduledSync.any_instance.expects(:plan_org_sync).times(Organization.unscoped.count)
     InventorySync::Async::InventoryScheduledSync.any_instance.expects(:plan_remove_insights_hosts).never
 
-    action = create_and_plan_action(InventorySync::Async::InventoryScheduledSync)
-    run_action(action)
+    ForemanTasks.sync_task(InventorySync::Async::InventoryScheduledSync)
   end
 end

data/test/jobs/inventory_self_host_sync_test.rb

@@ -2,7 +2,7 @@ require 'test_plugin_helper'
 require 'foreman_tasks/test_helpers'
 
 class InventorySelfHostSyncTest < ActiveSupport::TestCase
-  include Dynflow::Testing::Factories
+  include ForemanTasks::TestHelpers::WithInThreadExecutor
   include MockCerts
 
   setup do

data/test/jobs/remove_insights_hosts_job_test.rb

@@ -2,7 +2,7 @@ require 'test_plugin_helper'
 require 'foreman_tasks/test_helpers'
 
 class RemoveInsightsHostJobTest < ActiveSupport::TestCase
-  include Dynflow::Testing::Factories
+  include ForemanTasks::TestHelpers::WithInThreadExecutor
 
   setup do
     User.current = User.find_by(login: 'secret_admin')
@@ -18,10 +18,10 @@ class RemoveInsightsHostJobTest < ActiveSupport::TestCase
 
     FactoryBot.create(:insights_missing_host, organization: @org)
 
-    action = create_and_plan_action(ForemanInventoryUpload::Async::RemoveInsightsHostsJob, '', @org.id)
-    run_action(action)
+    task = ForemanTasks.sync_task(ForemanInventoryUpload::Async::RemoveInsightsHostsJob, '', @org.id)
 
     assert_equal 0, InsightsMissingHost.count
+    assert_equal 'success', task.result
   end
 
   test 'Does not delete hosts on cloud failure' do
@@ -31,13 +31,14 @@ class RemoveInsightsHostJobTest < ActiveSupport::TestCase
 
     FactoryBot.create(:insights_missing_host, organization: @org)
 
-    action = create_and_plan_action(ForemanInventoryUpload::Async::RemoveInsightsHostsJob, '', @org.id)
-
-    assert_raises(StandardError) do
-      run_action(action)
+    begin
+      ForemanTasks.sync_task(ForemanInventoryUpload::Async::RemoveInsightsHostsJob, '', @org.id)
+    rescue ForemanTasks::TaskError => ex
+      task = ex.task
     end
 
     assert_equal 1, InsightsMissingHost.count
+    assert_equal 'error', task.result
   end
 
   test 'Paginates the hosts list' do
@@ -53,12 +54,12 @@ class RemoveInsightsHostJobTest < ActiveSupport::TestCase
     FactoryBot.create(:insights_missing_host, organization: @org)
     FactoryBot.create(:insights_missing_host, organization: @org)
 
-    action = create_and_plan_action(ForemanInventoryUpload::Async::RemoveInsightsHostsJob, '', @org.id)
-    action = run_action(action)
+    task = ForemanTasks.sync_task(ForemanInventoryUpload::Async::RemoveInsightsHostsJob, '', @org.id)
 
     assert_equal 0, InsightsMissingHost.count
-    assert_equal 'response1', action.output[:response_page1]
-    assert_equal 'response2', action.output[:response_page2]
+    assert_equal 'success', task.result
+    assert_equal 'response1', task.output[:response_page1]
+    assert_equal 'response2', task.output[:response_page2]
   end
 
   test 'Uses scoped_search to select hosts' do
@@ -72,12 +73,12 @@ class RemoveInsightsHostJobTest < ActiveSupport::TestCase
     FactoryBot.create(:insights_missing_host, name: 'test a', organization: @org)
     FactoryBot.create(:insights_missing_host, name: 'test b', organization: @org)
 
-    action = create_and_plan_action(ForemanInventoryUpload::Async::RemoveInsightsHostsJob, 'name ~ b', @org.id)
-    action = run_action(action)
+    task = ForemanTasks.sync_task(ForemanInventoryUpload::Async::RemoveInsightsHostsJob, 'name ~ b', @org.id)
 
     assert_equal 1, InsightsMissingHost.count
     assert_equal 'test a', InsightsMissingHost.first.name
-    assert_equal 'response1', action.output[:response_page1]
+    assert_equal 'success', task.result
+    assert_equal 'response1', task.output[:response_page1]
   end
 
   def mock_response(code: 200, body: '')

data/test/jobs/upload_report_job_test.rb

@@ -0,0 +1,36 @@
+require 'test_plugin_helper'
+require 'foreman_tasks/test_helpers'
+
+class UploadReportJobTest < ActiveSupport::TestCase
+  include ForemanTasks::TestHelpers::WithInThreadExecutor
+  include FolderIsolation
+
+  test 'returns aborted state when disconnected' do
+    organization = FactoryBot.create(:organization)
+    Organization.any_instance.stubs(:owner_details).returns(
+      'upstreamConsumer' => {
+        'idCert' => 'TEST_CERT',
+      }
+    )
+    ForemanInventoryUpload::Async::UploadReportJob.any_instance.expects(:content_disconnected?).returns(true)
+
+    ForemanTasks.sync_task(ForemanInventoryUpload::Async::UploadReportJob, '', organization.id)
+
+    label = ForemanInventoryUpload::Async::UploadReportJob.output_label(organization.id)
+    progress_output = ForemanInventoryUpload::Async::ProgressOutput.get(label)
+    assert_match(/Upload canceled/, progress_output.full_output)
+    assert_match(/exit 1/, progress_output.status)
+  end
+
+  test 'returns aborted state when no certificate defined on organization' do
+    organization = FactoryBot.create(:organization)
+    Organization.any_instance.expects(:owner_details).returns(nil)
+
+    ForemanTasks.sync_task(ForemanInventoryUpload::Async::UploadReportJob, '', organization.id)
+
+    label = ForemanInventoryUpload::Async::UploadReportJob.output_label(organization.id)
+    progress_output = ForemanInventoryUpload::Async::ProgressOutput.get(label)
+    assert_match(/Skipping organization/, progress_output.full_output)
+    assert_match(/exit 1/, progress_output.status)
+  end
+end

data/test/unit/fact_helpers_test.rb

@@ -313,51 +313,4 @@ class FactHelpersTest < ActiveSupport::TestCase
       assert_equal '10.230.230.3', ip3
     end
   end
-
-  describe 'IoP smart proxy checks' do
-    test 'obfuscate_hostname? returns false when global setting is enabled but IoP is present' do
-      Setting.expects(:[]).with(:obfuscate_inventory_hostnames).returns(true)
-      ForemanRhCloud.expects(:with_iop_smart_proxy?).returns(true)
-      host = mock('host')
-      # When IoP is present, it falls back to checking host-specific facts
-      @instance.expects(:fact_value).with(host, 'insights_client::obfuscate_hostname_enabled').returns(nil)
-
-      result = @instance.obfuscate_hostname?(host)
-
-      refute result
-    end
-
-    test 'obfuscate_hostname? returns true when global setting is enabled and IoP is not present' do
-      Setting.expects(:[]).with(:obfuscate_inventory_hostnames).returns(true)
-      ForemanRhCloud.expects(:with_iop_smart_proxy?).returns(false)
-      host = mock('host')
-
-      result = @instance.obfuscate_hostname?(host)
-
-      assert result
-    end
-
-    test 'obfuscate_ips? returns false when global setting is enabled but IoP is present' do
-      Setting.expects(:[]).with(:obfuscate_inventory_ips).returns(true)
-      ForemanRhCloud.expects(:with_iop_smart_proxy?).returns(true)
-      host = mock('host')
-      # When IoP is present, it falls back to checking host-specific facts
-      @instance.expects(:fact_value).with(host, 'insights_client::obfuscate_ipv4_enabled').returns(nil)
-      @instance.expects(:fact_value).with(host, 'insights_client::obfuscate_ipv6_enabled').returns(nil)
-
-      result = @instance.obfuscate_ips?(host)
-
-      refute result
-    end
-
-    test 'obfuscate_ips? returns true when global setting is enabled and IoP is not present' do
-      Setting.expects(:[]).with(:obfuscate_inventory_ips).returns(true)
-      ForemanRhCloud.expects(:with_iop_smart_proxy?).returns(false)
-      host = mock('host')
-
-      result = @instance.obfuscate_ips?(host)
-
-      assert result
-    end
-  end
 end

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

@@ -40,18 +40,4 @@ class TagsAuthTest < ActiveSupport::TestCase
 
     @auth.update_tag
   end
-
-  test 'Generates tags with wildcard location when location is nil' do
-    auth_with_nil_loc = ::ForemanRhCloud::TagsAuth.new(@user, @org, nil, @logger)
-    uuid1 = 'test_uuid1'
-
-    auth_with_nil_loc.expects(:allowed_hosts).returns([uuid1])
-    auth_with_nil_loc.expects(:execute_cloud_request).with do |actual_params|
-      actual = JSON.parse(actual_params[:payload])
-      assert_includes actual['host_id_list'], uuid1
-      assert_equal "U:\"#{@user.login}\"O:\"#{@org.name}\"L:\"*\"", actual['tags'].first['value']
-    end
-
-    auth_with_nil_loc.update_tag
-  end
 end

data/test/unit/slice_generator_test.rb

@@ -107,36 +107,6 @@ class SliceGeneratorTest < ActiveSupport::TestCase
     assert_equal 'test_nic1', actual_nic['name']
   end
 
-  test 'does not generate a report with minimal data collection when iop is present' do
-    Setting[:insights_minimal_data_collection] = true
-    ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-
-    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_equal '00000000-0000-0000-0000-000000000000', actual['report_slice_id']
-    assert_not_nil(actual_host = actual['hosts'].first)
-    assert_nil actual_host['ip_addresses']
-    assert_nil actual_host['mac_addresses']
-    assert_equal @host.fqdn, actual_host['fqdn']
-    assert_equal '1234', actual_host['account']
-    assert_equal 1, generator.hosts_count
-    assert_not_nil(actual_system_profile = actual_host['system_profile'])
-    assert_nil actual_system_profile['number_of_cpus']
-    assert_nil actual_system_profile['number_of_sockets']
-    assert_nil actual_system_profile['cores_per_socket']
-    assert_nil actual_system_profile['system_memory_bytes']
-    assert_nil actual_system_profile['os_release']
-    assert_not_nil(actual_network_interfaces = actual_system_profile['network_interfaces'])
-    assert_not_nil(actual_nic = actual_network_interfaces.first)
-    refute actual_nic.key?('mtu')
-    refute actual_nic.key?('mac_address')
-    assert_equal 'test_nic1', actual_nic['name']
-  end
-
   test 'generates a report with minimal data collection' do
     Setting[:insights_minimal_data_collection] = true
     create_fact_values(@host,
@@ -960,33 +930,6 @@ class SliceGeneratorTest < ActiveSupport::TestCase
     assert_equal 'alibaba', actual_profile['cloud_provider']
   end
 
-  test 'do not exclude packages when iop is present' do
-    Setting[:exclude_installed_packages] = true
-    ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
-    installed_package = ::Katello::InstalledPackage.create(name: 'test-package', nvrea: 'test-package-1.0.x86_64', nvra: 'test-package-1.0.x86_64')
-
-    another_host = FactoryBot.create(
-      :host,
-      :with_subscription,
-      :with_content,
-      content_view: @host.content_views.first,
-      lifecycle_environment: @host.lifecycle_environments.first,
-      organization: @host.organization,
-      installed_packages: [installed_package]
-    )
-
-    batch = Host.where(id: another_host.id).in_batches.first
-    generator = create_generator(batch)
-
-    json_str = generator.render
-    actual = JSON.parse(json_str.join("\n"))
-
-    assert_equal '00000000-0000-0000-0000-000000000000', actual['report_slice_id']
-    assert_not_nil(actual_host = actual['hosts'].first)
-    assert_not_nil(actual_profile = actual_host['system_profile'])
-    assert_not_nil(actual_profile['installed_packages'])
-  end
-
   test 'include packages installed in the report' do
     Setting[:exclude_installed_packages] = false
     installed_package = ::Katello::InstalledPackage.create(name: 'test-package', nvrea: 'test-package-1.0.x86_64', nvra: 'test-package-1.0.x86_64')

data/webpack/ForemanColumnExtensions/index.js

@@ -79,8 +79,6 @@ const hostsIndexColumnExtensions = [
     categoryName: insightsCategoryName,
     categoryKey: 'insights',
     isSorted: false,
-    // eslint-disable-next-line camelcase
-    isRelevant: contextData => contextData?.metadata?.foreman_rh_cloud?.iop,
   },
 ];
 

data/webpack/ForemanInventoryUpload/Components/PageHeader/PageHeader.js

@@ -6,33 +6,26 @@ import InventoryFilter from '../InventoryFilter';
 import ToolbarButtons from './components/ToolbarButtons';
 import SettingsWarning from './components/SettingsWarning';
 import PageTitle from './PageTitle';
-import { useIopConfig } from '../../../common/Hooks/ConfigHooks';
 import './PageHeader.scss';
 
-const PageHeader = () => {
-  const isIop = useIopConfig();
-
-  return (
-    <div className="inventory-upload-header">
-      <SettingsWarning />
-      <PageTitle />
-      {!isIop && (
-        <div className="inventory-upload-header-description">
-          <InventorySettings />
-          <PageDescription />
-        </div>
-      )}
-      <Grid.Row>
-        <Grid.Col xs={4}>
-          <InventoryFilter />
-        </Grid.Col>
-        <Grid.Col xs={7} xsOffset={1}>
-          <ToolbarButtons />
-        </Grid.Col>
-      </Grid.Row>
+const PageHeader = () => (
+  <div className="inventory-upload-header">
+    <SettingsWarning />
+    <PageTitle />
+    <div className="inventory-upload-header-description">
+      <InventorySettings />
+      <PageDescription />
     </div>
-  );
-};
+    <Grid.Row>
+      <Grid.Col xs={4}>
+        <InventoryFilter />
+      </Grid.Col>
+      <Grid.Col xs={7} xsOffset={1}>
+        <ToolbarButtons />
+      </Grid.Col>
+    </Grid.Row>
+  </div>
+);
 
 PageHeader.propTypes = {};