supermicro 0.1.7 → 0.1.8

data/lib/supermicro/system.rb

@@ -12,7 +12,19 @@ module Supermicro
         begin
           data = JSON.parse(response.body)
           
-          memory = data["Members"].map do |m|
+          memory = data["Members"].map do |member|
+            # If member is just a reference, fetch the full data
+            if member["@odata.id"] && !member["CapacityMiB"]
+              mem_response = authenticated_request(:get, member["@odata.id"])
+              if mem_response.status == 200
+                m = JSON.parse(mem_response.body)
+              else
+                next nil
+              end
+            else
+              m = member
+            end
+            
             next if m["CapacityMiB"].nil? || m["CapacityMiB"] == 0
             
             dimm_name = m["DeviceLocator"] || m["Name"]
@@ -26,16 +38,17 @@ module Supermicro
             end
             
             {
-              "model" => m["Manufacturer"],
+              "manufacturer" => m["Manufacturer"],
               "name" => dimm_name,
               "capacity_bytes" => m["CapacityMiB"].to_i * 1024 * 1024,
               "health" => m.dig("Status", "Health") || "N/A",
-              "speed_mhz" => m["OperatingSpeedMhz"],
+              "speed_mhz" => m["OperatingSpeedMHz"],
               "part_number" => m["PartNumber"],
               "serial" => m["SerialNumber"],
               "bank" => bank,
               "index" => index.to_i,
-              "memory_type" => m["MemoryDeviceType"]
+              "memory_device_type" => m["MemoryDeviceType"],
+              "base_module_type" => m["BaseModuleType"]
             }
           end.compact
           
@@ -99,6 +112,7 @@ module Supermicro
                 "name" => fan["Name"],
                 "rpm" => rpm,
                 "status" => health,
+                "state" => fan.dig("Status", "State"),
                 "min_rpm" => fan["MinReadingRange"],
                 "max_rpm" => fan["MaxReadingRange"]
               }
@@ -167,24 +181,52 @@ module Supermicro
     end
 
     def nics
-      response = authenticated_request(:get, "/redfish/v1/Systems/1/EthernetInterfaces?$expand=*($levels=1)")
+      response = authenticated_request(:get, "/redfish/v1/Systems/1/EthernetInterfaces")
       
       if response.status == 200
         begin
           data = JSON.parse(response.body)
           
-          nics = data["Members"].map do |nic|
+          # If Members are not expanded, fetch each interface individually
+          nics = data["Members"].map do |member|
+            # Check if this is just a reference or full data
+            if member["@odata.id"] && !member["Id"]
+              # It's just a reference, fetch the full data
+              interface_response = authenticated_request(:get, member["@odata.id"])
+              if interface_response.status == 200
+                nic = JSON.parse(interface_response.body)
+              else
+                puts "Failed to fetch NIC #{member["@odata.id"]}: #{interface_response.status}".yellow
+                next
+              end
+            else
+              # We already have the full data
+              nic = member
+            end
+            
+            # Create adapter structure to match iDRAC format
             {
-              "id" => nic["Id"],
               "name" => nic["Name"],
-              "mac" => nic["MACAddress"],
-              "speed_mbps" => nic["SpeedMbps"],
-              "status" => nic.dig("Status", "Health") || "N/A",
-              "link_status" => nic["LinkStatus"],
-              "ipv4" => nic.dig("IPv4Addresses", 0, "Address"),
-              "ipv6" => nic.dig("IPv6Addresses", 0, "Address")
+              "manufacturer" => nic["Manufacturer"] || "Supermicro",
+              "model" => nil,
+              "part_number" => nil,
+              "serial" => nil,
+              "ports" => [
+                {
+                  "name" => nic["Id"],
+                  "status" => nic["LinkStatus"] == "LinkDown" ? "Down" : "Up",
+                  "mac" => nic["MACAddress"],
+                  "ipv4" => nic.dig("IPv4Addresses", 0, "Address"),
+                  "mode" => nic.dig("IPv4Addresses", 0, "AddressOrigin"), # DHCP or Static
+                  "mask" => nic.dig("IPv4Addresses", 0, "SubnetMask"),
+                  "port" => 0,
+                  "speed_mbps" => nic["SpeedMbps"] || 0,
+                  "kind" => "ethernet",
+                  "linux_device" => nil
+                }
+              ]
             }
-          end
+          end.compact  # Remove any nil entries from failed fetches
           
           return nics
         rescue JSON::ParserError
@@ -202,12 +244,21 @@ module Supermicro
         begin
           data = JSON.parse(response.body)
           
+          # Get Manager UUID for service tag (BMC MAC address)
+          manager_uuid = nil
+          manager_response = authenticated_request(:get, "/redfish/v1/Managers/1")
+          if manager_response.status == 200
+            manager_data = JSON.parse(manager_response.body)
+            manager_uuid = manager_data["UUID"]
+          end
+          
           {
             "name" => data["Name"],
             "model" => data["Model"],
             "manufacturer" => data["Manufacturer"],
             "serial" => data["SerialNumber"],
             "uuid" => data["UUID"],
+            "manager_uuid" => manager_uuid,
             "bios_version" => data["BiosVersion"],
             "power_state" => data["PowerState"],
             "health" => data.dig("Status", "Health"),
@@ -254,6 +305,45 @@ module Supermicro
       data = power_consumption
       data["consumed_watts"] if data.is_a?(Hash)
     end
+    
+    # Get system health status
+    def system_health
+      response = authenticated_request(:get, "/redfish/v1/Systems/1")
+      
+      if response.status == 200
+        begin
+          data = JSON.parse(response.body)
+          
+          health = {
+            "rollup" => data.dig("Status", "HealthRollup") || data.dig("Status", "Health"),
+            "system" => data.dig("Status", "Health"),
+            "processor" => data.dig("ProcessorSummary", "Status", "Health"),
+            "memory" => data.dig("MemorySummary", "Status", "Health")
+          }
+          
+          # Try to get storage health
+          storage_response = authenticated_request(:get, "/redfish/v1/Systems/1/Storage")
+          if storage_response.status == 200
+            storage_data = JSON.parse(storage_response.body)
+            if storage_data["Members"] && !storage_data["Members"].empty?
+              # Get first storage controller's health
+              storage_url = storage_data["Members"].first["@odata.id"]
+              storage_detail = authenticated_request(:get, storage_url)
+              if storage_detail.status == 200
+                storage_detail_data = JSON.parse(storage_detail.body)
+                health["storage"] = storage_detail_data.dig("Status", "Health")
+              end
+            end
+          end
+          
+          health
+        rescue JSON::ParserError
+          raise Error, "Failed to parse system health information: #{response.body}"
+        end
+      else
+        raise Error, "Failed to get system health. Status code: #{response.status}"
+      end
+    end
 
     def temperatures
       response = authenticated_request(:get, "/redfish/v1/Chassis/1/Thermal")

data/lib/supermicro/system_config.rb

@@ -5,81 +5,8 @@ require 'colorize'
 
 module Supermicro
   module SystemConfig
-    def bios_attributes
-      response = authenticated_request(:get, "/redfish/v1/Systems/1/Bios")
-      
-      if response.status == 200
-        begin
-          data = JSON.parse(response.body)
-          
-          {
-            "attributes" => data["Attributes"],
-            "attribute_registry" => data["AttributeRegistry"]
-          }
-        rescue JSON::ParserError
-          raise Error, "Failed to parse BIOS attributes response: #{response.body}"
-        end
-      else
-        raise Error, "Failed to get BIOS attributes. Status code: #{response.status}"
-      end
-    end
-
-    def set_bios_attribute(attribute_name, value)
-      puts "Setting BIOS attribute #{attribute_name} to #{value}...".yellow
-      
-      body = {
-        "Attributes" => {
-          attribute_name => value
-        }
-      }
-      
-      response = authenticated_request(
-        :patch,
-        "/redfish/v1/Systems/1/Bios/Settings",
-        body: body.to_json,
-        headers: { 'Content-Type': 'application/json' }
-      )
-      
-      if response.status.between?(200, 299)
-        puts "BIOS attribute set successfully. Changes will be applied on next reboot.".green
-        return true
-      else
-        raise Error, "Failed to set BIOS attribute: #{response.status} - #{response.body}"
-      end
-    end
-
-    def pending_bios_settings
-      response = authenticated_request(:get, "/redfish/v1/Systems/1/Bios/Settings")
-      
-      if response.status == 200
-        begin
-          data = JSON.parse(response.body)
-          data["Attributes"] || {}
-        rescue JSON::ParserError
-          raise Error, "Failed to parse pending BIOS settings response: #{response.body}"
-        end
-      else
-        {}
-      end
-    end
-
-    def reset_bios_defaults
-      puts "Resetting BIOS to defaults...".yellow
-      
-      response = authenticated_request(
-        :post,
-        "/redfish/v1/Systems/1/Bios/Actions/Bios.ResetBios",
-        body: {}.to_json,
-        headers: { 'Content-Type': 'application/json' }
-      )
-      
-      if response.status.between?(200, 299)
-        puts "BIOS reset to defaults successfully. Changes will be applied on next reboot.".green
-        return true
-      else
-        raise Error, "Failed to reset BIOS: #{response.status} - #{response.body}"
-      end
-    end
+    # BIOS methods have been moved to the Bios module for better organization
+    # Use client.bios_attributes, client.set_bios_attribute, etc. instead
 
     def manager_network_protocol
       response = authenticated_request(:get, "/redfish/v1/Managers/1/NetworkProtocol")

data/lib/supermicro/tasks.rb

@@ -61,6 +61,37 @@ module Supermicro
               sleep 1
               next
             end
+          elsif task_response.status == 404
+            # TaskMonitor endpoint not found - this is an error
+            spinner&.stop("Task endpoint not found", success: false)
+            debug "Task endpoint returned 404 - cannot monitor task", 1, :red
+            return { success: false, error: 'task_endpoint_not_found' }
+          elsif task_response.status == 400
+            # 400 can indicate task completion with error details in body
+            begin
+              if task_response.body && !task_response.body.empty?
+                task_info = JSON.parse(task_response.body)
+                
+                # Check if this is a completed task with error
+                if task_info['TaskState'] == 'Exception'
+                  spinner&.stop("Task failed: #{task_info['Message']}", success: false)
+                  debug "✗ Task failed: #{task_info['Message']}", 1, :red
+                  return { success: false, task: task_info, error: task_info['Message'] }
+                elsif task_info['TaskState'] == 'Completed'
+                  spinner&.stop("Task completed", success: true)
+                  debug "✓ Task completed", 2, :green
+                  return { success: true, task: task_info }
+                else
+                  # Unknown 400 response
+                  debug "Unexpected 400 response with TaskState: #{task_info['TaskState']}", 2, :yellow
+                  sleep 1
+                  next
+                end
+              end
+            rescue JSON::ParserError
+              debug "400 response but couldn't parse body: #{task_response.body}", 1, :red
+              return { success: false, error: 'bad_request' }
+            end
           else
             debug "Unexpected task response: #{task_response.status}", 2, :yellow
             sleep 1
@@ -121,7 +152,14 @@ module Supermicro
       if !task_location && response.body && !response.body.empty?
         begin
           task_data = JSON.parse(response.body)
-          task_location = task_data['@odata.id'] || task_data['TaskMonitor']
+          # Prefer @odata.id over TaskMonitor as TaskMonitor may return 404
+          task_location = task_data['@odata.id']
+          
+          # Only use TaskMonitor if @odata.id is not available
+          if !task_location && task_data['TaskMonitor']
+            debug "Using TaskMonitor endpoint (may not be supported): #{task_data['TaskMonitor']}", 2, :yellow
+            task_location = task_data['TaskMonitor']
+          end
         rescue JSON::ParserError
           # No task info in body
         end

data/lib/supermicro/utility.rb

@@ -6,44 +6,80 @@ require 'colorize'
 module Supermicro
   module Utility
     def sel_log
-      response = authenticated_request(:get, "/redfish/v1/Managers/1/LogServices/SEL/Entries?$expand=*($levels=1)")
+      # Supermicro uses Systems/1/LogServices/Log1/Entries for system health event logs
+      # Try Systems log first (health events)
+      response = authenticated_request(:get, "/redfish/v1/Systems/1/LogServices/Log1/Entries?$expand=*($levels=1)")
       
       if response.status == 200
-        begin
-          data = JSON.parse(response.body)
-          
-          entries = data["Members"]&.map do |entry|
-            {
-              "id" => entry["Id"],
-              "name" => entry["Name"],
-              "created" => entry["Created"],
-              "severity" => entry["Severity"],
-              "message" => entry["Message"],
-              "message_id" => entry["MessageId"],
-              "sensor_type" => entry["SensorType"],
-              "sensor_number" => entry["SensorNumber"]
-            }
-          end || []
-          
-          return entries.sort_by { |e| e["created"] || "" }.reverse
-        rescue JSON::ParserError
-          raise Error, "Failed to parse SEL log response: #{response.body}"
+        debug "Retrieved system health event logs", 2, :green
+        return parse_log_entries(response)
+      elsif response.status == 404
+        # Fallback to Manager logs (maintenance events)
+        debug "Systems log not found, trying Manager logs", 2, :yellow
+        response = authenticated_request(:get, "/redfish/v1/Managers/1/LogServices/Log1/Entries?$expand=*($levels=1)")
+        
+        if response.status == 200
+          debug "Retrieved manager maintenance logs", 2, :green
+          return parse_log_entries(response)
+        elsif response.status == 404
+          debug "No log services available on this system", 1, :yellow
+          return []
         end
-      else
-        raise Error, "Failed to get SEL log. Status code: #{response.status}"
       end
+      
+      if response.status != 200
+        debug "Failed to get system logs. Status code: #{response.status}", 1, :yellow
+        return []
+      end
+    end
+    
+    private
+    
+    def parse_log_entries(response)
+      data = JSON.parse(response.body)
+      
+      entries = data["Members"]&.map do |entry|
+        {
+          "id" => entry["Id"],
+          "name" => entry["Name"],
+          "created" => entry["Created"],
+          "severity" => entry["Severity"],
+          "message" => entry["Message"],
+          "message_id" => entry["MessageId"],
+          "sensor_type" => entry["SensorType"],
+          "sensor_number" => entry["SensorNumber"]
+        }
+      end || []
+      
+      return entries.sort_by { |e| e["created"] || "" }.reverse
+    rescue JSON::ParserError
+      debug "Failed to parse log response", 1, :yellow
+      return []
     end
+    
+    public
 
     def clear_sel_log
       puts "Clearing System Event Log...".yellow
       
+      # Try to clear System health logs first
       response = authenticated_request(
         :post,
-        "/redfish/v1/Managers/1/LogServices/SEL/Actions/LogService.ClearLog",
+        "/redfish/v1/Systems/1/LogServices/Log1/Actions/LogService.ClearLog",
         body: {}.to_json,
         headers: { 'Content-Type': 'application/json' }
       )
       
+      if response.status == 404
+        # Fallback to Manager logs
+        response = authenticated_request(
+          :post,
+          "/redfish/v1/Managers/1/LogServices/Log1/Actions/LogService.ClearLog",
+          body: {}.to_json,
+          headers: { 'Content-Type': 'application/json' }
+        )
+      end
+      
       if response.status.between?(200, 299)
         puts "SEL cleared successfully.".green
         return true

data/lib/supermicro/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Supermicro
-  VERSION = "0.1.7"
+  VERSION = "0.1.8"
 end

data/lib/supermicro/virtual_media.rb

@@ -155,6 +155,7 @@ module Supermicro
       
       tries = 0
       max_tries = 3
+      last_error = nil
       
       while tries < max_tries
         begin
@@ -233,21 +234,26 @@ module Supermicro
                 inserted_media = current_media.find { |m| m[:device] == device }
                 if inserted_media && inserted_media[:image] == iso_url
                   if inserted_media[:connected_via] == "NotConnected"
-                    debug "ERROR: Media mounted but NOT CONNECTED!", 1, :red
-                    debug "The ISO will NOT boot! ConnectedVia must be 'URI', not 'NotConnected'.", 1, :red
-                    return false
+                    error_msg = "Media mounted but NOT CONNECTED! The ISO will NOT boot! ConnectedVia must be 'URI', not 'NotConnected'."
+                    debug "ERROR: #{error_msg}", 1, :red
+                    raise Error, error_msg
                   else
                     debug "Media mounted with status: #{inserted_media[:connected_via]}", 1, :green
                     return true
                   end
                 else
-                  debug "Failed to verify media mount.", 1, :red
-                  return false
+                  error_msg = "Failed to verify media mount - media not found on device #{device}"
+                  debug error_msg, 1, :red
+                  raise Error, error_msg
                 end
               end
             else
               debug "Task failed or timed out", 1, :red
-              return false
+              last_error = task_result[:error] || "Task failed with unknown error"
+              debug "Task error details: #{last_error}", 1, :red
+              tries += 1
+              sleep 2
+              next
             end
           elsif response.status.between?(200, 299)
             # Synchronous success (rare)
@@ -261,14 +267,16 @@ module Supermicro
                 debug "✓ Media connected via URI", 1, :green
                 return true
               elsif inserted_media[:connected_via] == "NotConnected"
-                debug "WARNING: Media mounted but not connected!", 1, :red
-                return false
+                error_msg = "Media mounted but not connected! ConnectedVia is 'NotConnected' - ISO will not boot"
+                debug "ERROR: #{error_msg}", 1, :red
+                raise Error, error_msg
               else
                 debug "Media mounted with status: #{inserted_media[:connected_via]}", 1, :yellow
                 return true
               end
             else
-              debug "Could not verify media mount", 1, :yellow
+              # If we can't verify, still treat as success but log warning
+              debug "WARNING: Could not verify media mount status", 1, :yellow
               return true
             end
           elsif response.status == 400 && response.body.include?("already")
@@ -278,19 +286,54 @@ module Supermicro
             tries += 1
           else
             debug "Failed to insert media: #{response.status} - #{response.body}", 1, :red
+            last_error = "HTTP #{response.status}: #{response.body}"
             tries += 1
             sleep 2
           end
         rescue => e
           debug "Error inserting media: #{e.message}", 1, :red
+          last_error = e.message
           tries += 1
           sleep 2
         end
       end
       
-      raise Error, "Failed to insert virtual media after #{max_tries} attempts"
+      error_msg = "Failed to insert virtual media after #{max_tries} attempts"
+      error_msg += ": #{last_error}" if last_error
+      raise Error, error_msg
     end
 
+    def test_iso_accessibility(iso_url)
+      debug "Testing if BMC can reach ISO URL: #{iso_url}", 1, :yellow
+      
+      # Try to use Redfish's built-in validation if available
+      path = "/redfish/v1/Managers/1/VirtualMedia/test"
+      body = {
+        "Image" => iso_url,
+        "TestOnly" => true
+      }
+      
+      begin
+        response = authenticated_request(
+          :post,
+          path,
+          body: body.to_json,
+          headers: { 'Content-Type': 'application/json' }
+        )
+        
+        if response.status == 200
+          debug "✓ BMC can reach ISO URL", 1, :green
+          return true
+        else
+          debug "✗ BMC cannot reach ISO URL: #{response.body}", 1, :red
+          return false
+        end
+      rescue => e
+        debug "Cannot test ISO accessibility: #{e.message}", 2, :yellow
+        return nil
+      end
+    end
+    
     def find_best_virtual_media_device
       media_list = virtual_media
       

data/lib/supermicro.rb

@@ -49,6 +49,7 @@ require 'supermicro/system'
 require 'supermicro/tasks'
 require 'supermicro/virtual_media'
 require 'supermicro/boot'
+require 'supermicro/bios'
 require 'supermicro/system_config'
 require 'supermicro/utility'
 require 'supermicro/license'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: supermicro
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.1.8
 platform: ruby
 authors:
 - Jonathan Siegel
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-08-29 00:00:00.000000000 Z
+date: 2025-09-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
@@ -147,6 +147,7 @@ files:
 - LICENSE
 - README.md
 - lib/supermicro.rb
+- lib/supermicro/bios.rb
 - lib/supermicro/boot.rb
 - lib/supermicro/client.rb
 - lib/supermicro/error.rb