radfish-ami 0.1.11 → 0.1.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 446d9d41aafcddee32b13d9ed75c5d7e31a9e0667f004994c024d2ac3f84dc84
-  data.tar.gz: 24cd429c550b252eaae8fb96179f8cfd95e9f694cb6959e33b82332e331fcadd
+  metadata.gz: a5eece6126f4c0d048e1aedcaa23d466e77c33dd4b69595bc6b53016815c7cec
+  data.tar.gz: 9da097025394a758dc0fa4a2347ce90993113d4d0086f985534580104a2298b3
 SHA512:
-  metadata.gz: ae78d572d0836d07ec7d1f08aa639ce67c43f5bcce13f4c772fa7300cc070ab73c8cc65be2045543d6e488a762a2d5d18f6839d3ba7442b0dedac0937a0fd68f
-  data.tar.gz: 9301cf72f12f266e031de0f5b06eae07bf27d973a0b09d7ae2e44efbcd14bc15134e2370cf47735cd1256487ff0cc35aef9b42baec431688518759cd3db80ebc
+  metadata.gz: 3ec9ee1430c502c6e0364a9d345d16c67d9d459ee3ce226d8d92bafb0c1151a9bf714cb5cd94d4490da9a950ff50550d773a9062a45782b4cf6d3af8a91b8a53
+  data.tar.gz: 2a347044b3101aff4f31e47aaabf4d3bf68e752aea0e48864920b38b2a8a9df1c140fd97fd91fd6938cf9e148d75af36503069bcb10ae6471b28d9438b867add

data/lib/radfish/ami/version.rb

@@ -2,6 +2,6 @@
 
 module Radfish
   module Ami
-    VERSION = "0.1.11"
+    VERSION = "0.1.12"
   end
 end

data/lib/radfish/ami_adapter.rb

@@ -484,7 +484,10 @@ module Radfish
       end
     end
 
-    def insert_virtual_media(iso_url, device: nil)
+    def insert_virtual_media(iso_url, device: nil, username: nil, password: nil)
+      # Ensure remote media is enabled (AMI-specific)
+      ensure_remote_media_enabled!
+
       devices = virtual_media
       target_device = if device
                         devices.find { |d| d["Id"] == device || d["Name"]&.include?(device.to_s) }
@@ -498,9 +501,20 @@ module Radfish
       device_path = target_device["@odata.id"]
       actions = target_device.dig("Actions", "#VirtualMedia.InsertMedia")
 
+      # Detect transfer protocol from URL (AMI requires this)
+      transfer_protocol = detect_transfer_protocol(iso_url)
+
       if actions && actions["target"]
         # Use the InsertMedia action
-        payload = { "Image" => iso_url, "Inserted" => true, "WriteProtected" => true }
+        # AMI requires UserName/Password even for anonymous access
+        payload = {
+          "Image" => iso_url,
+          "Inserted" => true,
+          "WriteProtected" => true,
+          "TransferProtocolType" => transfer_protocol,
+          "UserName" => username || "",
+          "Password" => password || ""
+        }
         response = authenticated_request(:post, actions["target"], body: payload.to_json)
       else
         # Fallback to PATCH method
@@ -508,12 +522,20 @@ module Radfish
         response = authenticated_request(:patch, device_path, body: payload.to_json)
       end
 
+      # 200, 202, 204 are all success (202 = accepted, async operation)
       if response.status.between?(200, 204)
-        debug "Virtual media inserted successfully", 1, :green
+        debug "Virtual media insert initiated successfully", 1, :green
         true
       else
         error_msg = begin
-                      JSON.parse(response.body).dig("error", "message")
+                      data = JSON.parse(response.body)
+                      # Try to get detailed error from ExtendedInfo
+                      extended = data.dig("error", "@Message.ExtendedInfo")
+                      if extended&.any?
+                        extended.map { |e| e["Message"] }.join("; ")
+                      else
+                        data.dig("error", "message")
+                      end
                     rescue
                       response.body
                     end
@@ -521,6 +543,34 @@ module Radfish
       end
     end
 
+    # Enable remote media support (AMI-specific, required before virtual media works)
+    def enable_remote_media
+      response = authenticated_request(
+        :post,
+        "/redfish/v1/Managers/#{MANAGER_ID}/Actions/Oem/AMIVirtualMedia.EnableRMedia",
+        body: { "RMediaState" => "Enable" }.to_json
+      )
+      response.status.between?(200, 204)
+    end
+
+    # Disable remote media support (AMI-specific)
+    def disable_remote_media
+      response = authenticated_request(
+        :post,
+        "/redfish/v1/Managers/#{MANAGER_ID}/Actions/Oem/AMIVirtualMedia.EnableRMedia",
+        body: { "RMediaState" => "Disable" }.to_json
+      )
+      response.status.between?(200, 204)
+    end
+
+    # Check if remote media is enabled
+    def remote_media_enabled?
+      response = authenticated_request(:get, "/redfish/v1/Managers/#{MANAGER_ID}")
+      return false unless response.status == 200
+      data = JSON.parse(response.body)
+      data.dig("Oem", "Ami", "VirtualMedia", "RMediaStatus") == "Enabled"
+    end
+
     def eject_virtual_media(device: nil)
       devices = virtual_media
       target_device = if device
@@ -962,6 +1012,38 @@ module Radfish
         end
       end
     end
+
+    # Detect transfer protocol from URL for AMI virtual media
+    def detect_transfer_protocol(url)
+      uri = URI.parse(url)
+      case uri.scheme&.downcase
+      when 'https'
+        'HTTPS'
+      when 'nfs'
+        'NFS'
+      when 'cifs', 'smb'
+        'CIFS'
+      when 'http'
+        # AMI doesn't support plain HTTP, try HTTPS
+        'HTTPS'
+      else
+        'HTTPS'  # Default to HTTPS
+      end
+    end
+
+    # Ensure remote media is enabled (AMI-specific requirement)
+    def ensure_remote_media_enabled!
+      return if remote_media_enabled?
+
+      response = authenticated_request(
+        :post,
+        "/redfish/v1/Managers/#{MANAGER_ID}/Actions/Oem/AMIVirtualMedia.EnableRMedia",
+        body: { "RMediaState" => "Enable" }.to_json
+      )
+
+      # Wait for the action to complete
+      sleep 3 if response.status.between?(200, 204)
+    end
   end
 
   # Register the AMI adapter

metadata

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