radfish-ami 0.1.4 → 0.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0b9d6c1504f5b0420021db42cf51f92f4864b57e829ab2167526758889f42a0a
-  data.tar.gz: 23b729993a4fd05476a6737f2b7eed8d21c6d4b297f850ea5b5dda35659897c8
+  metadata.gz: 8437153ec6bf1a793ee23f287fdea85dc0b3511b2e8a87e7fb615dd8ea9737d4
+  data.tar.gz: 18c7ea5001939cc76bbdd31673e1ad012ec94b79ab373f43ef216ff7fe9b6f35
 SHA512:
-  metadata.gz: aa22eff985917551a7b889041c667f888b82dcc07c8f6dd95303c5b46c7d79152aa785bf9b3f01a22ffe4deb8683fb8a4d3486add7541e2fb51e3be658087cef
-  data.tar.gz: 94b80ed7c35a5bbf43d8f4ee73f5b988e34f8d95a7932ed6df295241d595aa941d2df896d4f670a71c516cebbc4000fde21a7d67c6357bbdc01128d8098688ff
+  metadata.gz: 7663274856b229b95ec203efcf2cf9921b3bd5553b675ba2ca0a9e016671d9872f657c725fb021e4dbca04f998521054d334586bfdbaeb6e041cbd29842e1907
+  data.tar.gz: f26cb16d15eb2ffa2c16a94a518049e358c7aa9977c9e6e7811151aee0b33f693c7ddc440605809377b2d9c0e70010e525c89a0a19abc4605831b6d084855c3f

data/lib/radfish/ami/version.rb

@@ -2,6 +2,6 @@
 
 module Radfish
   module Ami
-    VERSION = "0.1.4"
+    VERSION = "0.1.6"
   end
 end

data/lib/radfish/ami_adapter.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'ostruct'
+
 module Radfish
   class AmiAdapter < Core::BaseClient
     include Core::Power
@@ -138,30 +140,12 @@ module Radfish
     def system_health
       info = system_info
       status = info["Status"] || {}
-      HealthStatus.new(
+      OpenStruct.new(
         health: status["Health"] || "Unknown",
         rollup: status["HealthRollup"] || status["Health"] || "Unknown"
       )
     end
 
-    # Simple struct for health status with rollup
-    class HealthStatus
-      attr_reader :health, :rollup
-
-      def initialize(health:, rollup:)
-        @health = health
-        @rollup = rollup
-      end
-
-      def to_s
-        @health
-      end
-
-      def ==(other)
-        other.to_s == to_s
-      end
-    end
-
     def bmc_info
       manager = get_manager_info
       network = get_bmc_network
@@ -186,7 +170,16 @@ module Radfish
       members.map do |member|
         cpu_response = authenticated_request(:get, member["@odata.id"])
         next nil unless cpu_response.status == 200
-        JSON.parse(cpu_response.body)
+        data = JSON.parse(cpu_response.body)
+        OpenStruct.new(
+          socket: data["Socket"] || data["Id"],
+          manufacturer: data["Manufacturer"],
+          model: data.dig("ProcessorId", "EffectiveFamily")&.strip || data["Model"],
+          cores: data["TotalCores"],
+          threads: data["TotalThreads"],
+          speed_mhz: data["MaxSpeedMHz"],
+          health: data.dig("Status", "Health") || "Unknown"
+        )
       end.compact
     end
 
@@ -200,7 +193,17 @@ module Radfish
       members.map do |member|
         mem_response = authenticated_request(:get, member["@odata.id"])
         next nil unless mem_response.status == 200
-        JSON.parse(mem_response.body)
+        data = JSON.parse(mem_response.body)
+        OpenStruct.new(
+          name: data["Name"] || data["Id"],
+          capacity_bytes: data["CapacityMiB"] ? data["CapacityMiB"] * 1024 * 1024 : nil,
+          speed_mhz: data["OperatingSpeedMhz"],
+          manufacturer: data["Manufacturer"],
+          part_number: data["PartNumber"],
+          serial_number: data["SerialNumber"],
+          memory_type: data["MemoryDeviceType"],
+          status: data.dig("Status", "Health") || "OK"
+        )
       end.compact
     end
 
@@ -214,23 +217,58 @@ module Radfish
       members.map do |member|
         nic_response = authenticated_request(:get, member["@odata.id"])
         next nil unless nic_response.status == 200
-        JSON.parse(nic_response.body)
+        data = JSON.parse(nic_response.body)
+        OpenStruct.new(
+          name: data["Name"] || data["Id"],
+          mac: data["MACAddress"],
+          speed_mbps: data["SpeedMbps"],
+          link_status: data["LinkStatus"],
+          ipv4_addresses: data["IPv4Addresses"],
+          ipv6_addresses: data["IPv6Addresses"],
+          status: data.dig("Status", "Health") || "OK"
+        )
       end.compact
     end
 
     def fans
       thermal = get_thermal_data
-      thermal["Fans"] || []
+      (thermal["Fans"] || []).map do |fan|
+        OpenStruct.new(
+          name: fan["Name"] || fan["MemberId"],
+          rpm: fan["Reading"],
+          status: fan.dig("Status", "Health") || "OK",
+          min_rpm: fan["MinReadingRange"],
+          max_rpm: fan["MaxReadingRange"]
+        )
+      end
     end
 
     def temperatures
       thermal = get_thermal_data
-      thermal["Temperatures"] || []
+      (thermal["Temperatures"] || []).map do |temp|
+        OpenStruct.new(
+          name: temp["Name"] || temp["MemberId"],
+          reading_celsius: temp["ReadingCelsius"],
+          status: temp.dig("Status", "Health") || "OK",
+          upper_threshold_critical: temp["UpperThresholdCritical"],
+          upper_threshold_fatal: temp["UpperThresholdFatal"]
+        )
+      end
     end
 
     def psus
       power_data = get_power_data
-      power_data["PowerSupplies"] || []
+      (power_data["PowerSupplies"] || []).map do |psu|
+        OpenStruct.new(
+          name: psu["Name"] || psu["MemberId"],
+          model: psu["Model"],
+          serial: psu["SerialNumber"],
+          watts: psu["PowerCapacityWatts"],
+          voltage: psu.dig("InputRanges", 0, "NominalVoltageVolts"),
+          voltage_human: psu.dig("InputRanges", 0, "InputType"),
+          status: psu.dig("Status", "Health") || "OK"
+        )
+      end
     end
 
     def power_consumption
@@ -279,7 +317,19 @@ module Radfish
       drive_refs.map do |ref|
         drive_response = authenticated_request(:get, ref["@odata.id"])
         next nil unless drive_response.status == 200
-        JSON.parse(drive_response.body)
+        drive_data = JSON.parse(drive_response.body)
+        OpenStruct.new(
+          name: drive_data["Name"] || drive_data["Id"],
+          model: drive_data["Model"],
+          manufacturer: drive_data["Manufacturer"],
+          serial: drive_data["SerialNumber"],
+          capacity_bytes: drive_data["CapacityBytes"],
+          capacity_gb: drive_data["CapacityBytes"] ? (drive_data["CapacityBytes"] / 1_000_000_000.0).round(2) : nil,
+          media_type: drive_data["MediaType"],
+          protocol: drive_data["Protocol"],
+          status: drive_data.dig("Status", "Health") || "OK",
+          certified: drive_data.dig("Oem", "Dell", "Certified") || drive_data.dig("Oem", "AMI", "Certified")
+        )
       end.compact
     end
 
@@ -332,7 +382,18 @@ module Radfish
       drive_refs.map do |ref|
         drive_response = authenticated_request(:get, ref["@odata.id"])
         next nil unless drive_response.status == 200
-        JSON.parse(drive_response.body)
+        drive_data = JSON.parse(drive_response.body)
+        OpenStruct.new(
+          name: drive_data["Name"] || drive_data["Id"],
+          model: drive_data["Model"],
+          manufacturer: drive_data["Manufacturer"],
+          serial: drive_data["SerialNumber"],
+          capacity_bytes: drive_data["CapacityBytes"],
+          capacity_gb: drive_data["CapacityBytes"] ? (drive_data["CapacityBytes"] / 1_000_000_000.0).round(2) : nil,
+          media_type: drive_data["MediaType"],
+          protocol: drive_data["Protocol"],
+          status: drive_data.dig("Status", "Health") || "OK"
+        )
       end.compact
     end
 

data/radfish-ami.gemspec

@@ -24,6 +24,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   spec.add_dependency "radfish", "~> 0.2"
+  spec.add_dependency "ostruct", ">= 0"
 
   spec.add_development_dependency "bundler", "~> 2.0"
   spec.add_development_dependency "rake", "~> 13.0"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: radfish-ami
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.1.6
 platform: ruby
 authors:
 - Jonathan Siegel
@@ -24,6 +24,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.2'
+- !ruby/object:Gem::Dependency
+  name: ostruct
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement