radfish 0.2.1 → 0.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e7628a9f6e1b6d8636e8755018a7b6468a525657b4e7d0c014b7eee13b93176f
-  data.tar.gz: 5923533c409bd4a0a4976c41b063e3a8af9d8f88380089627e7aed35151077af
+  metadata.gz: 73011007c472b46cd1b41258548fdc37da25f15a4d14c370809aeafe5d67953b
+  data.tar.gz: 0b649b8aa1b6c5d05eae02641b2fef7fc6bb9d45b913fdd7727c875c69c62562
 SHA512:
-  metadata.gz: fb93b21b54e808e17a557440d029438291098f5bfffeafd7a79cde3c4353e517f1100aa01107b60c796e77ad48bf0c739d7aa5a79742cbca0e92d5d3a3c67324
-  data.tar.gz: 10f0e4b034e548b63d119b818c00a66884d865d681145aeb605fd959cd6d43c27701a556e340ec48c47a13641152c8ca594a0b28164cdb86a1a7cf10f1284ac7
+  metadata.gz: 65195d4125e3afb35707adc1b87d1370de1df0b2e19afe6a5a7e56a0ace94a8437b83f9c70d1d6ada0861074d7dede5d3ccb06c5861e1e20aec95388d79ae763
+  data.tar.gz: c99fcf649bb95dd02a861457920c310f276ec544f86fa3016771953dd277ac8a9c996add49207c59fa7f200384db570ae2b45d64abe2612663145e5b13085b98

data/lib/radfish/client.rb

@@ -152,6 +152,7 @@ module Radfish
       features << :boot if @adapter.respond_to?(:boot_options)
       features << :jobs if @adapter.respond_to?(:jobs)
       features << :utility if @adapter.respond_to?(:sel_log)
+      features << :network if @adapter.respond_to?(:get_bmc_network) || @adapter.respond_to?(:set_bmc_network)
       
       features
     end
@@ -187,18 +188,30 @@ module Radfish
     private
 
     def build_controller(raw)
-      # Only extract friendly name/id if plainly available; keep raw in adapter_data
-      id = if raw.respond_to?(:[])
-             raw['id'] || raw[:id]
-           elsif raw.respond_to?(:id)
-             raw.id
-           end
-      name = if raw.respond_to?(:[])
-               raw['name'] || raw[:name]
-             elsif raw.respond_to?(:name)
-               raw.name
-             end
-      Controller.new(client: self, id: id, name: name, vendor: @vendor, adapter_data: raw)
+      # Expect adapters to provide normalized fields; keep mapping straightforward.
+      fetch = ->(key) do
+        if raw.respond_to?(:[])
+          raw[key.to_s] || raw[key.to_sym]
+        elsif raw.respond_to?(key.to_sym)
+          raw.public_send(key.to_sym)
+        end
+      end
+
+      Controller.new(
+        client: self,
+        id: fetch.call(:id),
+        name: fetch.call(:name),
+        model: fetch.call(:model),
+        firmware_version: fetch.call(:firmware_version),
+        encryption_mode: fetch.call(:encryption_mode),
+        encryption_capability: fetch.call(:encryption_capability),
+        controller_type: fetch.call(:controller_type),
+        pci_slot: fetch.call(:pci_slot),
+        status: fetch.call(:status),
+        drives_count: fetch.call(:drives_count),
+        vendor: @vendor,
+        adapter_data: raw
+      )
     end
   end
 end

data/lib/radfish/controller.rb

@@ -4,14 +4,26 @@ module Radfish
   # Lightweight value object representing a storage controller.
   # Holds a stable identifier and optional name.
   class Controller
-    attr_reader :id, :name, :vendor, :adapter_data
+    attr_reader :id, :name, :model, :vendor, :adapter_data, :firmware_version,
+                :encryption_mode, :encryption_capability, :controller_type,
+                :pci_slot, :status, :drives_count
 
-    def initialize(client:, id:, name: nil, vendor: nil, adapter_data: nil)
+    def initialize(client:, id:, name: nil, model: nil, vendor: nil, adapter_data: nil,
+                   firmware_version: nil, encryption_mode: nil, encryption_capability: nil,
+                   controller_type: nil, pci_slot: nil, status: nil, drives_count: nil)
       @client = client
       @id = id
       @name = name
+      @model = model
       @vendor = vendor
       @adapter_data = adapter_data
+      @firmware_version = firmware_version
+      @encryption_mode = encryption_mode
+      @encryption_capability = encryption_capability
+      @controller_type = controller_type
+      @pci_slot = pci_slot
+      @status = status
+      @drives_count = drives_count
     end
 
     # Convenience accessors delegate to the client wrappers
@@ -24,11 +36,24 @@ module Radfish
     end
 
     def to_h
-      { id: id, name: name, vendor: vendor }
+      {
+        id: id,
+        name: name,
+        model: model,
+        vendor: vendor,
+        firmware_version: firmware_version,
+        encryption_mode: encryption_mode,
+        encryption_capability: encryption_capability,
+        controller_type: controller_type,
+        pci_slot: pci_slot,
+        status: status,
+        drives_count: drives_count
+      }.compact
     end
 
     def ==(other)
       other.is_a?(Controller) && other.id == id && other.vendor.to_s == vendor.to_s
     end
+
   end
 end

data/lib/radfish/core/storage.rb

@@ -18,6 +18,7 @@ module Radfish
       def storage_summary
         raise NotImplementedError, "Adapter must implement #storage_summary"
       end
+
     end
   end
 end

data/lib/radfish/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Radfish
-  VERSION = "0.2.1"
+  VERSION = "0.2.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: radfish
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.3
 platform: ruby
 authors:
 - Jonathan Siegel