idrac 0.1.30 → 0.1.31

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6992752c6eee7bfb3c858d8bb465c68c68bff2008732ecfc88553ba7ac0524b5
-  data.tar.gz: ab1e816e1c1d903c3095c5131652f917a61b3c4006e0941fd05d24555230f9f5
+  metadata.gz: b2c0dae199436d9ce49a07eab55be6f44e9260ece8c64b07f7221384174a72de
+  data.tar.gz: 5b89e9ef776654053fe6ac4e8d6cd87ad121dfaa37194fd65b3cde2c5d44c683
 SHA512:
-  metadata.gz: b65362922f5a5728e6840a005ee7a79f2c08a835bba4ae7394eb304d2992426d9a9a52457173eb579e8e792413767c9249c64f5eacaee9dd47105971b10b3187
-  data.tar.gz: 8f97a86b72b0fb3544cb85c61f94b8228b037f63adeb345d9da856f7159d7e698c176fd303e212182dd779496b77c07b0b2efe83cc8d8fe8cee281060132d0e0
+  metadata.gz: 682eeec6204b0a7f105473fb0c61a44e11e067b48bd8db4c1f693d9895676ae7e8f1dfb6f655d19aeec421fb7fc83c1710508a78ab9f696b2adc6cb6b509bddb
+  data.tar.gz: bdfd016c0353de003a0182a6c5f582472f9f8071cf6f47a4fd448bccd62ac60c1747634b349a0afcd3422ec2b62842109b662ad500d6bdeee6cfeb55e9d6e47c

data/README.md

@@ -134,6 +134,16 @@ To install this gem onto your local machine, run `bundle exec rake install`. To
 
 ## Changelog
 
+### Version 0.1.31
+- **Enhanced Job Monitoring**: Completely redesigned the job monitoring system for firmware updates
+- Improved handling of HTTP 202 status codes during job status checks
+- Added detailed progress reporting with percentage complete during firmware updates
+- Enhanced job ID extraction with multiple fallback mechanisms to ensure proper job tracking
+- Improved error handling for common firmware update issues with clear user guidance
+- Added better handling of "deployment already in progress" scenarios
+- Implemented a more robust firmware download process with better error reporting
+- Enhanced the interactive update process with clearer user prompts and status messages
+
 ### Version 0.1.30
 - **Enhanced Terminal Output with Colors**: Added colorized output for better readability and user experience
 - Integrated the `colorize` gem to provide color-coded messages throughout the application

data/lib/idrac/firmware.rb

@@ -276,6 +276,7 @@ module IDRAC
     end
     
     def interactive_update(catalog_path = nil, selected_updates = nil)
+      # Check if updates are available
       updates = selected_updates || check_updates(catalog_path)
       
       if updates.empty?
@@ -283,261 +284,407 @@ module IDRAC
         return
       end
       
+      # Display available updates
       puts "\nAvailable Updates:".green.bold
       updates.each_with_index do |update, index|
         puts "#{index + 1}. #{update[:name]}: #{update[:current_version]} -> #{update[:available_version]}".light_cyan
       end
       
-      # If selected_updates is provided, use those directly
-      if selected_updates
-        selected = selected_updates
-      else
-        # Otherwise prompt the user for selection
+      # If no specific updates were selected, ask the user which ones to install
+      if selected_updates.nil?
         puts "\nEnter the number of the update to install (or 'all' for all updates, 'q' to quit):".light_yellow
-        choice = STDIN.gets.chomp
-        
-        return if choice.downcase == 'q'
-        
-        selected = if choice.downcase == 'all'
-                      updates
-                    else
-                      index = choice.to_i - 1
-                      if index >= 0 && index < updates.size
-                        [updates[index]]
-                      else
-                        puts "Invalid selection.".red
-                        return
-                      end
-                    end
+        input = STDIN.gets.chomp
+        
+        if input.downcase == 'q'
+          puts "Update cancelled.".yellow
+          return
+        elsif input.downcase == 'all'
+          selected_updates = updates
+        else
+          begin
+            index = input.to_i - 1
+            if index >= 0 && index < updates.length
+              selected_updates = [updates[index]]
+            else
+              puts "Invalid selection. Please enter a number between 1 and #{updates.length}.".red
+              return
+            end
+          rescue
+            puts "Invalid input. Please enter a number, 'all', or 'q'.".red
+            return
+          end
+        end
       end
       
-      selected.each do |update|
-        puts "Downloading #{update[:name]} version #{update[:available_version]}...".light_cyan.bold
-        
-        # Create temp directory
-        temp_dir = Dir.mktmpdir
+      # Process each selected update
+      selected_updates.each do |update|
+        puts "\nDownloading #{update[:name]} version #{update[:available_version]}...".light_cyan
         
         begin
-          # Download the update
-          update_filename = File.basename(update[:path])
-          update_path = File.join(temp_dir, update_filename)
+          # Download the firmware
+          firmware_file = download_firmware(update)
           
-          uri = URI.parse(update[:download_url])
-          Net::HTTP.start(uri.host, uri.port, use_ssl: true) do |http|
-            request = Net::HTTP::Get.new(uri)
-            http.request(request) do |response|
-              if response.code == "200"
-                File.open(update_path, 'wb') do |file|
-                  response.read_body do |chunk|
-                    file.write(chunk)
-                  end
+          if firmware_file
+            puts "Installing #{update[:name]} version #{update[:available_version]}...".light_cyan
+            
+            begin
+              # Upload and install the firmware
+              job_id = upload_firmware(firmware_file)
+              
+              if job_id
+                puts "Firmware update job created with ID: #{job_id}".green
+                
+                # Wait for the job to complete
+                success = wait_for_job_completion(job_id, 1800) # 30 minutes timeout
+                
+                if success
+                  puts "Successfully updated #{update[:name]} to version #{update[:available_version]}".green.bold
+                else
+                  puts "Failed to update #{update[:name]}. Check the iDRAC web interface for more details.".red
+                  puts "You may need to wait for any existing jobs to complete before trying again.".yellow
                 end
               else
-                puts "Failed to download update: #{response.code} #{response.message}".red
-                next
+                puts "Failed to create update job for #{update[:name]}".red
+              end
+            rescue IDRAC::Error => e
+              if e.message.include?("already in progress")
+                puts "Error: A firmware update is already in progress.".red.bold
+                puts "Please wait for the current update to complete before starting another.".yellow
+                puts "You can check the status in the iDRAC web interface under Maintenance > System Update.".light_cyan
+              elsif e.message.include?("job ID not found") || e.message.include?("Failed to get job status")
+                puts "Error: Could not monitor the update job.".red.bold
+                puts "The update may still be in progress. Check the iDRAC web interface for status.".yellow
+                puts "This can happen if the iDRAC is busy processing the update request.".light_cyan
+              else
+                puts "Error during firmware update: #{e.message}".red.bold
+              end
+              
+              # If we encounter an error with one update, ask if the user wants to continue with others
+              if selected_updates.length > 1 && update != selected_updates.last
+                puts "\nDo you want to continue with the remaining updates? (y/n)".light_yellow
+                continue = STDIN.gets.chomp.downcase
+                break unless continue == 'y'
               end
             end
+          else
+            puts "Failed to download firmware for #{update[:name]}".red
           end
+        rescue => e
+          puts "Error processing update for #{update[:name]}: #{e.message}".red.bold
           
-          puts "Installing #{update[:name]} version #{update[:available_version]}...".light_cyan.bold
+          # If we encounter an error with one update, ask if the user wants to continue with others
+          if selected_updates.length > 1 && update != selected_updates.last
+            puts "\nDo you want to continue with the remaining updates? (y/n)".light_yellow
+            continue = STDIN.gets.chomp.downcase
+            break unless continue == 'y'
+          end
+        ensure
+          # Clean up temporary files
+          FileUtils.rm_f(firmware_file) if firmware_file && File.exist?(firmware_file)
+        end
+      end
+    end
+
+    def download_firmware(update)
+      return false unless update && update[:download_url]
+      
+      begin
+        # Create a temporary directory for the download
+        temp_dir = Dir.mktmpdir
+        
+        # Extract the filename from the path
+        filename = File.basename(update[:path])
+        local_path = File.join(temp_dir, filename)
+        
+        puts "Downloading firmware from #{update[:download_url]}".light_cyan
+        puts "Saving to #{local_path}".light_cyan
+        
+        # Download the file
+        uri = URI.parse(update[:download_url])
+        
+        Net::HTTP.start(uri.host, uri.port, use_ssl: uri.scheme == 'https', verify_mode: OpenSSL::SSL::VERIFY_NONE) do |http|
+          request = Net::HTTP::Get.new(uri)
           
-          begin
-            job_id = update(update_path, wait: true)
-            puts "Update completed with job ID: #{job_id}".green.bold
-          rescue IDRAC::Error => e
-            if e.message.include?("already in progress")
-              puts "Error: #{e.message}".red.bold
-              puts "\nTips for resolving this issue:".yellow.bold
-              puts "1. Wait for any existing firmware updates to complete (check iDRAC web interface)".light_yellow
-              puts "2. Restart the iDRAC if no updates appear to be in progress (Settings > iDRAC Settings > Reset iDRAC)".light_yellow
-              puts "3. Try again after a few minutes".light_yellow
-            elsif e.message.include?("job ID") || e.message.include?("job not found")
-              puts "Error: #{e.message}".red.bold
-              puts "\nThe job ID could not be found or monitored. This could be because:".yellow
-              puts "1. The iDRAC is busy processing other requests".light_yellow
-              puts "2. The job was created but not properly tracked".light_yellow
-              puts "3. The iDRAC firmware may need to be updated first".light_yellow
-              puts "\nCheck the iDRAC web interface to see if the update was actually initiated.".light_cyan
+          http.request(request) do |response|
+            if response.code == "200"
+              File.open(local_path, 'wb') do |file|
+                response.read_body do |chunk|
+                  file.write(chunk)
+                end
+              end
+              
+              puts "Download completed successfully".green
+              return local_path
             else
-              puts "Error during firmware update: #{e.message}".red.bold
+              puts "Failed to download firmware: #{response.code} #{response.message}".red
+              return false
             end
           end
-          
-        ensure
-          # Clean up temp directory
-          FileUtils.remove_entry(temp_dir)
         end
+      rescue => e
+        puts "Error downloading firmware: #{e.message}".red.bold
+        return false
       end
     end
 
     private
 
     def upload_firmware(firmware_path)
-      puts "Uploading firmware file: #{firmware_path}"
-      
-      # Get the HttpPushUri from UpdateService
-      update_service_response = client.authenticated_request(:get, "/redfish/v1/UpdateService")
-      update_service_data = JSON.parse(update_service_response.body)
-      
-      http_push_uri = update_service_data['HttpPushUri']
-      if http_push_uri.nil?
-        http_push_uri = "/redfish/v1/UpdateService/FirmwareInventory"
-        puts "HttpPushUri not found, using default: #{http_push_uri}"
-      else
-        puts "Found HttpPushUri: #{http_push_uri}"
-      end
+      puts "Uploading firmware file: #{firmware_path}".light_cyan
       
-      # Get the ETag for the HttpPushUri
-      etag_response = client.authenticated_request(:get, http_push_uri)
-      etag = etag_response.headers['etag']
-      
-      puts "Got ETag: #{etag}"
-      
-      # Create a boundary for multipart/form-data
-      boundary = "----WebKitFormBoundary#{SecureRandom.hex(16)}"
-      
-      # Read the file content
-      file_content = File.binread(firmware_path)
-      filename = File.basename(firmware_path)
-      
-      # Create the multipart body
-      post_body = []
-      post_body << "--#{boundary}\r\n"
-      post_body << "Content-Disposition: form-data; name=\"file\"; filename=\"#{filename}\"\r\n"
-      post_body << "Content-Type: application/octet-stream\r\n\r\n"
-      post_body << file_content
-      post_body << "\r\n--#{boundary}--\r\n"
-      
-      # Upload the firmware
       begin
+        # First, get the HttpPushUri from the UpdateService
         response = client.authenticated_request(
+          :get,
+          "/redfish/v1/UpdateService"
+        )
+        
+        if response.status != 200
+          puts "Failed to get UpdateService information: #{response.status}".red
+          raise Error, "Failed to get UpdateService information: #{response.status}"
+        end
+        
+        update_service = JSON.parse(response.body)
+        http_push_uri = update_service['HttpPushUri']
+        
+        if http_push_uri.nil?
+          puts "HttpPushUri not found in UpdateService".red
+          raise Error, "HttpPushUri not found in UpdateService"
+        end
+        
+        puts "Found HttpPushUri: #{http_push_uri}".light_cyan
+        
+        # Get the ETag for the firmware inventory
+        etag_response = client.authenticated_request(
+          :get,
+          http_push_uri
+        )
+        
+        if etag_response.status != 200
+          puts "Failed to get ETag: #{etag_response.status}".red
+          raise Error, "Failed to get ETag: #{etag_response.status}"
+        end
+        
+        etag = etag_response.headers['ETag']
+        
+        if etag.nil?
+          puts "ETag not found in response headers".yellow
+          # Some iDRACs don't require ETag, so we'll continue
+        else
+          puts "Got ETag: #{etag}".light_cyan
+        end
+        
+        # Upload the firmware file
+        file_content = File.read(firmware_path)
+        
+        headers = {
+          'Content-Type' => 'application/octet-stream',
+          'If-Match' => etag
+        }
+        
+        upload_response = client.authenticated_request(
           :post,
           http_push_uri,
           {
-            headers: {
-              'Content-Type' => "multipart/form-data; boundary=#{boundary}",
-              'If-Match' => etag
-            },
-            body: post_body.join
+            headers: headers,
+            body: file_content
           }
         )
-      rescue => e
-        # Check if the error is about a deployment already in progress
-        if e.message.include?("A deployment or update operation is already in progress")
-          raise Error, "A firmware update is already in progress on the iDRAC. Please wait for it to complete before starting another update. You can check the status in the iDRAC web interface under Maintenance > System Update."
-        else
-          # Re-raise the original error
-          raise
+        
+        if upload_response.status != 201 && upload_response.status != 200
+          puts "Failed to upload firmware: #{upload_response.status} - #{upload_response.body}".red
+          
+          if upload_response.body.include?("already in progress")
+            raise Error, "A deployment or update operation is already in progress. Please wait for it to complete before attempting another update."
+          else
+            raise Error, "Failed to upload firmware: #{upload_response.status} - #{upload_response.body}"
+          end
         end
-      end
-      
-      if response.status < 200 || response.status >= 300
-        error_message = response.body.to_s
         
-        # Check for specific error messages in the response
-        if error_message.include?("A deployment or update operation is already in progress")
-          raise Error, "A firmware update is already in progress on the iDRAC. Please wait for it to complete before starting another update. You can check the status in the iDRAC web interface under Maintenance > System Update."
-        elsif error_message.include?("SUP0108")
-          raise Error, "iDRAC Error SUP0108: A deployment or update operation is already in progress. Wait for the operation to conclude and then try again."
+        # Extract the firmware ID from the response
+        begin
+          upload_data = JSON.parse(upload_response.body)
+          firmware_id = upload_data['Id'] || upload_data['@odata.id']&.split('/')&.last
+          
+          if firmware_id.nil?
+            # Try to extract from the Location header
+            location = upload_response.headers['Location']
+            firmware_id = location&.split('/')&.last
+          end
+          
+          if firmware_id.nil?
+            puts "Warning: Could not extract firmware ID from response".yellow
+            puts "Response body: #{upload_response.body}"
+            # We'll try to continue with the SimpleUpdate action anyway
+          else
+            puts "Firmware file uploaded successfully with ID: #{firmware_id}".green
+          end
+        rescue JSON::ParserError => e
+          puts "Warning: Could not parse upload response: #{e.message}".yellow
+          puts "Response body: #{upload_response.body}"
+          # We'll try to continue with the SimpleUpdate action anyway
+        end
+        
+        # Now initiate the firmware update using SimpleUpdate action
+        puts "Initiating firmware update using SimpleUpdate...".light_cyan
+        
+        # Construct the image URI
+        image_uri = nil
+        
+        if firmware_id
+          image_uri = "#{http_push_uri}/#{firmware_id}"
         else
-          raise Error, "Firmware upload failed with status #{response.status}: #{response.body}"
+          # If we couldn't extract the firmware ID, try using the Location header
+          image_uri = upload_response.headers['Location']
         end
-      end
-      
-      # Extract upload ID from response
-      response_data = JSON.parse(response.body)
-      upload_id = response_data['Id'] || response_data['TaskId']
-      
-      if upload_id.nil?
-        raise Error, "Failed to extract upload ID from firmware upload response"
-      end
-      
-      puts "Firmware file uploaded successfully with ID: #{upload_id}"
-      
-      # Step 2: Initiate the firmware update using SimpleUpdate
-      puts "Initiating firmware update using SimpleUpdate..."
-      
-      # Construct the image URI using the uploaded file
-      image_uri = "#{http_push_uri}/#{upload_id}"
-      puts "Using ImageURI: #{image_uri}"
-      
-      # Call the SimpleUpdate action
-      simple_update_response = client.authenticated_request(
-        :post,
-        "/redfish/v1/UpdateService/Actions/UpdateService.SimpleUpdate",
-        {
-          headers: {
-            'Content-Type' => 'application/json'
-          },
-          body: JSON.generate({
-            'ImageURI' => image_uri,
-            '@Redfish.OperationApplyTime' => 'Immediate'
-          })
+        
+        # If we still don't have an image URI, try to use the HTTP push URI as a fallback
+        if image_uri.nil?
+          puts "Warning: Could not determine image URI, using HTTP push URI as fallback".yellow
+          image_uri = http_push_uri
+        end
+        
+        puts "Using ImageURI: #{image_uri}".light_cyan
+        
+        # Initiate the SimpleUpdate action
+        simple_update_payload = {
+          "ImageURI" => image_uri,
+          "TransferProtocol" => "HTTP"
         }
-      )
-      
-      if simple_update_response.status < 200 || simple_update_response.status >= 300
-        raise Error, "Firmware update initiation failed with status #{simple_update_response.status}: #{simple_update_response.body}"
-      end
-      
-      # Extract job ID from response
-      job_id = nil
-      
-      # Try to get job ID from Location header
-      if simple_update_response.headers['location']
-        job_id = simple_update_response.headers['location'].split('/').last
-      end
-      
-      # If not found in header, try to get from response body
-      if job_id.nil? && !simple_update_response.body.empty?
+        
+        update_response = client.authenticated_request(
+          :post,
+          "/redfish/v1/UpdateService/Actions/UpdateService.SimpleUpdate",
+          {
+            headers: { 'Content-Type' => 'application/json' },
+            body: simple_update_payload.to_json
+          }
+        )
+        
+        if update_response.status != 202 && update_response.status != 200
+          puts "Failed to initiate firmware update: #{update_response.status} - #{update_response.body}".red
+          raise Error, "Failed to initiate firmware update: #{update_response.status} - #{update_response.body}"
+        end
+        
+        # Extract the job ID from the response
+        job_id = nil
+        
+        # Try to extract from the response body first
         begin
-          update_data = JSON.parse(simple_update_response.body)
-          if update_data['@odata.id']
-            job_id = update_data['@odata.id'].split('/').last
-          elsif update_data['Id']
-            job_id = update_data['Id']
-          end
+          update_data = JSON.parse(update_response.body)
+          job_id = update_data['Id'] || update_data['JobID']
         rescue JSON::ParserError
-          # Not JSON, ignore
+          # If we can't parse the body, that's okay, we'll try other methods
         end
+        
+        # If we couldn't get the job ID from the body, try the Location header
+        if job_id.nil?
+          location = update_response.headers['Location']
+          job_id = location&.split('/')&.last
+        end
+        
+        # If we still don't have a job ID, try the response headers
+        if job_id.nil?
+          # Some iDRACs return the job ID in a custom header
+          update_response.headers.each do |key, value|
+            if key.downcase.include?('job') && value.is_a?(String) && value.match?(/JID_\d+/)
+              job_id = value
+              break
+            end
+          end
+        end
+        
+        # If we still don't have a job ID, check for any JID_ pattern in the response body
+        if job_id.nil? && update_response.body.is_a?(String)
+          match = update_response.body.match(/JID_\d+/)
+          job_id = match[0] if match
+        end
+        
+        # If we still don't have a job ID, check the task service for recent jobs
+        if job_id.nil?
+          puts "Could not extract job ID from response, checking task service for recent jobs...".yellow
+          
+          tasks_response = client.authenticated_request(
+            :get,
+            "/redfish/v1/TaskService/Tasks"
+          )
+          
+          if tasks_response.status == 200
+            begin
+              tasks_data = JSON.parse(tasks_response.body)
+              
+              if tasks_data['Members'] && tasks_data['Members'].any?
+                # Get the most recent task
+                most_recent_task = tasks_data['Members'].first
+                task_id = most_recent_task['@odata.id']&.split('/')&.last
+                
+                if task_id && task_id.match?(/JID_\d+/)
+                  job_id = task_id
+                  puts "Found recent job ID: #{job_id}".light_cyan
+                end
+              end
+            rescue JSON::ParserError
+              # If we can't parse the tasks response, we'll have to give up
+            end
+          end
+        end
+        
+        if job_id.nil?
+          puts "Could not extract job ID from response".red
+          raise Error, "Could not extract job ID from response"
+        end
+        
+        puts "Firmware update job created with ID: #{job_id}".green
+        return job_id
+      rescue => e
+        puts "Error during firmware upload: #{e.message}".red.bold
+        raise Error, "Error during firmware upload: #{e.message}"
       end
-      
-      # If still no job ID, use the upload ID as a fallback
-      if job_id.nil?
-        job_id = upload_id
-        puts "No job ID found in SimpleUpdate response, using upload ID as job ID"
-      end
-      
-      puts "Firmware update job created with ID: #{job_id}"
-      job_id
     end
 
     def wait_for_job_completion(job_id, timeout)
-      puts "Waiting for firmware update job #{job_id} to complete..."
+      puts "Waiting for firmware update job #{job_id} to complete...".light_cyan
       
       start_time = Time.now
-      loop do
-        status = get_job_status(job_id)
-        
-        case status
-        when 'Completed'
-          puts "Firmware update completed successfully"
-          return true
-        when 'Failed'
-          raise Error, "Firmware update job failed"
-        when 'Scheduled', 'Running', 'Downloading', 'Pending'
-          # Job still in progress
-        else
-          puts "Unknown job status: #{status}"
-        end
-        
-        if Time.now - start_time > timeout
-          raise Error, "Firmware update timed out after #{timeout} seconds"
+      last_percent = -1
+      
+      while Time.now - start_time < timeout
+        begin
+          status = get_job_status(job_id)
+          
+          # Only show percentage updates when they change
+          if status[:percent_complete] && status[:percent_complete] != last_percent
+            puts "Job progress: #{status[:percent_complete]}% complete".light_cyan
+            last_percent = status[:percent_complete]
+          end
+          
+          case status[:state]
+          when 'Completed'
+            puts "Firmware update completed successfully".green
+            return true
+          when 'Failed', 'CompletedWithErrors'
+            message = status[:message] || "Unknown error"
+            puts "Firmware update failed: #{message}".red.bold
+            return false
+          when 'Stopped'
+            puts "Firmware update stopped".yellow
+            return false
+          when 'New', 'Starting', 'Running', 'Pending', 'Scheduled', 'Downloaded', 'Downloading', 'Staged'
+            # Job still in progress, continue waiting
+            sleep 10
+          else
+            puts "Unknown job status: #{status[:state]}".yellow
+            sleep 10
+          end
+        rescue => e
+          puts "Error checking job status: #{e.message}".red
+          puts "Will retry in 15 seconds...".yellow
+          sleep 15
         end
-        
-        # Wait before checking again
-        sleep 10
       end
+      
+      puts "Timeout waiting for firmware update to complete".red.bold
+      false
     end
 
     def get_job_status(job_id)
@@ -546,12 +693,37 @@ module IDRAC
         "/redfish/v1/TaskService/Tasks/#{job_id}"
       )
       
-      if response.status != 200
-        raise Error, "Failed to get job status with status #{response.status}: #{response.body}"
+      # Status 202 means the request was accepted but still processing
+      # This is normal for jobs that are in progress
+      if response.status == 202 || response.status == 200
+        begin
+          data = JSON.parse(response.body)
+          
+          # Extract job state and percent complete
+          job_state = data.dig('Oem', 'Dell', 'JobState') || data['TaskState']
+          percent_complete = data.dig('Oem', 'Dell', 'PercentComplete') || data['PercentComplete']
+          
+          # Format the percent complete for display
+          percent_str = percent_complete.nil? ? "unknown" : "#{percent_complete}%"
+          
+          puts "Job #{job_id} status: #{job_state} (#{percent_str} complete)".light_cyan
+          
+          return {
+            id: job_id,
+            state: job_state,
+            percent_complete: percent_complete,
+            status: data['TaskStatus'],
+            message: data.dig('Oem', 'Dell', 'Message') || (data['Messages'].first && data['Messages'].first['Message']),
+            raw_data: data
+          }
+        rescue JSON::ParserError => e
+          puts "Error parsing job status response: #{e.message}".red.bold
+          raise Error, "Failed to parse job status response: #{e.message}"
+        end
+      else
+        puts "Failed to get job status with status #{response.status}: #{response.body}".red
+        raise Error, "Failed to get job status with status #{response.status}"
       end
-      
-      response_data = JSON.parse(response.body)
-      response_data['TaskState'] || 'Unknown'
     end
 
     # Helper method to extract identifiers from component names

data/lib/idrac/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module IDRAC
-  VERSION = "0.1.30"
+  VERSION = "0.1.31"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: idrac
 version: !ruby/object:Gem::Version
-  version: 0.1.30
+  version: 0.1.31
 platform: ruby
 authors:
 - Jonathan Siegel