foreman_rh_cloud 14.3.0 → 14.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1c2ad7078753ff687e7dfa1cf410efbfbc3f1ad9b8dc74500f9e77bf0c9a7fc9
-  data.tar.gz: 9eb2679eb38c91fa3c1163915328d91959f7e6408e73bd4fc0c4d7fe98b9a0cf
+  metadata.gz: b3d41bcf3b5de41dab8c50751aecf9374fcb0bce44f1e0896011161b25ace53e
+  data.tar.gz: 8eb6a78729500fcb4ae06e8a35e452a0ed7a57e8dbc9f1d8ddf46b8c2a341016
 SHA512:
-  metadata.gz: 0e4cb92c3a41b257ebf018a988ced92b086aa5cec373214888bb22a3dc43f765a007f42182b1da6a058c97b8633d59e1239efef658c3e0d53744bb9b8fc41125
-  data.tar.gz: 780195de89dcac76c3774a8b48ae67c7c52ad8b2b4f1029020e1c1e42ec701092f2122ff4caec64de083cf01094ebac40b42cf1d0985d06fdbbc46838331d541
+  metadata.gz: d42639d1e4cb269096eb8cabe57a4337ee9c3e04e6339d2fdf8ac79ccf87188d60293820b5c6c61d79d03ecf40b5c042c2da8e81bb70251dda030cc24c74d1e2
+  data.tar.gz: c709b8044deff700aa3c541c6c21a9afdec88a449366785251057c64f1d85f73b9080b4391aca99d7564657509e3ee53c03de90f9f6d98dc9239d9de9a29f053

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

@@ -3,6 +3,7 @@ module Api::V2::RhCloud
     include ForemanRhCloud::IopSmartProxyAccess
     layout false
 
+    before_action :set_admin_user, only: [:update]
     before_action :require_non_iop_smart_proxy, only: [:update]
 
     KNOWN_DIRECTIVES = {

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

@@ -7,7 +7,7 @@ module Api
         include InventoryUpload::TaskActions
         include ForemanRhCloud::IopSmartProxyAccess
 
-        before_action :require_non_iop_smart_proxy, only: [:enable_cloud_connector]
+        before_action :require_non_iop_smart_proxy, only: [:announce_to_sources]
 
         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")
@@ -70,10 +70,16 @@ module Api
           render json: { message: error.message }, status: :bad_request
         end
 
-        api :POST, "/rh_cloud/enable_connector", N_("Enable cloud connector")
-        def enable_cloud_connector
-          cloud_connector = ForemanRhCloud::CloudConnector.new
-          render json: cloud_connector.install.to_json
+        api :POST, "/rh_cloud/announce_to_sources", N_("Schedule a task to register in Red Hat Sources for cloud connector")
+        param :instance_id, String, required: true, desc: N_("RHC instance ID to register with Red Hat Sources")
+        def announce_to_sources
+          Setting[:rhc_instance_id] = params[:instance_id]
+
+          task = ForemanTasks.async_task(InsightsCloud::Async::CloudConnectorAnnounceTask, true)
+
+          render json: {
+            task: task,
+          }, status: :accepted
         end
       end
     end

data/app/controllers/foreman_inventory_upload/accounts_controller.rb

@@ -29,7 +29,6 @@ module ForemanInventoryUpload
 
       render json: {
         accounts: accounts,
-        CloudConnectorStatus: ForemanInventoryUpload::UploadsSettingsController.cloud_connector_status,
       }, status: :ok
     end
 

data/app/controllers/foreman_inventory_upload/uploads_controller.rb

@@ -1,22 +1,11 @@
 module ForemanInventoryUpload
   class UploadsController < ::ApplicationController
     include InventoryUpload::ReportActions
-    include ForemanRhCloud::IopSmartProxyAccess
-
-    before_action :require_non_iop_smart_proxy, only: [:enable_cloud_connector]
 
     def download_file
       filename, file = report_file(params[:organization_id])
 
       send_file file, disposition: 'attachment', filename: filename
     end
-
-    def enable_cloud_connector
-      # Set the autoupload to true, since it's required by the feature.
-      Setting[:allow_auto_inventory_upload] = true
-
-      cloud_connector = ForemanRhCloud::CloudConnector.new
-      render json: cloud_connector.install.to_json
-    end
   end
 end

data/app/controllers/foreman_inventory_upload/uploads_settings_controller.rb

@@ -9,7 +9,6 @@ module ForemanInventoryUpload
         ipsObfuscationEnabled: Setting[:obfuscate_inventory_ips],
         excludePackagesEnabled: Setting[:exclude_installed_packages],
         allowAutoInsightsMismatchDelete: Setting[:allow_auto_insights_mismatch_delete],
-        CloudConnectorStatus: ForemanInventoryUpload::UploadsSettingsController.cloud_connector_status,
         lastSyncTask: last_successful_inventory_sync_task,
       }, status: :ok
     end
@@ -19,13 +18,6 @@ module ForemanInventoryUpload
       index
     end
 
-    def self.cloud_connector_status
-      cloud_connector = ForemanRhCloud::CloudConnector.new
-      job = cloud_connector&.latest_job
-      return nil unless job
-      { id: job.id, task: ForemanTasks::Task.where(:id => job.task_id).first }
-    end
-
     def last_successful_inventory_sync_task
       task = ForemanTasks::Task.where(label: 'InventorySync::Async::InventoryFullSync', result: 'success')
                         .reorder('ended_at desc').first

data/app/services/foreman_rh_cloud/cloud_presence.rb

@@ -74,6 +74,9 @@ module ForemanRhCloud
 
     def register_rhc_instance
       raise Foreman::Exception.new('rhc_instance_id is empty, cannot register RHC to the cloud') if Setting[:rhc_instance_id].empty?
+
+      return :already_registered if rhc_connection_exists?
+
       source_id = satellite_instance_source || create_satellite_instance_source
 
       create_response = JSON.parse(
@@ -95,6 +98,22 @@ module ForemanRhCloud
       @satellite_instance_source = create_response['id']
     end
 
+    def rhc_connection_exists?
+      response = JSON.parse(
+        execute_cloud_request(
+          method: :get,
+          url: rhc_connection_url,
+          headers: {
+            content_type: :json,
+          },
+          ssl_client_cert: OpenSSL::X509::Certificate.new(certs[:cert]),
+          ssl_client_key: OpenSSL::PKey.read(certs[:key])
+        )
+      )
+
+      response['data']&.any?
+    end
+
     private
 
     def sources_url(path)
@@ -113,6 +132,10 @@ module ForemanRhCloud
       sources_url('/sources')
     end
 
+    def rhc_connection_url
+      sources_url("/rhc_connections?filter[rhc_id]=#{Setting[:rhc_instance_id]}")
+    end
+
     def create_rhc_connections_url
       sources_url('/rhc_connections')
     end

data/config/routes.rb

@@ -7,8 +7,6 @@ Rails.application.routes.draw do
     get 'settings', to: 'uploads_settings#index'
     post 'setting', to: 'uploads_settings#set_advanced_setting'
 
-    post 'cloud_connector', to: 'uploads#enable_cloud_connector'
-
     resources :tasks, only: [:create, :show]
 
     get 'status', to: 'cloud_status#index'
@@ -72,7 +70,7 @@ Rails.application.routes.draw do
       end
 
       namespace 'rh_cloud' do
-        post 'enable_connector', to: 'inventory#enable_cloud_connector'
+        post 'announce_to_sources', to: 'inventory#announce_to_sources'
         post 'cloud_request', to: 'cloud_request#update'
         get 'advisor_engine_config', to: 'advisor_engine_config#show'
 

data/lib/foreman_rh_cloud/engine.rb

@@ -61,6 +61,7 @@ module ForemanRhCloud
           ForemanRhCloud::Engine.register_scheduled_task(InventorySync::Async::InventoryScheduledSync, '0 0 * * *')
           ForemanRhCloud::Engine.register_scheduled_task(InsightsCloud::Async::InsightsScheduledSync, '0 0 * * *')
           ForemanRhCloud::Engine.register_scheduled_task(InsightsCloud::Async::InsightsClientStatusAging, '0 0 * * *')
+          ForemanRhCloud::Engine.register_scheduled_task(InsightsCloud::Async::CloudConnectorAnnounceTask, '0 0 * * *')
         end
       end
     rescue ActiveRecord::NoDatabaseError
@@ -80,12 +81,6 @@ module ForemanRhCloud
         host_action_button: false,
         provided_inputs: ['playbook_url', 'report_url', 'correlation_id', 'report_interval']
       )
-      RemoteExecutionFeature.register(
-        :ansible_configure_cloud_connector,
-        N_('Configure Cloud Connector on given hosts'),
-        :description => N_('Configure Cloud Connector on given hosts'),
-        :proxy_selector_override => ::RemoteExecutionProxySelector::INTERNAL_PROXY
-      )
     end
 
     # Ideally this code belongs to an initializer. The problem is that Katello controllers are not initialized completely until after the end of the to_prepare blocks

data/lib/foreman_rh_cloud/plugin.rb

@@ -31,8 +31,7 @@ module ForemanRhCloud
             :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],
+            'api/v2/rh_cloud/inventory': [:get_hosts, :remove_hosts, :sync_inventory_status, :download_file, :generate_report, :announce_to_sources],
             'foreman_inventory_upload/uploads_settings': [:set_advanced_setting],
             'foreman_inventory_upload/missing_hosts': [:remove_hosts],
             'insights_cloud/settings': [:update],
@@ -123,6 +122,8 @@ module ForemanRhCloud
         role 'ForemanRhCloud Read Only', read_only_permissions, 'Role granting read-only permissions to view
                                                                  Insights Compliance, Vulnerability, Advisor, and host inventory'
 
+        role 'Cloud Connector', [:dispatch_cloud_requests], 'Role granting permission to dispatch cloud connector requests'
+
         add_permissions_to_default_roles Role::ORG_ADMIN => plugin_permissions,
           Role::MANAGER => plugin_permissions,
           Role::SYSTEM_ADMIN => plugin_permissions

data/lib/foreman_rh_cloud/version.rb

@@ -1,3 +1,3 @@
 module ForemanRhCloud
-  VERSION = '14.3.0'.freeze
+  VERSION = '14.5.0'.freeze
 end

data/lib/insights_cloud/async/cloud_connector_announce_task.rb

@@ -1,39 +1,96 @@
 module InsightsCloud
   module Async
     class CloudConnectorAnnounceTask < ::Actions::EntryAction
-      def self.subscribe
-        Actions::RemoteExecution::RunHostsJob
-      end
+      include ::Actions::RecurringAction
+      include ::ForemanRhCloud::CertAuth
+      include ForemanInventoryUpload::Async::DelayedStart
 
-      def self.connector_feature_id
-        @connector_feature_id ||= RemoteExecutionFeature.feature!(ForemanRhCloud::CloudConnector::CLOUD_CONNECTOR_FEATURE).id
-      end
+      def plan(immediate = false)
+        if ForemanRhCloud.with_iop_smart_proxy?
+          logger.debug('Sources announcement skipped: running in IoP mode')
+          return
+        end
 
-      def plan(job_invocation)
-        return unless connector_playbook_job?(job_invocation)
+        if Setting[:rhc_instance_id].blank?
+          logger.debug('Sources announcement skipped: rhc_instance_id is not set')
+          return
+        end
 
-        plan_self
+        unless Setting[:allow_auto_inventory_upload]
+          logger.debug(
+            'Cloud connector is configured (rhc_instance_id is set) but automatic inventory upload is disabled. ' \
+            'Enable the "Automatic inventory upload" setting for full cloud connector functionality.'
+          )
+        end
+
+        if immediate
+          plan_self
+        else
+          after_delay do
+            plan_self
+          end
+        end
       end
 
-      def finalize
+      def run
+        registered = []
+        confirmed_by_remediation = []
+        already_registered = []
+        skipped = []
+        failed = {}
+
         Organization.unscoped.each do |org|
+          unless cert_auth_available?(org)
+            skipped << org.name
+            next
+          end
+
+          if recent_cloud_remediation?(org)
+            confirmed_by_remediation << org.name
+            next
+          end
+
           presence = ForemanRhCloud::CloudPresence.new(org, logger)
-          presence.announce_to_sources
+          result = presence.announce_to_sources
+          if result == :already_registered
+            already_registered << org.name
+          else
+            registered << org.name
+          end
         rescue StandardError => ex
-          logger.warn(ex)
+          logger.warn("Failed to announce to Sources for organization #{org.name}: #{ex}")
+          logger.debug { ex.backtrace.join("\n") }
+          failed[org.name] = ex.message
         end
-      end
 
-      def rescue_strategy_for_self
-        Dynflow::Action::Rescue::Skip
+        parts = []
+        parts << "Registered: #{registered.join(', ')}" if registered.any?
+        parts << "Already registered: #{already_registered.join(', ')}" if already_registered.any?
+        parts << "Already registered (recent cloud remediation): #{confirmed_by_remediation.join(', ')}" if confirmed_by_remediation.any?
+        parts << "Skipped (no manifest): #{skipped.join(', ')}" if skipped.any?
+        if failed.any?
+          failed_details = failed.map { |name, msg| "#{name}: #{msg}" }.join('; ')
+          parts << "Failed: #{failed_details}"
+        end
+        output[:status] = parts.join('. ')
+
+        error!("Sources announcement failed for: #{failed.keys.join(', ')}") if failed.any?
       end
 
-      def connector_playbook_job?(job_invocation)
-        job_invocation&.remote_execution_feature_id == connector_feature_id
+      def recent_cloud_remediation?(org)
+        feature = RemoteExecutionFeature.find_by(label: 'rh_cloud_connector_run_playbook')
+        return false unless feature
+
+        JobInvocation.where(remote_execution_feature_id: feature.id)
+                     .joins(:task)
+                     .joins(targeting: :hosts)
+                     .where(hosts: { organization_id: org.id })
+                     .where('foreman_tasks_tasks.started_at > ?', 24.hours.ago)
+                     .exists?
       end
 
-      def connector_feature_id
-        self.class.connector_feature_id
+      def rescue_strategy_for_self
+        Dynflow::Action::Rescue::Skip
       end
 
       def logger

data/package.json

@@ -1,6 +1,6 @@
 {
   "name": "foreman_rh_cloud",
-  "version": "14.3.0",
+  "version": "14.5.0",
   "description": "Inventory Upload =============",
   "main": "index.js",
   "scripts": {

data/test/controllers/inventory_upload/api/inventory_controller_test.rb

@@ -35,20 +35,22 @@ module InventoryUpload::Api
       assert_equal test_task.id.to_s, actual_task['id']
     end
 
-    test 'Starts cloud connector configuration job' do
-      test_job = FactoryBot.create(:job_invocation)
+    test 'Triggers Sources announcement task' do
+      test_task = FactoryBot.create(:some_task)
 
-      ForemanRhCloud::CloudConnector.any_instance
-        .expects(:install)
-        .returns(test_job)
+      ForemanTasks.expects(:async_task)
+        .with(InsightsCloud::Async::CloudConnectorAnnounceTask, true)
+        .returns(test_task)
 
-      post :enable_cloud_connector
+      post :announce_to_sources, params: { instance_id: 'test-instance-id' }
 
       assert_response :success
 
-      actual_job = @response.parsed_body
+      assert_equal 'test-instance-id', Setting[:rhc_instance_id]
 
-      assert_equal test_job.id, actual_job['id']
+      actual_task = @response.parsed_body['task']
+      assert_not_nil actual_task
+      assert_equal test_task.id.to_s, actual_task['id']
     end
   end
 end

data/test/jobs/cloud_connector_announce_task_test.rb

@@ -1,125 +1,135 @@
-require 'json'
 require 'test_plugin_helper'
 require 'foreman_tasks/test_helpers'
-require "#{ForemanTasks::Engine.root}/test/support/dummy_dynflow_action"
 
 class CloudConnectorAnnounceTaskTest < ActiveSupport::TestCase
   include Dynflow::Testing::Factories
 
   setup do
-    RemoteExecutionFeature.register(
-      :ansible_configure_cloud_connector,
-      N_('Configure Cloud Connector on given hosts'),
-      :description => N_('Configure Cloud Connector on given hosts'),
-      :proxy_selector_override => ::RemoteExecutionProxySelector::INTERNAL_PROXY
-    )
+    InsightsCloud::Async::CloudConnectorAnnounceTask.any_instance
+                                                    .stubs(:recent_cloud_remediation?).returns(false)
+  end
+
+  teardown do
+    ForemanRhCloud.unstub(:with_iop_smart_proxy?)
+  end
+
+  test 'announces to sources for all organizations when rhc_instance_id is set' do
+    Setting[:rhc_instance_id] = 'test-rhc-id'
+
+    InsightsCloud::Async::CloudConnectorAnnounceTask.any_instance
+                                                    .stubs(:cert_auth_available?).returns(true)
 
-    @job_invocation = generate_job_invocation(:ansible_configure_cloud_connector)
+    ForemanRhCloud::CloudPresence.any_instance
+                                 .expects(:announce_to_sources)
+                                 .times(Organization.unscoped.count)
 
-    # reset connector feature ID cache
-    InsightsCloud::Async::CloudConnectorAnnounceTask.instance_variable_set(:@connector_feature_id, nil)
+    action = create_and_plan_action(InsightsCloud::Async::CloudConnectorAnnounceTask)
+    action = run_action(action)
+
+    status = action.output[:status].to_s
+    assert_match(/Registered:|Already registered:/, status)
+    refute_match(/Skipped/, status)
+    refute_match(/Failed/, status)
   end
 
-  test 'It executes cloud presence announcer' do
-    ForemanRhCloud::CloudPresence.any_instance.expects(:announce_to_sources).times(Organization.unscoped.count)
+  test 'skips when rhc_instance_id is not set' do
+    Setting[:rhc_instance_id] = nil
+
+    action = create_and_plan_action(InsightsCloud::Async::CloudConnectorAnnounceTask)
 
-    action = create_and_plan_action(InsightsCloud::Async::CloudConnectorAnnounceTask, @job_invocation)
-    finalize_action(action)
+    assert_empty action.execution_plan.planned_run_steps
   end
 
-  private
-
-  def generate_job_invocation(feature_name)
-    job_template = FactoryBot.build(
-      :job_template,
-      :template => 'BLEH'
-    )
-    feature = RemoteExecutionFeature.feature!(feature_name).id
-
-    job_invocation = FactoryBot.create(
-      :job_invocation,
-      remote_execution_feature_id: feature,
-      task_id: FactoryBot.create(:dynflow_task).id
-    )
-
-    job_template.template_inputs << playbook_url_input = FactoryBot.build(:template_input,
-      :name => 'playbook_url',
-      :input_type => 'user',
-      :required => true)
-    job_template.template_inputs << report_url_input = FactoryBot.build(:template_input,
-      :name => 'report_url',
-      :input_type => 'user',
-      :required => true)
-    job_template.template_inputs << correlation_id_input = FactoryBot.build(:template_input,
-      :name => 'correlation_id',
-      :input_type => 'user',
-      :required => true)
-    job_template.template_inputs << report_interval_input = FactoryBot.build(:template_input,
-      :name => 'report_interval',
-      :input_type => 'user',
-      :required => true)
-
-    template_invocation = FactoryBot.build(:template_invocation,
-      :template => job_template,
-      :job_invocation => job_invocation)
-
-    template_invocation.input_values << FactoryBot.create(
-      :template_invocation_input_value,
-      :template_invocation => template_invocation,
-      :template_input => playbook_url_input,
-      :value => 'http://example.com/TEST_PLAYBOOK'
-    )
-    template_invocation.input_values << FactoryBot.create(
-      :template_invocation_input_value,
-      :template_invocation => template_invocation,
-      :template_input => report_url_input,
-      :value => 'http://example.com/TEST_REPORT'
-    )
-    template_invocation.input_values << FactoryBot.create(
-      :template_invocation_input_value,
-      :template_invocation => template_invocation,
-      :template_input => correlation_id_input,
-      :value => 'TEST_CORRELATION'
-    )
-    template_invocation.input_values << FactoryBot.create(
-      :template_invocation_input_value,
-      :template_invocation => template_invocation,
-      :template_input => report_interval_input,
-      :value => '1'
-    )
-
-    @host1 = FactoryBot.create(:host, :with_insights_hits, name: 'host1')
-    @host1.insights.uuid = 'TEST_UUID1'
-    @host1.insights.save!
-    @host2 = FactoryBot.create(:host, :with_insights_hits, name: 'host2')
-    @host2.insights.uuid = 'TEST_UUID2'
-    @host2.insights.save!
-
-    targeting = FactoryBot.create(:targeting, hosts: [@host1, @host2])
-    job_invocation.targeting = targeting
-    job_invocation.save!
-
-    job_invocation.template_invocations << FactoryBot.create(
-      :template_invocation,
-      run_host_job_task: FactoryBot.create(:dynflow_task),
-      host_id: @host1.id
-    )
-    job_invocation.template_invocations << FactoryBot.create(
-      :template_invocation,
-      run_host_job_task: FactoryBot.create(:dynflow_task),
-      host_id: @host2.id
-    )
-
-    fake_output = (1..5).map do |i|
-      { 'timestamp' => (Time.now - (5 - i)).to_f, 'output' => "#{i}\n" }
-    end
-    Support::DummyDynflowAction.any_instance.stubs(:live_output).returns(fake_output)
-    Support::DummyDynflowAction.any_instance.stubs(:exit_status).returns(0)
+  test 'skips when rhc_instance_id is empty string' do
+    Setting[:rhc_instance_id] = ''
+
+    action = create_and_plan_action(InsightsCloud::Async::CloudConnectorAnnounceTask)
+
+    assert_empty action.execution_plan.planned_run_steps
+  end
+
+  test 'skips when in IoP mode' do
+    Setting[:rhc_instance_id] = 'test-rhc-id'
+    ForemanRhCloud.stubs(:with_iop_smart_proxy?).returns(true)
+
+    action = create_and_plan_action(InsightsCloud::Async::CloudConnectorAnnounceTask)
+
+    assert_empty action.execution_plan.planned_run_steps
+  end
+
+  test 'skips organizations without a manifest' do
+    Setting[:rhc_instance_id] = 'test-rhc-id'
+
+    InsightsCloud::Async::CloudConnectorAnnounceTask.any_instance
+                                                    .stubs(:cert_auth_available?).returns(false)
+
+    ForemanRhCloud::CloudPresence.any_instance
+                                 .expects(:announce_to_sources)
+                                 .never
 
-    job_invocation
+    action = create_and_plan_action(InsightsCloud::Async::CloudConnectorAnnounceTask)
+    action = run_action(action)
+
+    status = action.output[:status].to_s
+    assert_match(/Skipped \(no manifest\):/, status)
+    refute_match(/Registered/, status)
+    refute_match(/Failed/, status)
+  end
+
+  test 'still runs when allow_auto_inventory_upload is disabled' do
+    Setting[:rhc_instance_id] = 'test-rhc-id'
+    Setting[:allow_auto_inventory_upload] = false
+
+    InsightsCloud::Async::CloudConnectorAnnounceTask.any_instance
+                                                    .stubs(:cert_auth_available?).returns(true)
+
+    ForemanRhCloud::CloudPresence.any_instance
+                                 .expects(:announce_to_sources)
+                                 .times(Organization.unscoped.count)
+
+    action = create_and_plan_action(InsightsCloud::Async::CloudConnectorAnnounceTask)
+    action = run_action(action)
+
+    assert_match(/Registered:|Already registered:/, action.output[:status].to_s)
+  end
+
+  test 'skips orgs with recent cloud remediation jobs' do
+    Setting[:rhc_instance_id] = 'test-rhc-id'
+
+    InsightsCloud::Async::CloudConnectorAnnounceTask.any_instance
+                                                    .stubs(:cert_auth_available?).returns(true)
+    InsightsCloud::Async::CloudConnectorAnnounceTask.any_instance
+                                                    .stubs(:recent_cloud_remediation?).returns(true)
+
+    ForemanRhCloud::CloudPresence.any_instance
+                                 .expects(:announce_to_sources)
+                                 .never
+
+    action = create_and_plan_action(InsightsCloud::Async::CloudConnectorAnnounceTask)
+    action = run_action(action)
+
+    status = action.output[:status].to_s
+    assert_match(/Already registered \(recent cloud remediation\):/, status)
+    refute_match(/\bRegistered:/, status)
   end
 
-  def read_jsonl(jsonl)
-    jsonl.lines.map { |l| JSON.parse(l) }
+  test 'continues processing other orgs when one fails and task errors' do
+    Setting[:rhc_instance_id] = 'test-rhc-id'
+
+    InsightsCloud::Async::CloudConnectorAnnounceTask.any_instance
+                                                    .stubs(:cert_auth_available?).returns(true)
+
+    call_count = 0
+    ForemanRhCloud::CloudPresence.any_instance.stubs(:announce_to_sources).with do
+      call_count += 1
+      raise(StandardError.new('API error')) if call_count == 1
+      true
+    end
+
+    action = create_and_plan_action(InsightsCloud::Async::CloudConnectorAnnounceTask)
+
+    error = assert_raises(StandardError) { run_action(action) }
+    assert_match(/Sources announcement failed for:/, error.message)
+    assert_equal Organization.unscoped.count, call_count
   end
 end

data/webpack/ForemanInventoryUpload/Components/InventorySettings/MinimalInventoryDropdown.js

@@ -79,6 +79,7 @@ const MinimalInventoryDropdown = ({ setChosenValue }) => {
           isFullWidth
           onClick={onToggleClick}
           isExpanded={isOpen}
+          ouiaId="inventory-dropdown-toggle"
         >
           {dropdownValues[currentDropdownValue].title}
         </MenuToggle>

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

@@ -59,6 +59,7 @@ const PageTitle = () => {
               variant="plain"
               onClick={() => setIsDropdownOpen(prev => !prev)}
               isExpanded={isDropdownOpen}
+              ouiaId="title-dropdown-toggle"
             >
               <EllipsisVIcon />
             </MenuToggle>