idrac 0.4.0 → 0.4.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 04a47e02d37a42167e7007aa92fc8f1d1b4023a52d44bb65ed1132a2035c110a
-  data.tar.gz: d17c640f8c8c238a6ea6aa4c037f39d2a9c2bfeaaebc478b1a464c75a648bbc4
+  metadata.gz: ffcb1d99822a0e7d0015f018b6967e3977d7506438a3e2d917e0615f281aa0e3
+  data.tar.gz: 188a912ccdcdb5712ead6b5f181d59902f9b50718e3c3e39e2949f6fbc7ad606
 SHA512:
-  metadata.gz: a64a9f69bf757578c9d3fb6d95aeb54b33b4a6604ba432a0b52cdca1e0fdbb2451b19191e2ebbe2468a025648a985a8e5122559f04063db7be8fb20b9a997759
-  data.tar.gz: 867beff66c1896abe949e87bf65b0c5df5ce65bda9d5e443f31a490792db7a63c737fe1295ea7a85c70a4f43af01ee1d6cc545b43939b581b76ade5b6cd641d1
+  metadata.gz: dfd1714f7a86b37991a7493be186043be14a400f6e36d7712c366c1007857eee7eb4ece8390187a56f7faa921f46ffdf9ad1250ff471daf3ad2ccd884c14cda2
+  data.tar.gz: 20d5a051050da7a5b75d08529d16e2806af72ac05161393dd1a5cca509f17b2ec497f7a9b2f9c3f0933af72394b81696ae88b48736b6de67bd8ab7ae43319b25

data/bin/idrac

@@ -681,22 +681,22 @@ module IDRAC
         
         puts "\nVirtual Disks (#{volumes.size}):".green.bold
         volumes.each do |volume|
-          capacity_gb = volume[:capacity_bytes].to_f / (1024**3)
-          health_color = volume[:health] == "OK" ? :green : :yellow
+          capacity_gb = volume.capacity_bytes.to_f / (1024**3)
+          health_color = volume.health == "OK" ? :green : :yellow
           
-          puts "#{volume[:name]}:".bold
-          puts "  RAID Type: #{volume[:raid_level] || volume[:volume_type]}".cyan
-          puts "  Health: #{volume[:health].send(health_color)}"
+          puts "#{volume.name}:".bold
+          puts "  RAID Type: #{volume.raid_level || volume.volume_type}".cyan
+          puts "  Health: #{volume.health.send(health_color)}"
           puts "  Capacity: #{capacity_gb.round(2)} GB".cyan
-          puts "  Stripe Size: #{volume[:stripe_size]}".cyan
-          puts "  Read Cache: #{volume[:read_cache_policy]}".cyan
-          puts "  Write Cache: #{volume[:write_cache_policy]}".cyan
-          puts "  FastPath: #{volume[:fastpath] == 'enabled' ? 'Enabled'.green : 'Disabled'.yellow}"
-          puts "  Encrypted: #{volume[:encrypted] ? 'Yes'.green : 'No'.yellow}" if volume[:encrypted] != nil
+          puts "  Stripe Size: #{volume.stripe_size}".cyan
+          puts "  Read Cache: #{volume.read_cache_policy}".cyan
+          puts "  Write Cache: #{volume.write_cache_policy}".cyan
+          puts "  FastPath: #{volume.fastpath == 'enabled' ? 'Enabled'.green : 'Disabled'.yellow}"
+          puts "  Encrypted: #{volume.encrypted ? 'Yes'.green : 'No'.yellow}" if volume.encrypted != nil
           
-          if volume[:progress]
-            puts "  Progress: #{volume[:progress]}%".cyan
-            puts "  Operation: #{volume[:message]}".cyan
+          if volume.progress
+            puts "  Progress: #{volume.progress}%".cyan
+            puts "  Operation: #{volume.message}".cyan
           end
           puts ""
         end
@@ -742,7 +742,7 @@ module IDRAC
         
         puts "Available volumes:".green
         volumes.each_with_index do |vol, idx|
-          puts "#{idx+1}. #{vol[:name]} (#{vol[:raid_level] || vol[:volume_type]}, #{(vol[:capacity_bytes].to_f / (1024**3)).round(2)} GB)"
+          puts "#{idx+1}. #{vol.name} (#{vol.raid_level || vol.volume_type}, #{(vol.capacity_bytes.to_f / (1024**3)).round(2)} GB)"
         end
         
         print "Enter the number of the volume to delete (or 'q' to quit): "
@@ -757,12 +757,12 @@ module IDRAC
         if index >= 0 && index < volumes.size
           volume = volumes[index]
           
-          puts "You are about to delete '#{volume[:name]}'. All data will be lost!".red.bold
+          puts "You are about to delete '#{volume.name}'. All data will be lost!".red.bold
           print "Type the volume name to confirm deletion: "
           confirm = $stdin.gets.chomp
           
           if confirm == volume[:name]
-            client.delete_volume(volume[:"@odata.id"])
+            client.delete_volume(volume.data_id)
             puts "Volume deleted successfully".green
           else
             puts "Volume name did not match. Operation cancelled".yellow
@@ -846,15 +846,15 @@ module IDRAC
         
         puts "\nMemory Modules (#{memory.size}):".green.bold
         memory.each do |dimm|
-          capacity_gb = dimm["capacity_bytes"].to_f / (1024**3)
-          health_color = dimm["health"] == "OK" ? :green : :red
+          capacity_gb = dimm.capacity_bytes.to_f / (1024**3)
+          health_color = dimm.health == "OK" ? :green : :red
           
-          puts "#{dimm['name']}:".bold
-          puts "  Model: #{dimm['model']}".cyan
-          puts "  Health: #{dimm['health'].send(health_color)}"
+          puts "#{dimm.name}:".bold
+          puts "  Model: #{dimm.model}".cyan
+          puts "  Health: #{dimm.health.send(health_color)}"
           puts "  Capacity: #{capacity_gb.round(2)} GB".cyan
-          puts "  Speed: #{dimm['speed_mhz']} MHz".cyan
-          puts "  Serial: #{dimm['serial']}".cyan
+          puts "  Speed: #{dimm.speed_mhz} MHz".cyan
+          puts "  Serial: #{dimm.serial}".cyan
           puts ""
         end
         
@@ -871,14 +871,14 @@ module IDRAC
         
         puts "\nPower Supplies (#{psus.size}):".green.bold
         psus.each do |psu|
-          health_color = psu["status"] == "OK" ? :green : :red
+          health_color = psu.status == "OK" ? :green : :red
           
-          puts "#{psu['name']}:".bold
-          puts "  Model: #{psu['model']}".cyan
-          puts "  Health: #{psu['status'].send(health_color)}"
-          puts "  Input: #{psu['voltage']} V (#{psu['voltage_human']})".cyan
-          puts "  Output: #{psu['watts']} W".cyan
-          puts "  Serial: #{psu['serial']}".cyan
+          puts "#{psu.name}:".bold
+          puts "  Model: #{psu.model}".cyan
+          puts "  Health: #{psu.status.send(health_color)}"
+          puts "  Input: #{psu.voltage} V (#{psu.voltage_human})".cyan
+          puts "  Output: #{psu.watts} W".cyan
+          puts "  Serial: #{psu.serial}".cyan
           puts ""
         end
       end
@@ -897,11 +897,11 @@ module IDRAC
         
         puts "\nFans (#{fans.size}):".green.bold
         fans.each do |fan|
-          health_color = fan["status"] == "OK" ? :green : :red
+          health_color = fan.status == "OK" ? :green : :red
           
-          puts "#{fan['name']}:".bold
-          puts "  Health: #{fan['status'].send(health_color)}"
-          puts "  Speed: #{fan['rpm']} RPM".cyan
+          puts "#{fan.name}:".bold
+          puts "  Health: #{fan.status.send(health_color)}"
+          puts "  Speed: #{fan.rpm} RPM".cyan
           puts ""
         end
       end

data/idrac.gemspec

@@ -41,6 +41,7 @@ Gem::Specification.new do |spec|
   spec.add_dependency "thor", ">= 1.2.0", "< 1.4.0"
   spec.add_dependency "base64", "~> 0.1", ">= 0.1.0"
   spec.add_dependency "colorize", "~> 1.1"
+  spec.add_dependency "recursive-open-struct", "~> 1.1"
 
   # Development dependencies
   spec.add_development_dependency "bundler", "~> 2.4", ">= 2.4.0"

data/lib/idrac/client.rb

@@ -286,16 +286,6 @@ module IDRAC
       web.capture_screenshot
     end
 
-    # Parse JSON string and convert to OpenStruct
-    # @param json [String] JSON string
-    # @param use_ostruct [Boolean] Whether to convert to OpenStruct (default: true)
-    # @return [OpenStruct, Hash] Parsed response
-    def parse_json(json, use_ostruct = true)
-      parsed = JSON.parse(json)
-      use_ostruct ? IDRAC::Util.to_ostruct(parsed) : parsed
-    end
-    
-    # Kept for backward compatibility 
     def base_url
       protocol = use_ssl ? 'https' : 'http'
       "#{protocol}://#{host}:#{port}"
@@ -304,8 +294,8 @@ module IDRAC
     def redfish_version
       response = authenticated_request(:get, "/redfish/v1")
       if response.status == 200
-        data = parse_json(response.body)
-        data.RedfishVersion
+        data = JSON.parse(response.body)
+        data["RedfishVersion"]
       else
         raise Error, "Failed to get Redfish version: #{response.status} - #{response.body}"
       end

data/lib/idrac/power.rb

@@ -30,9 +30,9 @@ module IDRAC
           break
         when 409
           begin
-            error_data = parse_json(response.body)
-            if error_data.error && error_data.error['@Message.ExtendedInfo'] &&
-               error_data.error['@Message.ExtendedInfo'].any? { |m| m.Message =~ /Server is already powered ON/ }
+            error_data = JSON.parse(response.body)
+            if error_data["error"] && error_data["error"]["@Message.ExtendedInfo"] &&
+               error_data["error"]["@Message.ExtendedInfo"].any? { |m| m["Message"] =~ /Server is already powered ON/ }
               puts "Server is already powered ON.".yellow
               return false
             else
@@ -84,9 +84,9 @@ module IDRAC
       when 409
         # Conflict -- Server is already off
         begin
-          error_data = parse_json(response.body)
-          if error_data.error && error_data.error['@Message.ExtendedInfo'] &&
-             error_data.error['@Message.ExtendedInfo'].any? { |m| m.Message =~ /Server is already powered OFF/ }
+          error_data = JSON.parse(response.body)
+          if error_data["error"] && error_data["error"]["@Message.ExtendedInfo"] &&
+             error_data["error"]["@Message.ExtendedInfo"].any? { |m| m["Message"] =~ /Server is already powered OFF/ }
             puts "Server is already powered OFF.".yellow
             return false
           else
@@ -98,8 +98,8 @@ module IDRAC
       else
         error_message = "Failed to power off server. Status code: #{response.status}"
         begin
-          error_data = parse_json(response.body)
-          error_message += ", Message: #{error_data.error.message}" if error_data.error && error_data.error.message
+          error_data = JSON.parse(response.body)
+          error_message += ", Message: #{error_data['error']['message']}" if error_data['error'] && error_data['error']['message']
         rescue
           # Ignore JSON parsing errors
         end
@@ -137,8 +137,8 @@ module IDRAC
       else
         error_message = "Failed to reboot server. Status code: #{response.status}"
         begin
-          error_data = parse_json(response.body)
-          error_message += ", Message: #{error_data.error.message}" if error_data.error && error_data.error.message
+          error_data = JSON.parse(response.body)
+          error_message += ", Message: #{error_data['error']['message']}" if error_data['error'] && error_data['error']['message']
         rescue
           # Ignore JSON parsing errors
         end
@@ -156,8 +156,8 @@ module IDRAC
       
       if response.status == 200
         begin
-          system_data = parse_json(response.body)
-          return system_data.PowerState
+          system_data = JSON.parse(response.body)
+          return system_data["PowerState"]
         rescue JSON::ParserError
           raise Error, "Failed to parse power state response: #{response.body}"
         end
@@ -174,8 +174,8 @@ module IDRAC
       
       if response.status == 200
         begin
-          data = parse_json(response.body)
-          watts = data.PowerControl[0].PowerConsumedWatts
+          data = JSON.parse(response.body)
+          watts = data["PowerControl"][0]["PowerConsumedWatts"]
           # puts "Power usage: #{watts} watts".light_cyan
           return watts.to_f
         rescue JSON::ParserError
@@ -184,8 +184,8 @@ module IDRAC
       else
         error_message = "Failed to get power usage. Status code: #{response.status}"
         begin
-          error_data = parse_json(response.body)
-          error_message += ", Message: #{error_data.error.message}" if error_data.error && error_data.error.message
+          error_data = JSON.parse(response.body)
+          error_message += ", Message: #{error_data['error']['message']}" if error_data['error'] && error_data['error']['message']
         rescue
           # Ignore JSON parsing errors
         end

data/lib/idrac/storage.rb

@@ -1,5 +1,6 @@
 require 'json'
 require 'colorize'
+require 'recursive-open-struct'
 
 module IDRAC
   module Storage
@@ -9,24 +10,24 @@ module IDRAC
       controller_list = controllers
       
       puts "Controllers".green
-      controller_list.each { |c| puts "#{c[:name]} > #{c[:drives_count]}" }
+      controller_list.each { |c| puts "#{c.name} > #{c.drives_count}" }
       
       puts "Drives".green
       controller_list.each do |c|
-        puts "Storage: #{c[:name]} > #{c[:status]} > #{c[:drives_count]}"
+        puts "Storage: #{c.name} > #{c.status} > #{c.drives_count}"
       end
       
       # Find the controller with the most drives (usually the PERC)
-      controller_info = controller_list.max_by { |c| c[:drives_count] }
+      controller_info = controller_list.max_by { |c| c.drives_count }
       
-      if controller_info[:name] =~ /PERC/
-        puts "Found #{controller_info[:name]}".green
+      if controller_info.name =~ /PERC/
+        puts "Found #{controller_info.name}".green
       else
-        puts "Found #{controller_info[:name]} but continuing...".yellow
+        puts "Found #{controller_info.name} but continuing...".yellow
       end
       
       # Return the raw controller data
-      controller_info[:raw]
+      controller_info.raw
     end
 
     # Get all storage controllers and return them as an array
@@ -37,9 +38,9 @@ module IDRAC
         begin
           data = JSON.parse(response.body)
           
-          # Transform and return all controllers as an array of hashes with consistent keys
+          # Transform and return all controllers as an array of RecursiveOpenStruct objects with consistent keys
           controllers = data["Members"].map do |controller|
-            {
+            controller_data = {
               name: controller["Name"],
               model: controller["Model"],
               drives_count: controller["Drives"].size,
@@ -51,9 +52,11 @@ module IDRAC
               pci_slot: controller.dig("Oem", "Dell", "DellController", "PCISlot"),
               raw: controller
             }
+            
+            RecursiveOpenStruct.new(controller_data, recurse_over_arrays: true)
           end
           
-          return controllers.sort_by { |c| c[:name] }
+          return controllers.sort_by { |c| c.name }
         rescue JSON::ParserError
           raise Error, "Failed to parse controllers response: #{response.body}"
         end
@@ -87,7 +90,7 @@ module IDRAC
           drives = data["Drives"].map do |body|
             serial = body["SerialNumber"] 
             serial = body["Identifiers"].first["DurableName"] if serial.blank?
-            { 
+            drive_data = { 
               serial: serial,
               model: body["Model"],
               name: body["Name"],
@@ -105,9 +108,11 @@ module IDRAC
               encryption_ability: body["EncryptionAbility"],
               "@odata.id": body["@odata.id"]
             }
+            
+            RecursiveOpenStruct.new(drive_data, recurse_over_arrays: true)
           end
           
-          return drives.sort_by { |d| d[:name] }
+          return drives.sort_by { |d| d.name }
         rescue JSON::ParserError
           raise Error, "Failed to parse drives response: #{response.body}"
         end
@@ -131,7 +136,7 @@ module IDRAC
           data = JSON.parse(response.body)
           volumes = data["Members"].map do |vol|
             drives = vol["Links"]["Drives"]
-            volume = { 
+            volume_data = { 
               name: vol["Name"], 
               capacity_bytes: vol["CapacityBytes"], 
               volume_type: vol["VolumeType"],
@@ -142,31 +147,32 @@ module IDRAC
               raid_level: vol["RAIDType"],
               encrypted: vol["Encrypted"],
               lock_status: vol.dig("Oem", "Dell", "DellVirtualDisk", "LockStatus"),
-              "@odata.id": vol["@odata.id"]
+              odata_id: vol["@odata.id"]
             }
             
             # Check FastPath settings
-            volume[:fastpath] = fastpath_good?(volume)
+            volume_data[:fastpath] = fastpath_good?(vol)
             
             # Handle volume operations and status
             if vol["Operations"].any?
-              volume[:health] = vol["Status"]["Health"] ? vol["Status"]["Health"] : "N/A"
-              volume[:progress] = vol["Operations"].first["PercentageComplete"]
-              volume[:message] = vol["Operations"].first["OperationName"]     
+              volume_data[:health] = vol["Status"]["Health"] ? vol["Status"]["Health"] : "N/A"
+              volume_data[:progress] = vol["Operations"].first["PercentageComplete"]
+              volume_data[:message] = vol["Operations"].first["OperationName"]     
             elsif vol["Status"]["Health"] == "OK"
-              volume[:health] = "OK"
-              volume[:progress] = nil
-              volume[:message] = nil
+              volume_data[:health] = "OK"
+              volume_data[:progress] = nil
+              volume_data[:message] = nil
             else
-              volume[:health] = "?"
-              volume[:progress] = nil
-              volume[:message] = nil
+              volume_data[:health] = "?"
+              volume_data[:progress] = nil
+              volume_data[:message] = nil
             end
             
-            volume
+            # Create the RecursiveOpenStruct after all properties are set
+            RecursiveOpenStruct.new(volume_data, recurse_over_arrays: true)
           end
           
-          return volumes.sort_by { |d| d[:name] }
+          return volumes.sort_by { |d| d.name }
         rescue JSON::ParserError
           raise Error, "Failed to parse volumes response: #{response.body}"
         end
@@ -360,7 +366,7 @@ module IDRAC
 
     # Check if all physical disks are Self-Encrypting Drives
     def all_seds?(drives)
-      drives.all? { |d| d[:encryption_ability] == "SelfEncryptingDrive" }
+      drives.all? { |d| d.encryption_ability == "SelfEncryptingDrive" }
     end
 
     # Check if the system is ready for SED operations

data/lib/idrac/system.rb

@@ -1,5 +1,6 @@
 require 'json'
 require 'colorize'
+require 'recursive-open-struct'
 
 module IDRAC
   module System
@@ -9,27 +10,29 @@ module IDRAC
       
       if response.status == 200
         begin
-          data = parse_json(response.body)
-          memory = data.Members.map do |m|
-            dimm_name = m.Name # e.g. DIMM A1
+          data = JSON.parse(response.body)
+          memory = data["Members"].map do |m|
+            dimm_name = m["Name"] # e.g. DIMM A1
             bank, index = /DIMM ([A-Z])(\d+)/.match(dimm_name).captures rescue [nil, nil]
             
-            puts "DIMM: #{m.Model} #{m.Name} > #{m.CapacityMiB}MiB > #{m.Status.Health} > #{m.OperatingSpeedMhz}MHz > #{m.PartNumber} / #{m.SerialNumber}"
+            puts "DIMM: #{m["Model"]} #{m["Name"]} > #{m["CapacityMiB"]}MiB > #{m["Status"]["Health"]} > #{m["OperatingSpeedMhz"]}MHz > #{m["PartNumber"]} / #{m["SerialNumber"]}"
             
-            { 
-              "model" => m.Model, 
-              "name" => m.Name, 
-              "capacity_bytes" => m.CapacityMiB.to_i.megabyte, 
-              "health" => m.Status.Health ? m.Status.Health : "N/A", 
-              "speed_mhz" => m.OperatingSpeedMhz, 
-              "part_number" => m.PartNumber, 
-              "serial" => m.SerialNumber,
-              "bank" => bank,
-              "index" => index.to_i
+            memory_data = { 
+              model: m["Model"], 
+              name: m["Name"], 
+              capacity_bytes: m["CapacityMiB"].to_i.megabyte, 
+              health: m["Status"]["Health"] ? m["Status"]["Health"] : "N/A", 
+              speed_mhz: m["OperatingSpeedMhz"], 
+              part_number: m["PartNumber"], 
+              serial: m["SerialNumber"],
+              bank: bank,
+              index: index.to_i
             }
+            
+            RecursiveOpenStruct.new(memory_data, recurse_over_arrays: true)
           end
           
-          return memory.sort_by { |a| [a["bank"] || "Z", a["index"] || 999] }
+          return memory.sort_by { |m| [m.bank || "Z", m.index || 999] }
         rescue JSON::ParserError
           raise Error, "Failed to parse memory response: #{response.body}"
         end
@@ -44,21 +47,23 @@ module IDRAC
       
       if response.status == 200
         begin
-          data = parse_json(response.body)
+          data = JSON.parse(response.body)
           puts "Power Supplies".green
           
-          psus = data.PowerSupplies.map do |psu|
-            puts "PSU: #{psu.Name} > #{psu.PowerInputWatts}W > #{psu.Status.Health}"
-            { 
-              "name" => psu.Name, 
-              "voltage" => psu.LineInputVoltage, 
-              "voltage_human" => psu.LineInputVoltageType, # AC240V
-              "watts"  => psu.PowerInputWatts,
-              "part"   => psu.PartNumber,
-              "model"  => psu.Model,
-              "serial" => psu.SerialNumber,
-              "status" => psu.Status.Health,
+          psus = data["PowerSupplies"].map do |psu|
+            puts "PSU: #{psu["Name"]} > #{psu["PowerInputWatts"]}W > #{psu["Status"]["Health"]}"
+            psu_data = { 
+              name: psu["Name"], 
+              voltage: psu["LineInputVoltage"], 
+              voltage_human: psu["LineInputVoltageType"], # AC240V
+              watts: psu["PowerInputWatts"],
+              part: psu["PartNumber"],
+              model: psu["Model"],
+              serial: psu["SerialNumber"],
+              status: psu["Status"]["Health"],
             }
+            
+            RecursiveOpenStruct.new(psu_data, recurse_over_arrays: true)
           end
           
           return psus
@@ -80,23 +85,25 @@ module IDRAC
           response = authenticated_request(:get, "/redfish/v1/Chassis/System.Embedded.1/Thermal?$expand=*($levels=1)")
           
           if response.status == 200
-            data = parse_json(response.body)
+            data = JSON.parse(response.body)
             
-            fans = data.Fans.map do |fan|
-              puts "Fan: #{fan.Name} > #{fan.Reading} > #{fan.Status.Health}"
-              { 
-                "name" => fan.Name, 
-                "rpm" => fan.Reading,
-                "serial" => fan.SerialNumber,
-                "status" => fan.Status.Health
+            fans = data["Fans"].map do |fan|
+              puts "Fan: #{fan["Name"]} > #{fan["Reading"]} > #{fan["Status"]["Health"]}"
+              fan_data = { 
+                name: fan["Name"], 
+                rpm: fan["Reading"],
+                serial: fan["SerialNumber"],
+                status: fan["Status"]["Health"]
               }
+              
+              RecursiveOpenStruct.new(fan_data, recurse_over_arrays: true)
             end
             
             return fans
           elsif response.status.between?(400, 499)
             # Check if system is powered off
             power_response = authenticated_request(:get, "/redfish/v1/Systems/System.Embedded.1?$select=PowerState")
-            if power_response.status == 200 && parse_json(power_response.body).PowerState == "Off"
+            if power_response.status == 200 && JSON.parse(power_response.body)["PowerState"] == "Off"
               puts "WARN: System is off. Fans are not available.".yellow
               return []
             end
@@ -120,12 +127,12 @@ module IDRAC
       
       if response.status == 200
         begin
-          adapters_data = parse_json(response.body)
+          adapters_data = JSON.parse(response.body)
           
           # Determine iDRAC version for different port paths
           idrac_version_response = authenticated_request(:get, "/redfish/v1")
-          idrac_version_data = parse_json(idrac_version_response.body)
-          server = idrac_version_data.RedfishVersion || idrac_version_response.headers["server"]
+          idrac_version_data = JSON.parse(idrac_version_response.body)
+          server = idrac_version_data["RedfishVersion"] || idrac_version_response.headers["server"]
           
           is_idrac9 = case server.to_s.downcase
                       when /idrac\/9/
@@ -141,32 +148,32 @@ module IDRAC
           
           port_part = is_idrac9 ? 'Ports' : 'NetworkPorts'
           
-          nics = adapters_data.Members.map do |adapter|
-            path = "#{adapter['@odata.id'].split("v1/").last}/#{port_part}?$expand=*($levels=1)"
+          nics = adapters_data["Members"].map do |adapter|
+            path = "#{adapter["@odata.id"].split("v1/").last}/#{port_part}?$expand=*($levels=1)"
             ports_response = authenticated_request(:get, "/redfish/v1/#{path}")
             
             if ports_response.status == 200
-              ports_data = parse_json(ports_response.body)
+              ports_data = JSON.parse(ports_response.body)
               
-              ports = ports_data.Members.map do |nic|
+              ports = ports_data["Members"].map do |nic|
                 if is_idrac9
-                  link_speed_mbps = nic.CurrentSpeedGbps.to_i * 1000
-                  mac_addr = nic.Ethernet.AssociatedMACAddresses.first
-                  port_num = nic.PortId
-                  network_technology = nic.LinkNetworkTechnology
-                  link_status = nic.LinkStatus =~ /up/i ? "Up" : "Down"
+                  link_speed_mbps = nic['CurrentSpeedGbps'].to_i * 1000
+                  mac_addr = nic['Ethernet']['AssociatedMACAddresses'].first
+                  port_num = nic['PortId']
+                  network_technology = nic['LinkNetworkTechnology']
+                  link_status = nic['LinkStatus'] =~ /up/i ? "Up" : "Down"
                 else # iDRAC 8
-                  link_speed_mbps = nic.SupportedLinkCapabilities.first.LinkSpeedMbps
-                  mac_addr = nic.AssociatedNetworkAddresses.first
-                  port_num = nic.PhysicalPortNumber
-                  network_technology = nic.SupportedLinkCapabilities.first.LinkNetworkTechnology
-                  link_status = nic.LinkStatus
+                  link_speed_mbps = nic["SupportedLinkCapabilities"].first["LinkSpeedMbps"]
+                  mac_addr = nic["AssociatedNetworkAddresses"].first
+                  port_num = nic["PhysicalPortNumber"]
+                  network_technology = nic["SupportedLinkCapabilities"].first["LinkNetworkTechnology"]
+                  link_status = nic['LinkStatus']
                 end
                 
-                puts "NIC: #{nic.Id} > #{mac_addr} > #{link_status} > #{port_num} > #{link_speed_mbps}Mbps"
+                puts "NIC: #{nic["Id"]} > #{mac_addr} > #{link_status} > #{port_num} > #{link_speed_mbps}Mbps"
                 
                 { 
-                  "name" => nic.Id, 
+                  "name" => nic["Id"], 
                   "status" => link_status,
                   "mac" => mac_addr,
                   "port" => port_num,
@@ -176,21 +183,21 @@ module IDRAC
               end
               
               {
-                "name" => adapter.Id,
-                "manufacturer" => adapter.Manufacturer,
-                "model" => adapter.Model,
-                "part_number" => adapter.PartNumber,
-                "serial" => adapter.SerialNumber,
+                "name" => adapter["Id"],
+                "manufacturer" => adapter["Manufacturer"],
+                "model" => adapter["Model"],
+                "part_number" => adapter["PartNumber"],
+                "serial" => adapter["SerialNumber"],
                 "ports" => ports
               }
             else
               # Return adapter info without ports if we can't get port details
               {
-                "name" => adapter.Id,
-                "manufacturer" => adapter.Manufacturer,
-                "model" => adapter.Model,
-                "part_number" => adapter.PartNumber,
-                "serial" => adapter.SerialNumber,
+                "name" => adapter["Id"],
+                "manufacturer" => adapter["Manufacturer"],
+                "model" => adapter["Model"],
+                "part_number" => adapter["PartNumber"],
+                "serial" => adapter["SerialNumber"],
                 "ports" => []
               }
             end
@@ -211,17 +218,17 @@ module IDRAC
       
       if response.status == 200
         begin
-          data = parse_json(response.body)
+          data = JSON.parse(response.body)
           
           idrac = {
-            "name" => data.Id,
-            "status" => data.Status.Health == 'OK' ? 'Up' : 'Down',
-            "mac" => data.MACAddress,
-            "mask" => data.IPv4Addresses.first.SubnetMask,
-            "ipv4" => data.IPv4Addresses.first.Address,
-            "origin" => data.IPv4Addresses.first.AddressOrigin, # DHCP or Static
+            "name" => data["Id"],
+            "status" => data["Status"]["Health"] == 'OK' ? 'Up' : 'Down',
+            "mac" => data["MACAddress"],
+            "mask" => data["IPv4Addresses"].first["SubnetMask"],
+            "ipv4" => data["IPv4Addresses"].first["Address"],
+            "origin" => data["IPv4Addresses"].first["AddressOrigin"], # DHCP or Static
             "port" => nil,
-            "speed_mbps" => data.SpeedMbps,
+            "speed_mbps" => data["SpeedMbps"],
             "kind" => "ethernet"
           }
           
@@ -240,32 +247,32 @@ module IDRAC
       
       if response.status == 200
         begin
-          data = parse_json(response.body)
+          data = JSON.parse(response.body)
           
-          pci = data.Members.map do |stub|
-            manufacturer = stub.Manufacturer
+          pci = data["Members"].map do |stub|
+            manufacturer = stub["Manufacturer"]
             
             # Get PCIe function details if available
             pcie_function = nil
-            if stub.dig(:Links, :PCIeFunctions, 0, :'@odata.id')
-              pcie_function_path = stub.dig(:Links, :PCIeFunctions, 0, :'@odata.id').split("v1/").last
+            if stub.dig("Links", "PCIeFunctions", 0, "@odata.id")
+              pcie_function_path = stub.dig("Links", "PCIeFunctions", 0, "@odata.id").split("v1/").last
               function_response = authenticated_request(:get, "/redfish/v1/#{pcie_function_path}")
               
               if function_response.status == 200
-                pcie_function = parse_json(function_response.body)
+                pcie_function = JSON.parse(function_response.body)
               end
             end
             
             # Create device info with available data
             device_info = {
-              device_class: pcie_function ? pcie_function.DeviceClass : nil,
+              device_class: pcie_function ? pcie_function["DeviceClass"] : nil,
               manufacturer: manufacturer,
-              name: stub.Name,
-              description: stub.Description,
-              id: pcie_function ? pcie_function.Id : stub.Id,
-              slot_type: pcie_function ? pcie_function.dig(:Oem, :Dell, :DellPCIeFunction, :SlotType) : nil,
-              bus_width: pcie_function ? pcie_function.dig(:Oem, :Dell, :DellPCIeFunction, :DataBusWidth) : nil,
-              nic: pcie_function ? pcie_function.dig(:Links, :EthernetInterfaces, 0, :'@odata.id') : nil
+              name: stub["Name"],
+              description: stub["Description"],
+              id: pcie_function ? pcie_function["Id"] : stub["Id"],
+              slot_type: pcie_function ? pcie_function.dig("Oem", "Dell", "DellPCIeFunction", "SlotType") : nil,
+              bus_width: pcie_function ? pcie_function.dig("Oem", "Dell", "DellPCIeFunction", "DataBusWidth") : nil,
+              nic: pcie_function ? pcie_function.dig("Links", "EthernetInterfaces", 0, "@odata.id") : nil
             }
             
             puts "PCI Device: #{device_info[:name]} > #{device_info[:manufacturer]} > #{device_info[:device_class]} > #{device_info[:description]} > #{device_info[:id]}"
@@ -331,15 +338,15 @@ module IDRAC
       
       if response.status == 200
         begin
-          data = parse_json(response.body)
+          data = JSON.parse(response.body)
           
-          logs = data.Members.map do |log|
-            puts "#{log.Id} : #{log.Created} : #{log.Message} : #{log.Severity}".yellow
+          logs = data["Members"].map do |log|
+            puts "#{log['Id']} : #{log['Created']} : #{log['Message']} : #{log['Severity']}".yellow
             log
           end
           
           # Sort by creation date, newest first
-          return logs.sort_by { |log| log.Created }.reverse
+          return logs.sort_by { |log| log['Created'] }.reverse
         rescue JSON::ParserError
           raise Error, "Failed to parse system event logs response: #{response.body}"
         end
@@ -364,7 +371,7 @@ module IDRAC
         error_message = "Failed to clear System Event Logs. Status code: #{response.status}"
         
         begin
-          error_data = parse_json(response.body, false)
+          error_data = JSON.parse(response.body)
           error_message += ", Message: #{error_data['error']['message']}" if error_data['error'] && error_data['error']['message']
         rescue
           # Ignore JSON parsing errors
@@ -376,7 +383,7 @@ module IDRAC
 
     # Get total memory in human-readable format
     def total_memory_human(memory_data)
-      total_memory = memory_data.sum { |m| m['capacity_bytes'] }
+      total_memory = memory_data.sum { |m| m.capacity_bytes }
       "%0.2f GB" % (total_memory.to_f / 1.gigabyte)
     end
   end

data/lib/idrac/version.rb

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

data/lib/idrac/virtual_media.rb

@@ -9,25 +9,31 @@ module IDRAC
       
       if response.status == 200
         begin
-          data = parse_json(response.body)
+          data = JSON.parse(response.body)
           
-          media = data.Members.map do |m|
-            {
-              "id" => m.Id,
-              "name" => m.Name,
-              "media_types" => m.MediaTypes,
-              "connected" => m.Inserted,
-              "image" => m.Image,
-              "write_protected" => m.WriteProtected
+          media = data["Members"].map do |m|
+            if m["Inserted"]
+              puts "#{m["Name"]} #{m["ConnectedVia"]} #{m["Image"]}".green
+            else
+              puts "#{m["Name"]} #{m["ConnectedVia"]}".yellow
+            end
+            
+            action_path = m.dig("Actions", "#VirtualMedia.InsertMedia", "target")
+            
+            { 
+              device: m["Id"], 
+              inserted: m["Inserted"], 
+              image: m["Image"] || m["ConnectedVia"],
+              action_path: action_path
             }
           end
           
           return media
-        rescue => e
-          raise Error, "Failed to get virtual media status: #{e.message}"
+        rescue JSON::ParserError
+          raise Error, "Failed to parse virtual media response: #{response.body}"
         end
       else
-        raise Error, "Failed to get virtual media status. Status code: #{response.status}"
+        raise Error, "Failed to get virtual media. Status code: #{response.status}"
       end
     end
 

data/lib/idrac.rb

@@ -7,7 +7,7 @@ require 'faraday/multipart'
 require 'base64'
 require 'uri'
 require 'colorize'
-require 'ostruct'
+require 'recursive-open-struct'
 # If dev, required debug
 require 'debug' if ENV['RUBY_ENV'] == 'development'
 
@@ -30,23 +30,6 @@ module IDRAC
     end
   end
 
-  # Utility methods for the IDRAC module
-  module Util
-    # Recursively converts a hash to an OpenStruct
-    # @param obj [Hash, Array, Object] The object to convert
-    # @return [OpenStruct, Array, Object] The converted object
-    def self.to_ostruct(obj)
-      case obj
-      when Hash
-        OpenStruct.new(obj.transform_values { |val| to_ostruct(val) })
-      when Array
-        obj.map { |item| to_ostruct(item) }
-      else
-        obj
-      end
-    end
-  end
-
   class Error < StandardError; end
   
   def self.new(options = {})

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: idrac
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.4.5
 platform: ruby
 authors:
 - Jonathan Siegel
@@ -144,6 +144,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.1'
+- !ruby/object:Gem::Dependency
+  name: recursive-open-struct
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.1'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.1'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement