Redesign inventory upload UI with improved status tracking (#1130)

Update inventory upload dashboard to display task status using new
generated_status and uploaded_status fields. Remove unused database
tables and improve test coverage.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Sonnet 4.5 <[email protected]>

Author: jeremylenz

.eslintrc

@@ -13,33 +13,42 @@
         "skipWords": [
           "advisor",
           "ansible",
+          "auditd",
           "camelcase",
           "cdn",
           "checkbox",
+          "cpol",
           "csrf",
+          "cryptopol",
           "cve",
           "cves",
-          "dropdown",
+          "db",
           "donut",
+          "dropdown",
+          "hardening",
+          "hostid",
           "hostnames",
           "href",
           "iop",
           "ips",
           "ipv4",
-          "dropdown",
           "jed",
           "katello",
           "knowledgebase",
+          "krb5",
           "noopener",
           "noreferrer",
           "nowrap",
+          "openssh",
           "pid",
           "redhat",
           "redux",
+          "rebootable",
           "remediate",
           "remediations",
           "repo",
           "rhc",
+          "rulename",
           "scalprum",
           "theforeman",
           "tooltip",

app/controllers/foreman_inventory_upload/accounts_controller.rb

@@ -52,18 +52,6 @@ def latest_task_for(org_id, job_class)
         .first
     end
 
-    def task_status_string(task)
-      return nil unless task
-
-      if task.state == 'stopped'
-        # Mimic old ProgressOutput format: "pid 12345 exit 0"
-        exit_code = task.result == 'success' ? 0 : 1
-        "pid #{Process.pid} exit #{exit_code}"
-      else
-        task.state
-      end
-    end
-
     def sub_action_status(task, action_class_name)
       return nil unless task
 
@@ -72,8 +60,7 @@ def sub_action_status(task, action_class_name)
 
       # For GenerateHostReport: always show status if task completed (generation always runs)
       if action_class_name == 'GenerateHostReport'
-        exit_code = task.result == 'success' ? 0 : 1
-        return "pid #{Process.pid} exit #{exit_code}"
+        return task.result
       end
 
       # For UploadReportDirectJob: only show status if task had upload enabled
@@ -84,9 +71,8 @@ def sub_action_status(task, action_class_name)
         # Check if upload was enabled for this task
         return nil unless main_action.input[:upload]
 
-        # Show same status as overall task (upload runs as part of the task)
-        exit_code = task.result == 'success' ? 0 : 1
-        return "pid #{Process.pid} exit #{exit_code}"
+        # Return the task result
+        return task.result
       end
 
       nil

app/models/task_output_line.rb

@@ -0,0 +1,24 @@
+class DropTaskOutputTables < ActiveRecord::Migration[6.0]
+  def up
+    drop_table :task_output_lines
+    drop_table :task_output_statuses
+  end
+
+  def down
+    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.timestamps
+
+      t.index :label, unique: true
+    end
+  end
+end

app/models/task_output_status.rb

@@ -42,7 +42,14 @@ def self.output_label(label)
         "upload_for_#{label}"
       end
 
+      def resource_locks
+        :link
+      end
+
       def plan(filename, organization_id)
+        organization = Organization.find(organization_id)
+        action_subject(organization)
+
         plan_self(
           filename: filename,
           organization_id: organization_id
@@ -102,7 +109,10 @@ def upload_file(cer_path)
       def move_to_done_folder
         FileUtils.mkdir_p(ForemanInventoryUpload.done_folder)
         done_file = ForemanInventoryUpload.done_file_path(File.basename(filename))
-        FileUtils.mv(filename, done_file)
+        if File.exist?(done_file)
+          logger.warn("Destination file #{done_file} already exists. Overwriting with new report.")
+        end
+        FileUtils.mv(filename, done_file, force: true)
         logger.debug("Moved #{filename} to #{done_file}")
       end
 
@@ -119,9 +129,20 @@ def manifest_certificate
       end
 
       def foreman_certificate
+        cert_path = Setting[:ssl_certificate]
+        key_path = Setting[:ssl_priv_key]
+
+        unless cert_path && File.readable?(cert_path)
+          raise "SSL certificate file not found or not readable: #{cert_path}"
+        end
+
+        unless key_path && File.readable?(key_path)
+          raise "SSL private key file not found or not readable: #{key_path}"
+        end
+
         {
-          cert: File.read(Setting[:ssl_certificate]),
-          key: File.read(Setting[:ssl_priv_key]),
+          cert: File.read(cert_path),
+          key: File.read(key_path),
         }
       end
 
@@ -134,6 +155,7 @@ def organization
       end
 
       def content_disconnected?
+        return false if ForemanRhCloud.with_iop_smart_proxy?
         !Setting[:subscription_connection_enabled]
       end
 

db/migrate/20251209163012_drop_task_output_tables.foreman_rh_cloud.rb

@@ -5,7 +5,7 @@ class AccountsControllerTest < ActionController::TestCase
 
   include FolderIsolation
 
-  test 'Returns statuses for each process type' do
+  test 'Returns statuses for each organization' do
     test_org = FactoryBot.create(:organization)
 
     get :index, session: set_session_user