foreman_rh_cloud 4.0.25 → 5.0.28

data/config/rh_cert-api_chain.pem

@@ -0,0 +1,74 @@
+-----BEGIN CERTIFICATE-----
+MIIHZDCCBUygAwIBAgIJAOb+QiglyeZeMA0GCSqGSIb3DQEBBQUAMIGwMQswCQYD
+VQQGEwJVUzEXMBUGA1UECAwOTm9ydGggQ2Fyb2xpbmExEDAOBgNVBAcMB1JhbGVp
+Z2gxFjAUBgNVBAoMDVJlZCBIYXQsIEluYy4xGDAWBgNVBAsMD1JlZCBIYXQgTmV0
+d29yazEeMBwGA1UEAwwVRW50aXRsZW1lbnQgTWFzdGVyIENBMSQwIgYJKoZIhvcN
+AQkBFhVjYS1zdXBwb3J0QHJlZGhhdC5jb20wHhcNMTAwMzE3MTkwMDQ0WhcNMzAw
+MzEyMTkwMDQ0WjCBsDELMAkGA1UEBhMCVVMxFzAVBgNVBAgMDk5vcnRoIENhcm9s
+aW5hMRAwDgYDVQQHDAdSYWxlaWdoMRYwFAYDVQQKDA1SZWQgSGF0LCBJbmMuMRgw
+FgYDVQQLDA9SZWQgSGF0IE5ldHdvcmsxHjAcBgNVBAMMFUVudGl0bGVtZW50IE1h
+c3RlciBDQTEkMCIGCSqGSIb3DQEJARYVY2Etc3VwcG9ydEByZWRoYXQuY29tMIIC
+IjANBgkqhkiG9w0BAQEFAAOCAg8AMIICCgKCAgEA2Z+mW7OYcBcGxWS+RSKG2GJ2
+csMXiGGfEp36vKVsIvypmNS60SkicKENMYREalbdSjrgfXxPJygZWsVWJ5lHPfBV
+o3WkFrFHTIXd/R6LxnaHD1m8Cx3GwEeuSlE/ASjc1ePtMnsHH7xqZ9wdl85b1C8O
+scgO7fwuM192kvv/veI/BogIqUQugtG6szXpV8dp4ml029LXFoNIy2lfFoa2wKYw
+MiUHwtYgAz7TDY63e8qGhd5PoqTv9XKQogo2ze9sF9y/npZjliNy5qf6bFE+24oW
+E8pGsp3zqz8h5mvw4v+tfIx5uj7dwjDteFrrWD1tcT7UmNrBDWXjKMG81zchq3h4
+etgF0iwMHEuYuixiJWNzKrLNVQbDmcLGNOvyJfq60tM8AUAd72OUQzivBegnWMit
+CLcT5viCT1AIkYXt7l5zc/duQWLeAAR2FmpZFylSukknzzeiZpPclRziYTboDYHq
+revM97eER1xsfoSYp4mJkBHfdlqMnf3CWPcNgru8NbEPeUGMI6+C0YvknPlqDDtU
+ojfl4qNdf6nWL+YNXpR1YGKgWGWgTU6uaG8Sc6qGfAoLHh6oGwbuz102j84OgjAJ
+DGv/S86svmZWSqZ5UoJOIEqFYrONcOSgztZ5tU+gP4fwRIkTRbTEWSgudVREOXhs
+bfN1YGP7HYvS0OiBKZUCAwEAAaOCAX0wggF5MB0GA1UdDgQWBBSIS6ZFxEbsj9bP
+pvYazyY8kMx/FzCB5QYDVR0jBIHdMIHagBSIS6ZFxEbsj9bPpvYazyY8kMx/F6GB
+tqSBszCBsDELMAkGA1UEBhMCVVMxFzAVBgNVBAgMDk5vcnRoIENhcm9saW5hMRAw
+DgYDVQQHDAdSYWxlaWdoMRYwFAYDVQQKDA1SZWQgSGF0LCBJbmMuMRgwFgYDVQQL
+DA9SZWQgSGF0IE5ldHdvcmsxHjAcBgNVBAMMFUVudGl0bGVtZW50IE1hc3RlciBD
+QTEkMCIGCSqGSIb3DQEJARYVY2Etc3VwcG9ydEByZWRoYXQuY29tggkA5v5CKCXJ
+5l4wDAYDVR0TBAUwAwEB/zALBgNVHQ8EBAMCAQYwEQYJYIZIAYb4QgEBBAQDAgEG
+MCAGA1UdEQQZMBeBFWNhLXN1cHBvcnRAcmVkaGF0LmNvbTAgBgNVHRIEGTAXgRVj
+YS1zdXBwb3J0QHJlZGhhdC5jb20wDQYJKoZIhvcNAQEFBQADggIBAJ1hEdNBDTRr
+6kI6W6stoogSUwjuiWPDY8DptwGhdpyIfbCoxvBR7F52DlwyXOpCunogfKMRklnE
+gH1Wt66RYkgNuJcenKHAhR5xgSLoPCOVF9rDjMunyyBuxjIbctM21R7BswVpsEIE
+OpV5nlJ6wkHsrn0/E+Zk5UJdCzM+Fp4hqHtEn/c97nvRspQcpWeDg6oUvaJSZTGM
+8yFpzR90X8ZO4rOgpoERukvYutUfJUzZuDyS3LLc6ysamemH93rZXr52zc4B+C9G
+Em8zemDgIPaH42ce3C3TdVysiq/yk+ir7pxW8toeavFv75l1UojFSjND+Q2AlNQn
+pYkmRznbD5TZ3yDuPFQG2xYKnMPACepGgKZPyErtOIljQKCdgcvb9EqNdZaJFz1+
+/iWKYBL077Y0CKwb+HGIDeYdzrYxbEd95YuVU0aStnf2Yii2tLcpQtK9cC2+DXjL
+Yf3kQs4xzH4ZejhG9wzv8PGXOS8wHYnfVNA3+fclDEQ1mEBKWHHmenGI6QKZUP8f
+g0SQ3PNRnSZu8R+rhABOEuVFIBRlaYijg2Pxe0NgL9FlHsNyRfo6EUrB2QFRKACW
+3Mo6pZyDjQt7O8J7l9B9IIURoJ1niwygf7VSJTMl2w3fFleNJlZTGgdXw0V+5g+9
+Kg6Ay0rrsi4nw1JHue2GvdjdfVOaWSWC
+-----END CERTIFICATE-----
+-----BEGIN CERTIFICATE-----
+MIIFfTCCA2WgAwIBAgIJAJGKz8qFAAADMA0GCSqGSIb3DQEBBQUAMIGwMQswCQYD
+VQQGEwJVUzEXMBUGA1UECAwOTm9ydGggQ2Fyb2xpbmExEDAOBgNVBAcMB1JhbGVp
+Z2gxFjAUBgNVBAoMDVJlZCBIYXQsIEluYy4xGDAWBgNVBAsMD1JlZCBIYXQgTmV0
+d29yazEeMBwGA1UEAwwVRW50aXRsZW1lbnQgTWFzdGVyIENBMSQwIgYJKoZIhvcN
+AQkBFhVjYS1zdXBwb3J0QHJlZGhhdC5jb20wHhcNMTUwNTA1MTMwMzQ4WhcNMjUw
+NTAyMTMwMzQ4WjCBiTELMAkGA1UEBhMCVVMxFzAVBgNVBAgTDk5PUlRIIENBUk9M
+SU5BMRAwDgYDVQQHEwdSYWxlaWdoMRAwDgYDVQQKEwdSZWQgSGF0MRgwFgYDVQQL
+Ew9SZWQgSGF0IE5ldHdvcmsxIzAhBgNVBAMTGmNlcnQtYXBpLmFjY2Vzcy5yZWRo
+YXQuY29tMIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEA9hTNMtZMa7Kg
+Jlux6pnuUinP0Rv0aiiPFr7qNHFore4loGrPlpzUvQbUByy3xm7lhf4R4qbINCls
+veWg6HDidvQr174RXb5YLMXuBrYAiPWQTrRRLNuvXFHKzREghRWTv48IXTIDEo0G
+fZJUO+myY2RfwqugZKic5dR6ZakHSSpQO70O6H5R0eHlKa13k4eEpG2fVY/xqFto
+WkfZyEmSacZpqxp7gIjZqreLc4MFwpiVjGFrK3Jk+Px1Z6J94LTLx2SxrYzWIeUs
+5j+lceQOvpV4/pkClnRCW1pkCKTccjFKQkpNPGwdIusRXUGl9IYc20Fa/7g9iUQc
+5fXu9EAzfwIDAQABo4G+MIG7MAkGA1UdEwQCMAAwEQYJYIZIAYb4QgEBBAQDAgZA
+MAsGA1UdDwQEAwIF4DATBgNVHSUEDDAKBggrBgEFBQcDATA5BglghkgBhvhCAQ0E
+LBYqTWFuYWdlZCBieSBSZWQgSGF0IChjYS1zdXBwb3J0QHJlZGhhdC5jb20pMB0G
+A1UdDgQWBBRfgCjd8aXf0U4VX8DKTVIn+paGBzAfBgNVHSMEGDAWgBSIS6ZFxEbs
+j9bPpvYazyY8kMx/FzANBgkqhkiG9w0BAQUFAAOCAgEAlC+r6UEEp5BUkI0Rj2T+
+1PH7oUCaGQeQoyVbGddz/WUcBk/lMMtyCEoxU+3tTwNWmCWWjYXtjoL9MlSAC/q+
+NZfBi1iq0uuSus9JI/Uu8aRhoxTK56qGRed/JNixOHEmFn891cahIPpF0exWwtYD
+ThwXo7Z6PI7t8EMKdSrGTOowp58yho8xYFL/Z7JmjL55Pf85GIrdiniNZd4i178J
+07R9zsiLvdXq9mT33iJwkm+uhO+FA9d8OE3ji21pBbGUAQSWOdkemvUCsy8zANW9
+fT+dBrMr5Buk7eaBBJ2PxECNiWLCRQJWmyff1O5zMT0daS2lBdEGUNhBZ0hnX13Q
+kabUp0bxRrNRq+WkomP7onZhfZS6SjKm0UmwoV6o3V1ED6y7muQNRmgDpA5PcbvO
+gl7OexNL4zcpyfMdAmTYf5yTRSvB42Yg5hVfuzPEnOIqupwES3mWkEHRlqbMUkHw
+qIQAxIwQqZd5PdPpElQ/6j/ZT9DwW/I6zgndX2rsS0oGYcwFTkSj0/rKKkC13hk7
+LchXMZu5ckdustM79U6OZIBairrJaL2OpR08un2nwIjgEGqhVFYc44UK1VpkE8mr
+qvqJS6OHVlTlKcEDnhVkPS3i5qjuS/PtSq0CwH8bzYKFJayLDY/z36Zv6PdttzmU
+Yb1NSDcJejHJ80pMINutyYQ=
+-----END CERTIFICATE-----

data/config/routes.rb

@@ -11,6 +11,8 @@ Rails.application.routes.draw do
     post 'cloud_connector', to: 'uploads#enable_cloud_connector'
 
     resources :tasks, only: [:create, :show]
+
+    get 'status', to: 'cloud_status#index'
   end
 
   namespace :insights_cloud do
@@ -37,7 +39,7 @@ Rails.application.routes.draw do
     end
 
     scope '/r/insights' do
-      match '/*path', :constraints => lambda { |req| !req.path.include?('view/api') }, to: 'machine_telemetries#forward_request', via: [:get, :post, :delete,:put, :patch]
+      match '(/*path)(/)', :constraints => lambda { |req| !req.path.include?('view/api') }, to: 'machine_telemetries#forward_request', via: [:get, :post, :delete,:put, :patch]
     end
   end
 

data/db/migrate/20211027000001_create_task_output.foreman_rh_cloud.rb

@@ -0,0 +1,18 @@
+class CreateTaskOutput < ActiveRecord::Migration[5.2]
+  def change
+    create_table :task_output_lines do |t|
+      t.string :label
+      t.string :line
+      t.timestamps
+
+      t.index :label
+    end
+
+    create_table :task_output_statuses do |t|
+      t.string :label
+      t.string :status
+
+      t.index :label, unique: true
+    end
+  end
+end

data/lib/foreman_inventory_upload/async/generate_all_reports_job.rb

@@ -1,7 +1,9 @@
 module ForemanInventoryUpload
   module Async
-    class GenerateAllReportsJob < ::ApplicationJob
-      def perform
+    class GenerateAllReportsJob < ::Actions::EntryAction
+      include ::Actions::RecurringAction
+
+      def plan
         unless Setting[:allow_auto_inventory_upload]
           logger.debug(
             'The scheduled process is disabled due to the "allow_auto_inventory_upload"
@@ -16,17 +18,19 @@ module ForemanInventoryUpload
           total_hosts = ForemanInventoryUpload::Generators::Queries.for_org(organization.id, use_batches: false).count
 
           if total_hosts <= ForemanInventoryUpload.max_org_size
-            GenerateReportJob.perform_later(ForemanInventoryUpload.generated_reports_folder, organization.id)
+            plan_generate_report(ForemanInventoryUpload.generated_reports_folder, organization)
           else
             logger.info("Skipping automatic uploads for organization #{organization.name}, too many hosts (#{total_hosts}/#{ForemanInventoryUpload.max_org_size})")
           end
         end.compact
-      ensure
-        self.class.set(:wait => 24.hours).perform_later
       end
 
-      def self.singleton_job_name
-        name
+      def rescue_strategy_for_self
+        Dynflow::Action::Rescue::Fail
+      end
+
+      def plan_generate_report(folder, organization)
+        plan_action(ForemanInventoryUpload::Async::GenerateReportJob, folder, organization.id)
       end
     end
   end

data/lib/foreman_inventory_upload/async/generate_report_job.rb

@@ -5,17 +5,21 @@ module ForemanInventoryUpload
         "report_for_#{label}"
       end
 
-      def perform(base_folder, organization)
-        @base_folder = base_folder
-        @organization = organization
+      def plan(base_folder, organization_id)
+        sequence do
+          super(
+            GenerateReportJob.output_label(organization_id),
+            organization_id: organization_id,
+            base_folder: base_folder
+          )
 
-        super(GenerateReportJob.output_label(organization))
-
-        QueueForUploadJob.perform_later(
-          base_folder,
-          ForemanInventoryUpload.facts_archive_name(organization),
-          organization
-        )
+          plan_action(
+            QueueForUploadJob,
+            base_folder,
+            ForemanInventoryUpload.facts_archive_name(organization_id),
+            organization_id
+          )
+        end
       end
 
       def rake_prefix
@@ -28,10 +32,18 @@ module ForemanInventoryUpload
 
       def env
         super.merge(
-          'target' => @base_folder,
-          'organization_id' => @organization
+          'target' => base_folder,
+          'organization_id' => organization_id
         )
       end
+
+      def base_folder
+        input[:base_folder]
+      end
+
+      def organization_id
+        input[:organization_id]
+      end
     end
   end
 end

data/lib/foreman_inventory_upload/async/progress_output.rb

@@ -6,6 +6,7 @@ module ForemanInventoryUpload
       end
 
       def self.register(label)
+        TaskOutputLine.where(label: @label).delete_all
         ProgressOutput.new(label, :writer)
       end
 
@@ -14,47 +15,23 @@ module ForemanInventoryUpload
         @mode = mode
       end
 
-      def buffer
-        @buffer ||= begin
-                      File.open(file_name, file_mode)
-                    rescue Errno::ENOENT
-                      StringIO.new
-                    end
-      end
-
       def full_output
-        buffer.read
+        TaskOutputLine.where(label: @label).order(:created_at).pluck(:line).join("\n")
       end
 
       def write_line(line)
-        buffer << line
-        buffer.fsync
+        TaskOutputLine.create!(label: @label, line: line)
       end
 
       def close
-        @buffer&.close
       end
 
       def status
-        File.read(file_name(:status))
-      rescue Errno::ENOENT
-        ''
+        TaskOutputStatus.where(label: @label).pluck(:status).first || ''
       end
 
       def status=(status)
-        File.atomic_write(file_name(:status)) do |status_file|
-          status_file.write(status)
-        end
-      end
-
-      private
-
-      def file_mode
-        (@mode == :reader) ? 'r' : 'w'
-      end
-
-      def file_name(type = 'out')
-        File.join(ForemanInventoryUpload.outputs_folder, "#{@label}.#{type}")
+        TaskOutputStatus.upsert({ label: @label, status: status }, unique_by: :label)
       end
     end
   end

data/lib/foreman_inventory_upload/async/queue_for_upload_job.rb

@@ -1,9 +1,12 @@
 module ForemanInventoryUpload
   module Async
-    class QueueForUploadJob < ::ApplicationJob
-      def perform(base_folder, report_file, organization_id)
-        @base_folder = base_folder
-        @report_file = report_file
+    class QueueForUploadJob < ::Actions::EntryAction
+      def plan(base_folder, report_file, organization_id)
+        enqueue_task = plan_self(base_folder: base_folder, report_file: report_file)
+        plan_upload_report(enqueue_task.output[:enqueued_file_name], organization_id)
+      end
+
+      def run
         logger.debug('Ensuring objects')
         ensure_ouput_folder
         ensure_output_script
@@ -12,7 +15,7 @@ module ForemanInventoryUpload
         FileUtils.mv(File.join(base_folder, report_file), enqueued_file_name)
         logger.debug("Done copying #{report_file} to #{enqueued_file_name}")
 
-        UploadReportJob.perform_later(enqueued_file_name, organization_id)
+        output[:enqueued_file_name] = enqueued_file_name
       end
 
       def uploads_folder
@@ -47,6 +50,18 @@ module ForemanInventoryUpload
       def logger
         Foreman::Logging.logger('background')
       end
+
+      def base_folder
+        input[:base_folder]
+      end
+
+      def report_file
+        input[:report_file]
+      end
+
+      def plan_upload_report(enqueued_file_name, organization_id)
+        plan_action(UploadReportJob, enqueued_file_name, organization_id)
+      end
     end
   end
 end

data/lib/foreman_inventory_upload/async/shell_process.rb

@@ -2,13 +2,18 @@ require 'open3'
 
 module ForemanInventoryUpload
   module Async
-    class ShellProcess < ::ApplicationJob
+    class ShellProcess < ::Actions::EntryAction
       include AsyncHelpers
 
-      def perform(instance_label)
+      def plan(instance_label, more_inputs = {})
+        inputs = more_inputs.merge(instance_label: instance_label)
+        plan_self(inputs)
+      end
+
+      def run
         klass_name = self.class.name
         logger.debug("Starting #{klass_name} with label #{instance_label}")
-        progress_output_for(instance_label) do |progress_output|
+        progress_output do |progress_output|
           Open3.popen2e(hash_to_s(env), *preprocess_command(command)) do |_stdin, stdout_stderr, wait_thread|
             progress_output.status = "Running in pid #{wait_thread.pid}"
 
@@ -25,7 +30,7 @@ module ForemanInventoryUpload
       def command
       end
 
-      def progress_output_for(instance_label)
+      def progress_output
         progress_output = ProgressOutput.register(instance_label)
         yield(progress_output)
       ensure
@@ -40,11 +45,19 @@ module ForemanInventoryUpload
         Foreman::Logging.logger('background')
       end
 
+      def rescue_strategy_for_self
+        Dynflow::Action::Rescue::Fail
+      end
+
       private
 
       def preprocess_command(command)
         command.kind_of?(Array) ? command : [command]
       end
+
+      def instance_label
+        input[:instance_label]
+      end
     end
   end
 end

data/lib/foreman_inventory_upload/async/upload_report_job.rb

@@ -7,44 +7,45 @@ module ForemanInventoryUpload
         "upload_for_#{label}"
       end
 
-      def perform(filename, organization_id)
+      def plan(filename, organization_id)
         label = UploadReportJob.output_label(organization_id)
-        @filename = filename
-        @organization = Organization.find(organization_id)
+        super(label, filename: filename, organization_id: organization_id)
+      end
 
+      def run
         if Setting[:content_disconnected]
-          progress_output_for(label) do |progress_output|
+          progress_output do |progress_output|
             progress_output.write_line('Upload was stopped since disconnected mode setting is enabled for content on this instance.')
             progress_output.status = "Task aborted, exit 1"
           end
           return
         end
 
-        unless @organization.owner_details&.fetch('upstreamConsumer')&.fetch('idCert')
-          logger.info("Skipping organization '#{@organization}', no candlepin certificate defined.")
-          progress_output_for(label) do |progress_output|
-            progress_output.write_line("Skipping organization #{@organization}, no candlepin certificate defined.")
+        unless organization.owner_details&.fetch('upstreamConsumer')&.fetch('idCert')
+          logger.info("Skipping organization '#{organization}', no candlepin certificate defined.")
+          progress_output do |progress_output|
+            progress_output.write_line("Skipping organization #{organization}, no candlepin certificate defined.")
             progress_output.status = "Task aborted, exit 1"
           end
           return
         end
 
-        Tempfile.create([@organization.name, '.pem']) do |cer_file|
+        Tempfile.create([organization.name, '.pem']) do |cer_file|
           cer_file.write(rh_credentials[:cert])
           cer_file.write(rh_credentials[:key])
           cer_file.flush
           @cer_path = cer_file.path
-          super(label)
+          super
         end
       end
 
       def command
-        ['/bin/bash', File.join(File.dirname(@filename), ForemanInventoryUpload.upload_script_file)]
+        ['/bin/bash', File.join(File.dirname(filename), ForemanInventoryUpload.upload_script_file)]
       end
 
       def env
         env_vars = super.merge(
-          'FILES' => @filename,
+          'FILES' => filename,
           'CER_PATH' => @cer_path
         )
 
@@ -58,13 +59,21 @@ module ForemanInventoryUpload
 
       def rh_credentials
         @rh_credentials ||= begin
-          candlepin_id_certificate = @organization.owner_details['upstreamConsumer']['idCert']
+          candlepin_id_certificate = organization.owner_details['upstreamConsumer']['idCert']
           {
             cert: candlepin_id_certificate['cert'],
             key: candlepin_id_certificate['key'],
           }
         end
       end
+
+      def filename
+        input[:filename]
+      end
+
+      def organization
+        @organization ||= Organization.find(input[:organization_id])
+      end
     end
   end
 end

data/lib/foreman_inventory_upload/generators/queries.rb

@@ -45,26 +45,10 @@ module ForemanInventoryUpload
           )
       end
 
-      def self.for_report(portal_user)
-        org_ids = organizations_for_user(portal_user).pluck(:id)
-        for_org(org_ids)
-      end
-
       def self.for_org(organization_id, use_batches: true)
         base_query = for_slice(Host.unscoped.where(organization_id: organization_id))
         use_batches ? base_query.in_batches(of: ForemanInventoryUpload.slice_size) : base_query
       end
-
-      def self.organizations_for_user(portal_user)
-        Organization
-          .joins(:telemetry_configuration)
-          .where(
-            redhat_access_telemetry_configurations: {
-              portal_user: portal_user,
-              enable_telemetry: true,
-            }
-          )
-      end
     end
   end
 end

data/lib/foreman_inventory_upload/generators/tags.rb

@@ -19,8 +19,7 @@ module ForemanInventoryUpload
       def generate_parameters
         return [] unless Setting[:include_parameter_tags]
 
-        (@host.host_inherited_params_objects || [])
-          .map { |item| [item.name, item.value] }
+        (@host.host_params || {})
           .select { |_name, value| value.present? || value.is_a?(FalseClass) }
       end
 

data/lib/foreman_rh_cloud/engine.rb

@@ -1,5 +1,4 @@
 require 'katello'
-require 'redhat_access'
 require 'foreman_ansible'
 
 module ForemanRhCloud
@@ -43,6 +42,8 @@ module ForemanRhCloud
       Foreman::Plugin.register :foreman_rh_cloud do
         requires_foreman '>= 2.3'
 
+        apipie_documented_controllers ["#{ForemanRhCloud::Engine.root}/app/controllers/api/v2/**/*.rb"]
+
         # Add permissions
         security_block :foreman_rh_cloud do
           permission(
@@ -114,15 +115,6 @@ module ForemanRhCloud
       ::Host::Managed.include RhCloudHost
     end
 
-    initializer "foreman_rh_cloud.set_dynflow.config.on_init", :before => :finisher_hook do |_app|
-      unless Rails.env.test?
-        ForemanTasks.dynflow.config.on_init do |world|
-          ForemanInventoryUpload::Async::GenerateAllReportsJob.spawn_if_missing(world)
-          InsightsCloud::Async::InsightsScheduledSync.spawn_if_missing(world)
-        end
-      end
-    end
-
     rake_tasks do
       Rake::Task['db:seed'].enhance do
         ForemanRhCloud::Engine.load_seed
@@ -138,7 +130,6 @@ module ForemanRhCloud
     config.to_prepare do
       # skip database manipulations while tables do not exist, like in migrations
       if ActiveRecord::Base.connection.data_source_exists?(ForemanTasks::Task.table_name) &&
-
         RemoteExecutionFeature.register(
           :rh_cloud_remediate_hosts,
           N_('Apply Insights recommendations'),
@@ -148,7 +139,9 @@ module ForemanRhCloud
         # skip object creation when admin user is not present, for example in test DB
         if User.unscoped.find_by_login(User::ANONYMOUS_ADMIN).present?
           ::ForemanTasks.dynflow.config.on_init(false) do |world|
+            ForemanRhCloud::Engine.register_scheduled_task(ForemanInventoryUpload::Async::GenerateAllReportsJob, '0 0 * * *')
             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 * * *')
           end
         end

data/lib/foreman_rh_cloud/version.rb

@@ -1,3 +1,3 @@
 module ForemanRhCloud
-  VERSION = '4.0.25'.freeze
+  VERSION = '5.0.28'.freeze
 end

data/lib/foreman_rh_cloud.rb

@@ -98,6 +98,21 @@ module ForemanRhCloud
 
   # For testing purposes we can override the default hostname with an environment variable SATELLITE_RH_CLOUD_FOREMAN_HOST
   def self.foreman_host
-    @foreman_host ||= ::Host.unscoped.friendly.find(ENV['SATELLITE_RH_CLOUD_FOREMAN_HOST'] || ::SmartProxy.default_capsule.name)
+    @foreman_host ||= begin
+                        fullname = foreman_host_name
+                        ::Host.unscoped.friendly.find(fullname)
+                      rescue ActiveRecord::RecordNotFound
+                        # fullname didn't work. Let's try shortname
+                        shortname = /(?<shortname>[^\.]*)\.?.*/.match(fullname)[:shortname]
+                        ::Host.unscoped.friendly.find(shortname)
+                      end
+  end
+
+  def self.foreman_host_name
+    ENV['SATELLITE_RH_CLOUD_FOREMAN_HOST'] || ::SmartProxy.default_capsule.name
+  end
+
+  def self.legacy_insights_ca
+    "#{ForemanRhCloud::Engine.root}/config/rh_cert-api_chain.pem"
   end
 end

data/lib/insights_cloud/async/insights_client_status_aging.rb

@@ -18,6 +18,10 @@ module InsightsCloud
       def logger
         action_logger
       end
+
+      def rescue_strategy_for_self
+        Dynflow::Action::Rescue::Fail
+      end
     end
   end
 end

data/lib/insights_cloud/async/insights_full_sync.rb

@@ -118,6 +118,10 @@ module InsightsCloud
 
         match || { id: nil }
       end
+
+      def rescue_strategy_for_self
+        Dynflow::Action::Rescue::Fail
+      end
     end
   end
 end

data/lib/insights_cloud/async/insights_generate_notifications.rb

@@ -53,6 +53,10 @@ module InsightsCloud
           }
         )
       end
+
+      def rescue_strategy_for_self
+        Dynflow::Action::Rescue::Fail
+      end
     end
   end
 end

data/lib/insights_cloud/async/insights_resolutions_sync.rb

@@ -19,8 +19,9 @@ module InsightsCloud
       def run
         InsightsResolution.transaction do
           InsightsResolution.delete_all
-          api_response = query_insights_resolutions(relevant_rules)
-          write_resolutions(api_response)
+          rule_ids = relevant_rules
+          api_response = query_insights_resolutions(rule_ids) unless rule_ids.empty?
+          write_resolutions(api_response) if api_response
         end
       end
 
@@ -70,6 +71,10 @@ module InsightsCloud
       def to_rule_id(resolution_rule_id)
         RULE_ID_REGEX.match(resolution_rule_id).named_captures.fetch('id', resolution_rule_id)
       end
+
+      def rescue_strategy_for_self
+        Dynflow::Action::Rescue::Fail
+      end
     end
   end
 end

data/lib/insights_cloud/async/insights_rules_sync.rb

@@ -11,8 +11,12 @@ module InsightsCloud
           return
         end
 
-        plan_self
-        plan_resolutions
+        # since the tasks are not connected, we need to force sequence execution here
+        # to make sure we don't run resolutions until we synced all our rules
+        sequence do
+          plan_self
+          plan_resolutions
+        end
       end
 
       def plan_resolutions
@@ -71,6 +75,10 @@ module InsightsCloud
           rating:  rule_hash['rating'],
         }
       end
+
+      def rescue_strategy_for_self
+        Dynflow::Action::Rescue::Fail
+      end
     end
   end
 end