radfish-ami 0.1.7 → 0.1.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7a8366ebff88acb6537e811e7e94e4e07ed0d00e0f8ee9ee48b7cd003133f2dd
-  data.tar.gz: cc728ce842585bd10f7e6fe97506da24ee8d1405c865288f9dcffff22d00695c
+  metadata.gz: fd85e3b625acf54dd8f3a6ab8e6e626e7678b2ee6bb893c20812eacebe901262
+  data.tar.gz: 25fda3a8c8187a2f5f8d6c0b1745aaa4223917d810940f7e3f86abec3ea8050a
 SHA512:
-  metadata.gz: 4c74710a833e5fa6af773b910e679a5d0ead7a444e4c0ef3f66097cf1b4e5a3586e46b143baaed3c9f44a5b745680206e558c19b7a00ecf2a1ac4374b8942169
-  data.tar.gz: 1f3841480ba7b7affead0434fddf6350f47f20f8ee8c4a5f904fe27a62d7750af0b19840d20da1901682107021ae76647a9d016c7925b5edd1c67a925961ccf5
+  metadata.gz: 502361dc9fbc00539f97539df2d8510d95682d5e90b1cc275f404087a248e53aa5f3547a9dde944b3ed30ce0b4da3875cbd9ba4ab86038c58519fcf79fa888cf
+  data.tar.gz: 8d5cc66f2b8c2271f1860782cecd6eecb3c5fa15c51e43a98d0256f57908b7b3af453bdfae888be03294bf5caea3850aeed9b10cdc56cf2ece0c28da80469681

data/lib/radfish/ami/version.rb

@@ -2,6 +2,6 @@
 
 module Radfish
   module Ami
-    VERSION = "0.1.7"
+    VERSION = "0.1.10"
   end
 end

data/lib/radfish/ami_adapter.rb

@@ -288,6 +288,34 @@ module Radfish
         0
     end
 
+    # PCI Devices
+    def pci_devices
+      # Try PCIeDevices endpoint first (Redfish standard)
+      response = authenticated_request(:get, "/redfish/v1/Systems/#{SYSTEM_ID}/PCIeDevices")
+      if response.status == 200
+        collection = JSON.parse(response.body)
+        members = collection["Members"] || []
+
+        return members.map do |member|
+          device_response = authenticated_request(:get, member["@odata.id"])
+          next nil unless device_response.status == 200
+          data = JSON.parse(device_response.body)
+          OpenStruct.new(
+            id: data["Id"],
+            name: data["Name"],
+            manufacturer: data["Manufacturer"],
+            model: data["Model"],
+            device_type: data["DeviceType"],
+            pcie_interface: data["PCIeInterface"],
+            status: data.dig("Status", "Health") || "OK"
+          )
+        end.compact
+      end
+
+      # Fallback: return empty array if not supported
+      []
+    end
+
     # Storage
     def storage_controllers
       response = authenticated_request(:get, "/redfish/v1/Systems/#{SYSTEM_ID}/Storage")
@@ -300,19 +328,25 @@ module Radfish
         controller_response = authenticated_request(:get, member["@odata.id"])
         next nil unless controller_response.status == 200
         data = JSON.parse(controller_response.body)
-        Radfish::Controller.new(
-          client: self,
+        # Return OpenStruct - Radfish::Client will wrap in Controller
+        OpenStruct.new(
           id: data["Id"],
           name: data["Name"],
           model: data["Model"],
           status: data.dig("Status", "Health"),
-          adapter_data: data
+          "@odata.id": member["@odata.id"]
         )
       end.compact
     end
 
     def drives(controller)
-      controller_id = controller.is_a?(Radfish::Controller) ? controller.id : controller
+      controller_id = if controller.is_a?(Radfish::Controller)
+                        controller.id
+                      elsif controller.respond_to?(:id)
+                        controller.id
+                      else
+                        controller
+                      end
       response = authenticated_request(:get, "/redfish/v1/Systems/#{SYSTEM_ID}/Storage/#{controller_id}")
       return [] unless response.status == 200
 
@@ -339,8 +373,13 @@ module Radfish
     end
 
     def volumes(controller)
-      controller_obj = controller.is_a?(Radfish::Controller) ? controller : nil
-      controller_id = controller.is_a?(Radfish::Controller) ? controller.id : controller
+      controller_id = if controller.is_a?(Radfish::Controller)
+                        controller.id
+                      elsif controller.respond_to?(:id)
+                        controller.id
+                      else
+                        controller
+                      end
       response = authenticated_request(:get, "/redfish/v1/Systems/#{SYSTEM_ID}/Storage/#{controller_id}/Volumes")
       return [] unless response.status == 200
 
@@ -351,37 +390,37 @@ module Radfish
         volume_response = authenticated_request(:get, member["@odata.id"])
         next nil unless volume_response.status == 200
         data = JSON.parse(volume_response.body)
-        Radfish::Volume.new(
-          client: self,
-          controller: controller_obj,
-          id: data["Id"],
-          name: data["Name"],
-          capacity_bytes: data["CapacityBytes"],
-          raid_type: data["RAIDType"],
-          health: data.dig("Status", "Health"),
-          adapter_data: data
-        )
+        # Return hash with normalized keys - Radfish::Client will wrap in Volume
+        # Keep the raw data for adapter_data access
+        data["id"] = data["Id"]
+        data["name"] = data["Name"]
+        data["capacity_bytes"] = data["CapacityBytes"]
+        data["raid_type"] = data["RAIDType"]
+        data["health"] = data.dig("Status", "Health")
+        data
       end.compact
     end
 
     def volume_drives(volume)
       volume_id = volume.is_a?(Radfish::Volume) ? volume.id : volume
-      # Get volume details to find linked drives
+      # Get volume details to find linked drives - adapter_data is the raw hash
       volume_data = volume.is_a?(Radfish::Volume) ? volume.adapter_data : nil
 
-      unless volume_data
+      # volume_data should be a hash with "Links" -> "Drives"
+      unless volume_data.is_a?(Hash)
         # Need to fetch volume data
         storage_controllers.each do |controller|
           vols = volumes(controller)
-          vol = vols.find { |v| v.id == volume_id }
+          # volumes() returns hashes now
+          vol = vols.find { |v| (v["id"] || v["Id"]) == volume_id }
           if vol
-            volume_data = vol.adapter_data
+            volume_data = vol
             break
           end
         end
       end
 
-      return [] unless volume_data
+      return [] unless volume_data.is_a?(Hash)
 
       drive_refs = volume_data.dig("Links", "Drives") || []
       drive_refs.map do |ref|

data/radfish-ami.gemspec

@@ -14,8 +14,6 @@ Gem::Specification.new do |spec|
   spec.license = "MIT"
   spec.required_ruby_version = ">= 3.1.0"
 
-  spec.metadata["homepage_uri"] = spec.homepage
-  spec.metadata["source_code_uri"] = spec.homepage
   spec.metadata["changelog_uri"] = "#{spec.homepage}/blob/main/CHANGELOG.md"
 
   spec.files = Dir.chdir(__dir__) do
@@ -24,7 +22,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   spec.add_dependency "radfish", "~> 0.2"
-  spec.add_dependency "ostruct", ">= 0"
+  spec.add_dependency "ostruct", "~> 0.6"
 
   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.7
+  version: 0.1.10
 platform: ruby
 authors:
 - Jonathan Siegel
@@ -28,16 +28,16 @@ dependencies:
   name: ostruct
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.6'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.6'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -113,8 +113,6 @@ homepage: https://github.com/buildio/radfish-ami
 licenses:
 - MIT
 metadata:
-  homepage_uri: https://github.com/buildio/radfish-ami
-  source_code_uri: https://github.com/buildio/radfish-ami
   changelog_uri: https://github.com/buildio/radfish-ami/blob/main/CHANGELOG.md
 post_install_message:
 rdoc_options: []